[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH] Use xattr (Linux) in qcopy-acl.c
From: |
Ondrej Valousek |
Subject: |
RE: [PATCH] Use xattr (Linux) in qcopy-acl.c |
Date: |
Fri, 13 Jan 2023 08:32:48 +0000 |
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
- Re: ACL complexity, (continued)
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/05
- RE: [PATCH] Use xattr (Linux) in qcopy-acl.c, Ondrej Valousek, 2023/01/05
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/05
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Paul Eggert, 2023/01/05
- RE: [PATCH] Use xattr (Linux) in qcopy-acl.c, Ondrej Valousek, 2023/01/11
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/12
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/13
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/13
- RE: [PATCH] Use xattr (Linux) in qcopy-acl.c,
Ondrej Valousek <=
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/13
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Paul Eggert, 2023/01/13
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Ondrej Valousek, 2023/01/14
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/14
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Ondrej Valousek, 2023/01/14
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/13
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/13
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/30
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/05
- Message not available
- Message not available
- Re: [PATCH] Use xattr (Linux) in qcopy-acl.c, Bruno Haible, 2023/01/12