Patches for KDE libs and base on solaris

[Simon Hausmann]

On Tue, Nov 12, 2002 at 10:47:29AM +0100, David Faure wrote:
> On Tuesday 12 November 2002 09:54, Simon Hausmann wrote:
> > Index: kdecore/kdesktopfile.h
> > ===================================================================
> > RCS file: /home/kde/kdelibs/kdecore/kdesktopfile.h,v
> > retrieving revision 1.30
> > diff -u -r1.30 kdesktopfile.h
> > --- kdecore/kdesktopfile.h	2002/09/28 12:28:47	1.30
> > +++ kdecore/kdesktopfile.h	2002/11/12 04:15:23
> > @@ -115,7 +115,6 @@
> >    /**
> >     * Returns the value of the "Path=" entry.
> >     * @return the path or QString::null if not specified
> > -   * @deprecated
> >     */
> >    QString readPath() const;
> > 
> > Agreed. Hell knows why this one was marked deprecated :)
> 
> Because we don't use it anywhere?
> Well, that's the subject of another discussion....
> KDE-1: Application .desktop files could have a Path= entry, which was used as
> the "current directory" (we chdir there before launching the app).
> KDE-2,3: The support for this was removed (by me I think), under the assumption
> that no GUI app had anything to do with the cwd - for sure KDE apps don't need
> this. However later discussions/bugreports showed that _some_ apps do care 
> about the cwd. (I personnally consider this quite broken, for GUI apps, but 
> who am I to change the world....)
> Therefore maybe we have to readd support for this. But since it now involves
> kdeinit/klauncher.... I'd happily let Waldo do that :)

Ahhh, ok, I missed that. I just saw that cvs ann told me that this
was added like two years ago and it looked like a stale entry to me.

> > Index: kio/kssl/Makefile.am
> > ===================================================================
> > RCS file: /home/kde/kdelibs/kio/kssl/Makefile.am,v
> > retrieving revision 1.27
> > diff -u -r1.27 Makefile.am
> > --- kio/kssl/Makefile.am	2002/05/25 10:58:51	1.27
> > +++ kio/kssl/Makefile.am	2002/11/12 04:15:46
> > @@ -1,4 +1,4 @@
> > -INCLUDES=$(all_includes) $(SSL_INCLUDES)
> > +INCLUDES=$(SSL_INCLUDES) $(all_includes)
> > 
> > Good catch.
> 
> Breaks with openssl is in /usr/include and the distro's KDE is there too
> (but the current "devel" KDE prefix is somewhere else) ?
> I know, one can always think of the reverse situation - but we're developing
> KDE, no openssl, so we're more likely to have two versions of KDE
> installed, than two versions of openssl.

Oops, ok, I missed that point.


Simon