On Fri, Aug 23, 2002 at 02:48:44PM -0700, Waldo Bastian wrote:
> - From the kdelibs patch I have commited the misc stuff.

Great!  Thanks.

> Issue 1.
> =======
> Can someone comment on the correctness of the following patch for
> kdecore/Makefile.am:
> 
> - -libkdecore_la_LIBADD = malloc/libklmalloc.la ../dcop/libDCOP.la 
> ../libltdl/libltdlc.la $(LIB_XEXT) $(LIBRESOLV) $(LIBXINERAMA) $(LIB_DMALLOC) 
> $(LIBART_LIBS) $(SVGICON_LIB)
> +libkdecore_la_LIBADD = malloc/libklmalloc.la ../dcop/libDCOP.la 
> ../libltdl/libltdlc.la $(LIB_XEXT) $(LIBRESOLV) $(LIBXINERAMA) $(LIB_DMALLOC) 
> $(SVGICON_LIB) $(LIBART_LIBS) $(LIB_QT)
> 
> Why does that need $(LIB_QT) ?

I'm not sure, that one seemed odd to me too.  Would an unsupported
version of automake or autoconf cause this sort of problem?  Should I
be running versions older than the latest?

> Issue 2.
> =======
> In kdeprint/cups/Makefile.am, instead of:
> 
> - -make_driver_db_cups_LDADD = -lz ../libdriverparse.a 
> $(top_builddir)/kdecore/libkdefakes.la $(LIBDL)
> +make_driver_db_cups_LDADD = ../libdriverparse.a 
> $(top_builddir)/kdecore/libkdefakes.la $(LIBDL) -L/usr/local/lib -lz
> 
> Does the following patch work for you as well?
> - -make_driver_db_cups_LDADD = -lz ../libdriverparse.a 
> $(top_builddir)/kdecore/libkdefakes.la $(LIBDL)
> +make_driver_db_cups_LDADD = ../libdriverparse.a 
> $(top_builddir)/kdecore/libkdefakes.la $(LIBDL) $(LIBZ)
> +make_driver_db_cups_LDFLAGS = $(all_libraries)

I'm sure this would work too, as long as the link path is included
somewhere.

> Issue 3.
> =======
> Can someone comment on the namespace issues?

Similar changes have been applied elsewhere, but I understand there's
a patch to Qt's moc that works around the problem as well.  If Troll
Tech were to conditionally enable that code on IRIX platforms (running
<= 7.3 versions of the compilers), none of the KDE workarounds would
be required.  I think a post on one of the earlier KDE on IRIX threads
contains the patch.

Thanks,
Jesse
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<