On Fri, 2011-09-09 at 17:37 -0400, Joshua Brindle wrote: > Guido Trentalancia wrote: > > -all install relabel clean test indent: > > +all relabel test indent: > > @for subdir in $(SUBDIRS); do \ > > - (cd $$subdir&& $(MAKE) $@) || exit 1; \ > > + (cd $$subdir&& $(MAKE) $@&& $(MAKE) install) || exit 1; \ > > + done > > + @for subdir_bin in $(SUBDIRS_BIN); do \ > > + (cd $$subdir_bin&& $(MAKE) $@) || exit 1; \ > > + done > > + > > +install: > > + @for subdir in $(SUBDIRS); do \ > > + (cd $$subdir&& $(MAKE) install) || exit 1; \ > > + done > > + @for subdir_bin in $(SUBDIRS_BIN); do \ > > + (cd $$subdir_bin&& $(MAKE) install) || exit 1; \ > > done > > > > install-pywrap swigify: > > @@ -17,6 +29,14 @@ install-pywrap swigify: > > (cd $$subdir&& $(MAKE) $@) || exit 1; \ > > done > > > > Nack. We can't require install in the build target and bail if it fails, this is > completely unfriendly to developers who don't want to install all the libraries > on their system during development, and those are the people using git. > > If you are doing work on libselinux and you install it to your system before > testing it locally you can easily break your machine and have to boot from a > rescue disk. Yes, sure, 100% agreed. It was clearly stated how dirty that was... It's just good for a laugh ;-) At the end, you can't really install anything without the user knowing (and explicitly requiring) that. But something like the other two solutions would probably be nice... Regards, Guido -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
