Thanks, applying with some fixes: On Tue, Mar 26, 2019 at 07:27:40PM +0530, Kenneth D'souza wrote: > With this patch the filename will not be printed. > > $ nfs4_getfacl /test/ > # file: /test/ > A::OWNER@:rwaDxtTcCy > A::GROUP@:rwaDxtcy > A::EVERYONE@:rwaDxtcy > > $ nfs4_getfacl -c /test/ > > A::OWNER@:rwaDxtTcCy > A::GROUP@:rwaDxtcy > A::EVERYONE@:rwaDxtcy I don't see any reason for that blank line, I've removed it. > @@ -115,7 +119,10 @@ static void print_acl_from_path(const char *fpath) > struct nfs4_acl *acl; > acl = nfs4_acl_for_path(fpath); > if (acl != NULL) { > + if(ignore_comment == 0) > printf("\n# file: %s\n", fpath); This needs to be indented. > + else > + printf("\n"); I've removed the "else" clause. And fixed a couple minor whitespace issues. --b. > nfs4_print_acl(stdout, acl); > nfs4_free_acl(acl); > } > @@ -125,7 +132,7 @@ static void usage(int label) > { > if (label) > fprintf(stderr, "%s %s -- get NFSv4 file or directory access control lists.\n", execname, VERSION); > - fprintf(stderr, "Usage: %s [-R] file ...\n -H, --more-help\tdisplay ACL format information\n -?, -h, --help\tdisplay this help text\n -R --recursive\trecurse into subdirectories\n", execname); > + fprintf(stderr, "Usage: %s [-R] file ...\n -H, --more-help\tdisplay ACL format information\n -?, -h, --help\tdisplay this help text\n -R --recursive\trecurse into subdirectories\n -c, --omit-header\tDo not display the comment header (Do not print filename)\n", execname); > } > > static void more_help() > -- > 2.20.1