Created attachment 276937 [details] Build log Hello there! You're getting this bug because the package in Summary failed to build in my tinderbox using the gold link editor from binutils. Before closing the bug as INVALID let me explain why this is still important! The gold link editor does not support underlinking of shared objects, which is something I have described in my blog post: http://blog.flameeyes.eu/2010/11/26/it-s-not-all-gold-that-shines-why-underlinking-is-a-bad-thing Even the basic link editor (ld.bfd) has an option to support this but it is a heck to enable and get passed, so linking with gold is simply quicker. Fixing underlinking provides optimized --as-needed builds (because the "softer" version no longer need to recover libraries that are underlinked), so it is a Good Thing To Do. Thank you very much for the attention!
CMakeFiles/kmuddyscriptingplugin.dir/cunixsocket.o:cunixsocket.cpp:function cUnixSocket::cUnixSocket(int, cRunningScript*): warning: the use of `tempnam' is dangerous, better use `mkstemp' /usr/lib/gcc/i686-pc-linux-gnu/4.6.0/../../../../i686-pc-linux-gnu/bin/ld: error: treating warnings as errors collect2: ld returned 1 exit status make[2]: *** [lib/kmuddyscriptingplugin.so] Error 1 Different problem...
... and this is not the only problem, also displays underlinking issues. As no users are cc'ed here at all, I suggest treecleaning.
Note that the version in git builds here - not that there is an official 9999 build, but I created one myself and it still builds. Perhaps this has been fixed in git? Perhaps we could add a -9999 build for kmuddy to hold over until there is a newer official release? (And the package.mask file has the wrong bug number attached :-) )
(not that I used gold - but if it is fixed in git, I'd hope to keep this around.)
the underlinking is not as serious as the tempnam problem (but it needs to be fixed as well in the long term). If you want to save this package then you need to step up and maintain it and provide a working patch as well. -9999 ebuild is not an appropriate solution (at least for portage inclusion)
Created attachment 295301 [details, diff] fix tempnam usage Please consider keeping this package. Old behaviour: write socket to /tmp/$tempnam New behaviour: write socket to /tmp/$mkdtemp/sock
Created attachment 295551 [details, diff] kmuddy zlib underlinking fix This patch should fix underlinking of cmccp.cpp using inflateEnd from zlib. This change is already in upstream's (so far unreleased) git. Unfortunately I do not have gold installed to test this fix. Since there's patches for both problems reported please consider not tree-cleaning the package just yet. Thanks!
I'll have a look during the next days.
All fixed and package mask removed (package is ~arch for now). Thank you for the patches!
