all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Po Lu <luangruo@yahoo.com>, Paul Eggert <eggert@cs.ucla.edu>
Cc: emacs-devel@gnu.org
Subject: MinGW build on master broken by Gnulib update
Date: Thu, 05 Sep 2024 08:43:23 +0300	[thread overview]
Message-ID: <86y1464pvo.fsf@gnu.org> (raw)

The last update from Gnulib broke the MinGW build on master:

  sig2str.c:321:1: warning: no previous prototype for 'str2sig' [-Wmissing-prototypes]
    321 | str2sig (char const *signame, int *signum)
	| ^~~~~~~
  sig2str.c:332:1: warning: no previous prototype for 'sig2str' [-Wmissing-prototypes]
    332 | sig2str (int signum, char *signame)
	| ^~~~~~~
  [...]
  process.c: In function 'abbr_to_signal':
  process.c:7265:9: warning: implicit declaration of function 'str2sig' [-Wimplicit-function-declaration]
   7265 |  return str2sig (sigbuf, &signo) == 0 ? signo : -1;
	|         ^~~~~~~
  process.c:7265:9: warning: nested extern declaration of 'str2sig' [-Wnested-externs]
  process.c: In function 'Fsignal_names':
  process.c:8539:13: error: 'SIG2STR_MAX' undeclared (first use in this function)
   8539 |   char name[SIG2STR_MAX];
	|             ^~~~~~~~~~~
  process.c:8539:13: note: each undeclared identifier is reported only once for each function it appears in
    CC       intervals.o
  process.c:8544:12: warning: implicit declaration of function 'sig2str' [-Wimplicit-function-declaration]
   8544 |       if (!sig2str (i, name))
	|            ^~~~~~~
  process.c:8544:12: warning: nested extern declaration of 'sig2str' [-Wnested-externs]
  process.c:8539:8: warning: unused variable 'name' [-Wunused-variable]
   8539 |   char name[SIG2STR_MAX];
	|        ^~~~
    CC       textprop.o
  Makefile:457: recipe for target `process.o' failed
  make[2]: *** [process.o] Error 1

Gnulib moved the prototypes of sig2str and str2sig from sig2str.h to
Gnulib's signal.h, evidently assuming that a build which uses the
Gnulib sig2str will also use the Gnulib signal.h header, but that
assumption is false for the MinGW build of Emacs, which omits
lib/signal.h (because it clashes with some w32 code in Emacs).

I fixed that temporarily by modifying lib/sig2str.h to include the
missing stuff for MinGW, but this is really a Gnulib issue, and should
be fixed in Gnulib, IMO.



             reply	other threads:[~2024-09-05  5:43 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-05  5:43 Eli Zaretskii [this message]
2024-09-05 16:28 ` MinGW build on master broken by Gnulib update Paul Eggert
2024-09-05 18:02   ` Eli Zaretskii
2024-09-05 18:49     ` Paul Eggert
2024-09-06  6:48       ` Eli Zaretskii
2024-09-06  9:52         ` Bruno Haible

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=86y1464pvo.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=eggert@cs.ucla.edu \
    --cc=emacs-devel@gnu.org \
    --cc=luangruo@yahoo.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.