[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add quotes in AS_IF test for gid_t

From: Nick Bowler
Subject: Re: [PATCH] Add quotes in AS_IF test for gid_t
Date: Tue, 6 Feb 2024 23:37:32 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.0

On 2024-02-06 22:33, Sam James wrote:
> Noticed when building Emacs:
> ```
> * checking type of array argument to getgroups... ./configure: 42782: test: 
> =: unexpected operator
> ```
> This turns out to be because of missing quotes in AS_IF for
> ac_cv_type_gid_t in AC_TYPE_GETGROUPS.

No, I don't think this is the right fix.  The lack of shell quotation is
not the cause of this problem.

In the AC_TYPE_GETGROUPS macro, ac_cv_type_gid_t should not be empty,
because Autoconf has code to assign this variable to a nonempty value.
The fact that it is suggests the problem is elsewhere.

Oh look, I see this line in emacs-29.2/


This is the actual cause of the problem, because AC_TYPE_UID_T is the
part of Autoconf that would have assigned this variable.  Since Emacs
has deleted its definition, it has therefore broken other Autoconf
macros (like AC_TYPE_GETGROUPS) which depend on it.

I don't think it's right to work around damage like this in Autoconf.

The right place to fix this problem is in Emacs.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]