RE: [PATCH] Use xattr (Linux) in qcopy-acl.c

[Ondrej Valousek]

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 <bruno@clisp.org> 
Sent: pátek 13. ledna 2023 9:10
To: bug-gnulib@gnu.org; Ondrej Valousek <ondrej.valousek.xm@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