KDE/kdelibs

[LiuCougar]

On 10/12/05, Ralf Habacker <ralf.habacker at freenet.de> wrote:
> Am Mittwoch, 12. Oktober 2005 04:24 schrieben Sie:
> > SVN commit 469699 by liucougar:
> >
> >   # better dummy.cpp support
> >   # updated amtool.py to read libraries/programs/headers/includes
> >   # new am2bksys.py module: use
> >         python bksys/am2bksys.py
> >         to run it
> >   # now subdirs: kparts, kutils, kdeprint, khtml, kde3support compilable
> >
> great work. I have some little issues, see below
>
> >
> >  A             khtml/css/SConscript
> >  A             khtml/dom/SConscript
> >  A             khtml/ecma/SConscript
> >  A             khtml/html/SConscript
> >  A             khtml/imload/SConscript
> >  A             khtml/imload/decoders/SConscript
> >  A             khtml/java/SConscript
> >  A             khtml/misc/SConscript
>
>
> > includes = '#/dcop #/kio #/libltdl #/khtml #/kutils '
>
> I think it would be better not to refer the build  dir instead the dirs in
> kdelibs should be referenced
>
> includes = 'dcop kio libltdl khtml kutils '
AFAICT (from generic.py fixpath function), without leading #, both
build and source dir will be included (not the opposite)

> obj = env.kdeobj('convenience',env)
> obj.target = 'libkhtmlmisc'
>                     ^^^
> the lib prefix is platform specific and should be avoided

Right, I will rectify the script

Regards

--
"People's characters are strengthened through struggle against
difficulties; they are weakened by comfort."
- Old Chinese adage