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 <[email protected]>
> 
> 
> > 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
> [email protected]
> Modify settings or unsubscribe at: 
> https://lists.ubuntu.com/mailman/listinfo/apparmor

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to