conditional building of examples

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

conditional building of examples

ohnobinki
I was wondering if building of the files in examples/ could be made
optional. I'd like to have a ./configure --disable-examples option for
two reasons:
1. When crosscompiling confuse for mingw32, the examples don't compile
because SIGUSR*, SIGHUP, etc. aren't defined
2. Sometimes the package is installed using a package manager such as
Gentoo's Portage. The examples--which are noinst_PROGRAMS that are not
even used for make check or for building the parts of libconfuse--are
compiled, never run, and then thrown away after running make install.
This wastes a small amount of time for the user which, given confuse's
short compile time, becomes somewhat significant.

I have attached a patch (which hopefully isn't eaten by the ML) that
adds the --enable-examples option to ./configure. It defaults to
enabling the compilation of examples so that those who download and
compile the source can easily play with the examples. It has worked on
my system to avoid the compilation of the examples--allowing make to
return a success while crosscompiling to mingw32.

Please respond with any suggestions or thoughts about my
patch/methodology/reasons.


_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel

confuse_conditional_examples.cvs.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: conditional building of examples

ohnobinki
Nathan Brink wrote:
>
> Please respond with any suggestions or thoughts about my
> patch/methodology/reasons.
>
Oops! Please remove the following line from the patch:

+echo enable_examples=$enable_examples


I've attached the fixed patch.

_______________________________________________
Confuse-devel mailing list
[hidden email]
http://lists.nongnu.org/mailman/listinfo/confuse-devel

confuse_conditional_examples.cvs.patch (1K) Download Attachment