On Wed, Jan 28, 2015 at 10:13:53PM +0100, Christian Boltz wrote:
> parser/Makefile also has a dist_clean:

Sigh, so much cruft.

> Note that "dist_clean" deletes files that are already deleted by 
> "clean", which means "dist_clean" is basically superfluous ;-)
> 
> Assuming that nobody uses dist_clean, the obvious patch is:
> 
> [ parser_Makefile_drop_dist_clean.diff ]
> 
> === modified file 'parser/Makefile'
> --- parser/Makefile     2015-01-23 23:52:09 +0000
> +++ parser/Makefile     2015-01-28 21:11:51 +0000
> @@ -370,7 +370,3 @@
>         $(MAKE) -s -C po clean
>         $(MAKE) -s -C tst clean
>  
> -.SILENT: dist_clean
> -dist_clean:
> -       @$(MAKE) clean
> -       @rm -f $(LEX_C_FILES) $(YACC_C_FILES)

Acked-by: Steve Beattie <[email protected]>. Thanks!

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

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