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

Oswald Buddenhagen ossi at kde.org
Mon Dec 1 23:55:24 GMT 2008


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,
kde definitely MUST NOT (yay, RFC terminology) write it to potentially
shareable .desktop files. that was alread fixed (maybe even multiple
times), but somehow it got re-enabled directly or indirectly, or this
particular case was not covered in the first place.

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Confusion, chaos, panic - my work here is done.




More information about the kde-core-devel mailing list