GNU bug report logs - #20666
[GNULIB v2 0/2] Improved acl handling

Previous Next

Full log

View this message in rfc822 format

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: andreas.gruenbacher <at> gmail.com, bug-gnulib <at> gnu.org, 20666 <at> debbugs.gnu.org
Subject: bug#20666: [GNULIB v2 2/2] qacl: Reimplement qset_acl and qcopy_acl
Date: Wed, 27 May 2015 11:50:41 -0700

On 05/26/2015 01:53 PM, Andreas Gruenbacher wrote:


> ---
>   lib/acl-internal.c    |  30 ++

This one is missing a patch to ChangeLog.  Please put the commit message 
into the ChangeLog.  Also, please put the string "Bug#20666" somewhere 
into the commit message body and the ChangeLog (they should be 
essentially identical).  I forgot, the "Bug#20667" should also be put 
into the other commit message and ChangeLog.

> +void free_permission_context (struct permission_context *ctx)

Please put the function name in column 1, with the type in the previous 
line.

>   Makefile.am:
> -lib_SOURCES += acl-errno-valid.c acl-internal.c qcopy-acl.c qset-acl.c
> +lib_SOURCES += acl-errno-valid.c acl-internal.c qcopy-acl.c qset-acl.c get-permissions.c set-permissions.c
>

Please break it into two lines (OK to use \ at the end of the 1st line) 
so that it's not longer than 80 characters.

Other than that, both patches look good to me; please install.  And thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.