qt-copy patch for QDesktopServices to allow path kcm to work

Aaron J. Seigo aseigo at kde.org
Tue Dec 2 00:47:16 GMT 2008


On Monday 01 December 2008, Oswald Buddenhagen wrote:
> On Mon, Dec 01, 2008 at 04:45:38PM -0700, Aaron J. Seigo wrote:
> > On Monday 01 December 2008, Thiago Macieira wrote:
> > > Anne-Marie Mahfouf wrote:
> > > >Is it OK to add it? All other ways to fix this bug would add more
> > > >code and I don't see a reasonable way anyway ;)
> > >
> > > No, I don't think so. This patch will be rejected in Qt.
> > >
> > > KDE should be fixed to not add the [$e] because nothing other than
> > > KDE will be able to read it.
> >
> > nothing else can read it because even when offered patches they reject
> > them.  *cough*
> >
> > what [$e] and [$i] provides is rather useful, and it would seem to
> > make sense to either propose an alternative or adopt what is really an
> > improvement.
>
> yes, but the actual implementation is non-trivial (at least the full one
> including $() syntax). apart from that, given that it is non-standard,

it's non-standard but ... it's a standard we created (ostensibly, anyways). 
but then we say "we can't do it because we have to stick within this standard" 
rather than try and improve that standard. seems a bit daft not to?

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20081201/32b20aae/attachment.sig>


More information about the kde-core-devel mailing list