Re: Last commit breaks macOS build

[Richard Copley]

On 5 January 2017 at 03:21, Paul Eggert <address@hidden> wrote:
> Angelo Graziosi wrote:
>
>> Interfering nt/gnulib.mk from lib/gnulib.mk ...
>
>
> Did it really say "Interfering"? :-)
>
>> sed: 46: "This is of m...": command c expects \ followed by text
>>
>> and the 'configure' is not created...
>
>
> This is caused by our recent attempt to better-automate MS-Windows builds. I
> fixed one nonportable construct, but evidently there are others.
>
> It's time to break out a bigger hammer. Please try the attached patch. I
> have tested it on Fedora, but not on MS-Windows and so have not installed it
> yet.

Hi Paul,
In MSYS2 on Windows, with the patch I get this error:

gcc  -DHAVE_CONFIG_H -I. -I../../repo/emacs/lib -I../src
-I../../repo/emacs/src   -march=native -mtune=native
-D_WIN32_WINNT=_WIN32_WINNT_WIN7 -DUSE_CRT_DLL=1 -I
/g/emacs/build/../repo/emacs/nt/inc  -fno-common /* -W... */ -Og -g
-ggdb -MT set-permissions.o -MD -MP -MF .deps/set-permissions.Tpo -c
-o set-permissions.o ../../repo/emacs/lib/set-permissions.c
../../repo/emacs/lib/set-permissions.c: In function 'set_acls':
../../repo/emacs/lib/set-permissions.c:500:6: error: #error Must have
acl_delete_def_file (see POSIX 1003.1e draft 17).
 #    error Must have acl_delete_def_file (see POSIX 1003.1e draft 17).
      ^~~~~
../../repo/emacs/lib/set-permissions.c:539:13: warning: implicit
declaration of function 'acl_delete_def_file'
[-Wimplicit-function-declaration]
       ret = acl_delete_def_file (name);
             ^~~~~~~~~~~~~~~~~~~
../../repo/emacs/lib/set-permissions.c:539:7: warning: nested extern
declaration of 'acl_delete_def_file' [-Wnested-externs]
       ret = acl_delete_def_file (name);
       ^~~

Without the patch, set-permissions.c isn't compiled.