[apparmor] [PATCH v2 2/6] utils: Initial implementation of aa-exec in C

[Tyler Hicks]

On 2015-12-18 00:41:49, Christian Boltz wrote:
> Hello,
> 
> Am Mittwoch, 16. Dezember 2015 schrieb Tyler Hicks:
> > Create a simple aa-exec implementation, written in C, matching the
> > --help, --debug, --verbose, and --profile options present in the
> > current Perl implementation.
> > 
> > The new aa-exec sources reside in the binutils/ directory.
> > 
> > Signed-off-by: Tyler Hicks <tyhicks at canonical.com>
> 
> 
> > diff --git a/binutils/Makefile b/binutils/Makefile
> > index 3b99c3e..aec2d62 100644
> > --- a/binutils/Makefile
> > +++ b/binutils/Makefile
> > @@ -50,7 +50,7 @@ EXTRA_CFLAGS+=-DPACKAGE=\"${NAME}\"
> > -DLOCALEDIR=\"${LOCALEDIR}\"
> > 
> >  SRCS = aa_enabled.c
> >  HDRS =
> > -TOOLS = aa-enabled
> > +TOOLS = aa-enabled aa-exec
> > 
> >  AALIB = -Wl,-Bstatic -lapparmor  -Wl,-Bdynamic -lpthread
> > 
> > @@ -106,7 +106,7 @@ all:	arch indep
> > 
> >  .PHONY: coverage
> >  coverage:
> > -	$(MAKE) clean aa-enabled COVERAGE=1
> > +	$(MAKE) clean aa-enabled aa-exec COVERAGE=1
> 
> better use
> +	$(MAKE) clean $(TOOLS) COVERAGE=1
> to make the Makefile self-maintaining (you'll only need to update TOOLS= 
> if we add a new C-based tool)
> 
> >  .SILENT: tests
> > -tests: aa-enabled $(TESTS)
> > +tests: aa-enabled aa-exec $(TESTS)
> 
> same here:
> +tests: $(TOOLS) $(TESTS)
> 
> 
> Feel free to include those changes in the commit - and keep the Ack from 
> John ;-)

Nice catches!

Tyler

> 
> 
> Regards,
> 
> Christian Boltz
> -- 
> Hochleistungswebspace
> Das sind public-html-Verzeichnisse, die jeden Morgen zwanzig Liegestütze
> machen, und mit Testosteron vollgepumpt sind.  [Markus Schaber]
> 
> 
> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
-------------- 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/20151217/0ceb6d78/attachment.pgp>