[Patch] fix compile kdeui with msvc
Ralf Habacker
ralf.habacker at freenet.de
Mon Oct 31 14:27:05 GMT 2005
Am Sonntag, 30. Oktober 2005 19:50 schrieb Thiago Macieira:
> Ralf Habacker wrote:
> >This isn't easy as it looks in the first case and have to worked out
> > more.
> >
> >For example if you are compiling libDCOP -DMAKE_DCOP_LIB should be
> > defined. If you compile dcopserver it should not although it is in the
> > same dir.
> >
> >If you compile a shared library in a subdir, which depends on another
> > lib in the same subdir, i can no image when to set this define or not
> > automatically.
> >
> >I think setting it manually it currently the only way.:-)
>
> It can't be that difficult.
>
> If you're building $(lib_prefix)FOO.$(lib_suffix), you define
> MAKE_FOO_LIB. If you're not building it, you don't define it.
>
> So dcopserver doesn't define MAKE_DCOP_LIB, even though it is in the same
> dir. And convenience libraries vanish, being replaced by simple
> compilation of the objects into the final library, even if they are in
> different dirs.
>
> (e.g., kdecore/network/*.cpp are compiled just as if they were in kdecore/
> itself)
okay.
> Convenience libraries are supported in a handful of platforms anyways, so
> we solve the problem by getting rid of them entirely.
Append is a patch for this, does anyhave see any problem ?
One question is if this should be activated by default ?
Index: generic.py
===================================================================
--- generic.py (revision 476063)
+++ generic.py (working copy)
@@ -285,6 +285,9 @@
if self.libprefix != '' and self.target[:len(self.libprefix)]
== self.libprefix:
self.target = self.target[len(self.libprefix):]
+ if self.type in ['shlib','kioslave','module']:
+ self.env.AppendUnique( CCFLAGS =
['-DMAKE_'+self.target.upper()+'_LIB'] )
+
if not self.p_localtarget: self.p_localtarget =
self.joinpath(self.target)
if not self.p_localsource: self.p_localsource =
self.joinpath(self.env.make_list(self.source))
More information about the kde-core-devel
mailing list