Hi Bruno, Ok thanks for the good finding - my bad.
To be honest, I did not test the automake stuff, I only tested the functionality via coreutils. In coreutils they have their own xattr.m4 which basically does the same. Do I need to resubmit another patch with your changes included? Thanks, Ondrej -----Original Message----- From: Bruno Haible <br...@clisp.org> Sent: pátek 13. ledna 2023 9:10 To: bug-gnulib@gnu.org; Ondrej Valousek <ondrej.valousek...@renesas.com> Subject: Re: [PATCH] Use xattr (Linux) in qcopy-acl.c This part > Also, protect against unsafe use of a configure option value. is needed when the user does ./configure --enable-xattr="no no" In this case we don't want to see an error during configuration: checking for opendir... yes ../configure: line 29819: test: too many arguments checking for raise... yes Always putting $use_attr in double-quotes achieves this. Bruno
