The package seems to be building examples during "make all" phase even though they are not installed at all. For details see my blog: http://blog.flameeyes.eu/articles/2008/06/11/please-dont-build-your-examples-by-default . It would be nice if the examples were not built by default and (maybe) only built with FEATURES=test (as a able-to-link test). You can find an example patch usually suitable for upstream on the blog above. Thanks, Diego
(In reply to comment #0) > The package seems to be building examples during "make all" phase even though > they are not installed at all. For details see my blog: > http://blog.flameeyes.eu/articles/2008/06/11/please-dont-build-your-examples-by-default I completely agree with this idea[l]. I found this unnecessary compilation a problem while trying to crosscompile dev-libs/confuse for mingw32. The examples and tests for confuse include usage of things like SIGUSR1 and SIGHUP that mingw32 doesn't define. Thus, I couldn't emerge confuse to $SYSROOT because make bailed out when GCC spat out errors for the examples. > > It would be nice if the examples were not built by default and (maybe) only > built with FEATURES=test (as a able-to-link test). I'm pretty sure that the files in examples/ are not used by make check. > You can find an example > patch usually suitable for upstream on the blog above. I submitted a patch to upstream via the mailing list ( http://lists.nongnu.org/archive/html/confuse-devel/2009-02/msg00002.html ). I haven't gotten any response yet, though. In making the patch, I got stuck for a few hours figuring out why my variable assignment didn't work (I prefixed the variable name with a tab instead of spaces to indent it which kept make from assigning the variable :-) ). The patch works for dev-libs/confuse-2.6 as well as the CVS version.
Created attachment 183640 [details, diff] improves ebuild Adds lines to run epatch ${P}-noexample.patch Also adds dodoc lines to actually install the examples for the user - they are examples, after all :-) (though maybe I should have dodoc'd the directory rather than the individual files?)
Created attachment 183644 [details, diff] similar to flameeye's sample patch
This bug is now fixed by the ebuild version bump requested in bug 306295. Please close it. From dev-libs/confuse/ChangeLog: *confuse-2.7 (19 Mar 2010) 19 Mar 2010; MATSUU Takuto <matsuu@gentoo.org> +confuse-2.7.ebuild: Version bumped. the ebuild was submitted by Nathan Phillip Brink, bug #306295.
Noone to mark this 2-year-old bug as resolved?
Aye this is fixed in the latest version in main tree.
