all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: 13026@debbugs.gnu.org
Subject: bug#13026: Assume POSIX 1003.1-1988 or later for signal.h.
Date: Tue, 04 Dec 2012 21:17:17 +0200	[thread overview]
Message-ID: <83k3sxr5j6.fsf@gnu.org> (raw)
In-Reply-To: <50BC0104.3070303@cs.ucla.edu>

> Date: Sun, 02 Dec 2012 17:31:48 -0800
> From: Paul Eggert <eggert@cs.ucla.edu>
> CC: 13026@debbugs.gnu.org
> 
> On 11/30/2012 12:54 AM, Eli Zaretskii wrote:
> > Maybe a better fix there would be to
> > have an array of data (name and value of each signal), and loop
> > through it, instead of using if-elseif.
> 
> Sure, that's easy enough.  Gnulib already has a module to do that.
> Revised patch attached (most of the new stuff is just code
> copied from Gnulib).  As a bonus, this fixes signal-process to
> handle several signal names that it currently doesn't support,
> such as those of the real-time signals.

Thanks.

This will need additions to lib/makefile.w32-in, to get the new
modules to compile into libgnu.a.

> > By the way, an alternative way of dealing with the lack of Posix
> > signals on Windows is to condition the related fragments on WINDOWSNT,
> > and remove the "#ifdef SIGFOO" conditions for Posix signals.
> 
> I'd rather use the Autoconf style, which is to test for individual
> features rather than test for operating systems, as that tends to
> make maintenance easier when operating systems or the emulations
> mutate.

The likes of "#ifdef SIGFOO" are not Autoconf tests.  They test macros
defined in header files, exactly like "#ifdef WINDOWSNT" does.

The advantage of the latter is that it clearly says why, say, SIGTSTP
is not used unconditionally, which could otherwise puzzle someone who
knows that this signal is universally supported by Posix platforms.





  reply	other threads:[~2012-12-04 19:17 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-29  6:52 bug#13026: Assume POSIX 1003.1-1988 or later for signal.h Paul Eggert
2012-11-29 17:33 ` Eli Zaretskii
2012-11-29 21:38   ` Paul Eggert
2012-11-30  8:54     ` Eli Zaretskii
2012-12-03  1:31       ` Paul Eggert
2012-12-04 19:17         ` Eli Zaretskii [this message]
2012-12-08  2:40           ` Paul Eggert
2012-12-08 10:09             ` Eli Zaretskii
2012-12-08 10:13             ` Eli Zaretskii
2012-12-08 17:34               ` Paul Eggert
2012-12-08 18:45                 ` Gnulib automation on Windows (was: bug#13026: Assume POSIX 1003.1-1988 or later for signal.h.) Eli Zaretskii

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=83k3sxr5j6.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=13026@debbugs.gnu.org \
    --cc=eggert@cs.ucla.edu \
    /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.