For the sake of completeness, also fix the same issue in policycoreutils (for the currently-disabled target mcstrans): --- selinux-13092011-rebuild2/policycoreutils/mcstrans/man/Makefile 2011-09-09 20:12:56.036662576 +0200 +++ selinux-13092011-rebuild/policycoreutils/mcstrans/man/Makefile 2011-09-14 03:11:39.994807156 +0200 @@ -1,6 +1,8 @@ # Installation directories. MAN8DIR ?= $(DESTDIR)/usr/share/man/man8 +all: + install: mkdir -p $(MAN8DIR) install -m 644 man8/*.8 $(MAN8DIR) On Wed, 2011-09-14 at 03:10 +0200, Guido Trentalancia wrote: > And similarly for the "include" and "man" directories available for the > libraries: > > Change the default "make" target for the libraries from "install" to > "all" in *all* makefiles. > > --- selinux/libselinux/include/Makefile 2011-09-09 20:12:55.981662183 +0200 > +++ selinux-13092011-rebuild/libselinux/include/Makefile 2011-09-14 02:48:03.159322088 +0200 > @@ -2,6 +2,8 @@ > PREFIX ?= $(DESTDIR)/usr > INCDIR ?= $(PREFIX)/include/selinux > > +all: > + > install: > test -d $(INCDIR) || install -m 755 -d $(INCDIR) > install -m 644 $(wildcard selinux/*.h) $(INCDIR) > --- selinux/libsepol/include/Makefile 2011-09-09 20:12:56.018662447 +0200 > +++ selinux-13092011-rebuild/libsepol/include/Makefile 2011-09-14 02:48:39.183551143 +0200 > @@ -2,6 +2,8 @@ > PREFIX ?= $(DESTDIR)/usr > INCDIR ?= $(PREFIX)/include/sepol > > +all: > + > install: > test -d $(INCDIR) || install -m 755 -d $(INCDIR) > test -d $(INCDIR)/policydb || install -m 755 -d $(INCDIR)/policydb > --- selinux/libsemanage/include/Makefile 2011-09-09 20:12:56.001662322 +0200 > +++ selinux-13092011-rebuild/libsemanage/include/Makefile 2011-09-14 02:48:23.919454058 +0200 > @@ -2,6 +2,8 @@ > PREFIX ?= $(DESTDIR)/usr > INCDIR ?= $(PREFIX)/include/semanage > > +all: > + > install: > test -d $(INCDIR) || install -m 755 -d $(INCDIR) > install -m 644 $(wildcard semanage/*.h) $(INCDIR) > --- selinux-13092011-rebuild2/libselinux/man/Makefile 2011-09-14 03:04:28.573741936 +0200 > +++ selinux-13092011-rebuild/libselinux/man/Makefile 2011-09-14 02:56:26.626528998 +0200 > @@ -4,6 +4,8 @@ MAN3DIR ?= $(PREFIX)/share/man/man3 > MAN5DIR ?= $(PREFIX)/share/man/man5 > MAN8DIR ?= $(PREFIX)/share/man/man8 > > +all: > + > install: > mkdir -p $(MAN3DIR) > mkdir -p $(MAN5DIR) > --- selinux-13092011-rebuild2/libsepol/man/Makefile 2011-09-14 03:04:36.648799423 +0200 > +++ selinux-13092011-rebuild/libsepol/man/Makefile 2011-09-14 02:57:38.237986215 +0200 > @@ -3,6 +3,8 @@ PREFIX ?= $(DESTDIR)/usr > MAN3DIR ?= $(PREFIX)/share/man/man3 > MAN8DIR ?= $(PREFIX)/share/man/man8 > > +all: > + > install: > mkdir -p $(MAN3DIR) > mkdir -p $(MAN8DIR) > --- selinux-13092011-rebuild2/libsemanage/man/Makefile 2011-09-14 03:04:43.361847205 +0200 > +++ selinux-13092011-rebuild/libsemanage/man/Makefile 2011-09-14 02:57:24.475898336 +0200 > @@ -3,6 +3,8 @@ PREFIX ?= $(DESTDIR)/usr > MAN3DIR ?= $(PREFIX)/share/man/man3 > MAN5DIR ?= $(PREFIX)/share/man/man5 > > +all: > + > install: > mkdir -p $(MAN3DIR) > mkdir -p $(MAN5DIR) > > On Wed, 2011-09-14 at 02:44 +0200, Guido Trentalancia wrote: > > And for the reason mentioned earlier (do not install unless explicitly > > required to do so): > > > > Change the default "make" target for sepolgen from "install" to "all". > > > > --- selinux/sepolgen/src/sepolgen/Makefile 2011-09-09 20:12:56.080662897 +0200 > > +++ selinux-13092011-rebuild/sepolgen/src/sepolgen/Makefile 2011-09-14 02:30:20.172601736 +0200 > > @@ -1,7 +1,9 @@ > > PYTHONLIBDIR ?= $(shell python -c "from distutils.sysconfig import *; print get_python_lib(1)") > > PACKAGEDIR ?= $(DESTDIR)/$(PYTHONLIBDIR)/sepolgen > > > > -install: > > +all: > > + > > +install: all > > -mkdir -p $(PACKAGEDIR) > > install -m 644 *.py $(PACKAGEDIR) > > > > --- selinux/sepolgen/src/share/Makefile 2011-09-09 20:12:56.082662911 +0200 > > +++ selinux-13092011-rebuild/sepolgen/src/share/Makefile 2011-09-14 02:36:02.173755672 +0200 > > @@ -1,8 +1,10 @@ > > SHAREDIR ?= $(DESTDIR)/var/lib/sepolgen > > > > -install: > > +all: > > + > > +install: all > > -mkdir -p $(SHAREDIR) > > install -m 644 perm_map $(SHAREDIR) > > > > clean: > > > > 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. > > > > > > -- > 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. > -- 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.
