[apparmor] [patch] more Make.rules cruft removal

Steve Beattie steve at nxnw.org
Sat Jan 24 01:23:32 UTC 2015


On Fri, Jan 23, 2015 at 09:53:43PM +0100, Christian Boltz wrote:
> > Index: b/Makefile
> > ===================================================================
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1,8 +1,7 @@
> >  #
> >  #
> > -OVERRIDE_TARBALL=yes
> > -
> > -include common/Make.rules
> > +COMMONDIR=common
> > +include ${COMMONDIR}/Make.rules
> > 
> >  DIRS=parser \
> >       profiles \
> 
> See previous mail - this should be commited as part of the "cleanup the 
> 'common' mess" patch.

I moved the COMMONDIR definition to the previous patch, but the
OVERRIDE_TARBALL bit needed to go with the rest of this patch.

> parser/Makefile is the only remaining user of DISTRO, so maybe we should 
> move it there in a follow-up patch.

I kind of want it to go away; it's only use is to determine which
initscript to install, which Debian and Ubuntu don't use.

> >  .PHONY: version
> >  .SILENT: version
> >  version:
> > -	 rpm -q --define "_sourcedir ." ${RPMARG} --specfile ${SPECFILE}
> > +	echo $(VERSION)
> 
> Is the "version" target used somewhere? grep didn't bring up anything 
> even in the current Makefiles, however "version" is a too common term so 
> that I might have overlooked something.

No, it's not used by any other target. It exists for debugging,
to ensure that VERSION is set properly at any given point (i.e. I
can do 'make version' and see a result that I expect). Given that
as you pointed out, the prior implementation wouldn't have worked on
debian/ubuntu (and wouldn't work anywhere without a specfile to query),
it's not crucial to keep. The 'make repo_version' target exists for
the same reason.

> grep -r REPO_VERSION brings up some interesting results in 
> libappparmor/Makefile.am - read and laugh yourself about what you'll 
> find there, and then please send another cleanup patch ;-)

Ohhhh, it needs to die with fire, it does. Patch follows, that also
converts the libapparmor/doc/ clean to get rid of the manpages on make
clean, not make maintainer-clean.

Thanks.

Signed-off-by: Steve Beattie <steve at nxnw.org>
---
 libraries/libapparmor/Makefile.am     |   18 ------------------
 libraries/libapparmor/doc/Makefile.am |    4 ++--
 2 files changed, 2 insertions(+), 20 deletions(-)

Index: b/libraries/libapparmor/Makefile.am
===================================================================
--- a/libraries/libapparmor/Makefile.am
+++ b/libraries/libapparmor/Makefile.am
@@ -4,22 +4,4 @@ SRCDIR = src
 
 SUBDIRS = doc src include swig testsuite
 
-REPO_VERSION=$(shell if [ -x /usr/bin/svn ] ; then \
-	/usr/bin/svn info . 2> /dev/null | grep "^Last Changed Rev:" | sed "s/^Last Changed Rev: //" ; \
-	fi)
-
-REPO_URL=$(shell if [ -x /usr/bin/svn ] ; then \
-	/usr/bin/svn info . 2> /dev/null | grep "^URL:" | sed "s/^URL: //" ; \
-	fi)
-RELEASE_DIR     = $(NAME)-$(VERSION)-${REPO_VERSION}
-
-SVNTARBALL = $(NAME)-$(VERSION)-${REPO_VERSION}.tar.gz
-SVNTAR = /bin/tar czvp -h --exclude .svn --exclude CVS --exclude .cvsignore --exclude ${SVNTARBALL} --exclude ${RELEASE_DIR}/${RELEASE_DIR}  $(shell test -f ${NAME}.exclude && echo "-X ${NAME}.exclude")
-
-distball: clean
-	rm -rf $(RELEASE_DIR)
-	svn export -r $(REPO_VERSION) $(REPO_URL) $(RELEASE_DIR)
-	$(SVNTAR) -f $(SVNTARBALL) $(RELEASE_DIR)
-	rm -rf $(RELEASE_DIR)
-	
 EXTRA_DIST = AUTHORS ChangeLog COPYING.LGPL INSTALL NEWS README
Index: b/libraries/libapparmor/doc/Makefile.am
===================================================================
--- a/libraries/libapparmor/doc/Makefile.am
+++ b/libraries/libapparmor/doc/Makefile.am
@@ -11,8 +11,8 @@ PODS = $(subst .2,.pod,$(man_MANS))
 
 EXTRA_DIST = $(man_MANS) $(PODS)
 
-## delete man pages at maintainer-clean
-BUILT_SOURCES = $(man_MANS)
+## delete man pages at make clean
+CLEANFILES = $(man_MANS)
 
 %.2: %.pod
 	$(PODCHECKER) -warnings -warnings $<

-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20150123/052a7308/attachment.pgp>


More information about the AppArmor mailing list