RE: [PATCH v1 3/4] git-compat-util.h: add FLOSS headers for HPE NonStop

From: Randall S. Becker <rsbecker@nexbridge.com>
Date: Dec 27, 2018, 15:55 UTC
Message-ID: <000001d49dfc$89cadd80$9d609880$@nexbridge.com>
In-Reply-To: <5712b2bf-8a90-42d7-9159-261dd990eb66@gmail.com>

On December 27, 2018 7:10, Derrick Stolee wrote:

Show 10 quoted lines

> On 12/26/2018 6:05 PM, randall.s.becker@rogers.com wrote:
> > The NSIG define is also not defined on __TANDEM, so we define it here
> > as 100 if it is not defined only for __TANDEM builds.
> [snip]
> > +#if ! defined NSIG
> 
> Why didn't you use "#ifndef" here?
> 
> Taking a look at the file, I see both "#ifdef" and "#if defined" but no "#if !
> defined".

I'm good with revising as follows and removing the irrelevant lines (stay tuned):

+#ifdef __TANDEM
+#include <floss.h(floss_execl,floss_execlp,floss_execv,floss_execvp)>
+#include <floss.h(floss_getpwuid)>
+#ifndef NSIG
+/* NonStop NSE and NSX do not provide NSIG. SIGGUARDIAN(99) is the highest
+   known, by detective work using kill -l as a list is all signals
+   instead of signal.h where it should be. */
+# define NSIG 100
+#endif
+#endif
+

Cheers, Randall