[Patches] [ python-Patches-639371 ] Removal of FreeBSD 5.0 specific test
noreply@sourceforge.net
noreply@sourceforge.net
Sun, 17 Nov 2002 15:19:00 -0800
Patches item #639371, was opened at 2002-11-16 17:48
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=639371&group_id=5470
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Marc Recht (marc)
Assigned to: Nobody/Anonymous (nobody)
Summary: Removal of FreeBSD 5.0 specific test
Initial Comment:
After latest Additions to the FreeBSD 5.0-current
headers, the
special case isn't needed any longer. The two last
problematic
functions are ctermid_r and setgroups which aren't
defined in the POSIX/XOPEN case. This patch works
around the problem by
setting CFLAGS with -Wall -Werror for gcc before
checking for
this two functions.
----------------------------------------------------------------------
>Comment By: Marc Recht (marc)
Date: 2002-11-18 00:19
Message:
Logged In: YES
user_id=205
The setting of CFLAGS is for the test below. For FreeBSD
XOPEN* and POSIX* would be sufficient, but IMO it's better
to set all (till this point) found CFLAGS. The setting of
OLDCFLAGS should have been one line above the if. It's used
to safe the CFLAGS before the tests and restore it later.
That allows to set the (problematic?) -Werror for the two
checks.
But, silly me., you're right.. By checking the address of
the function it works without -Wall -Werror. Though the
CFLAGS are still needed.
----------------------------------------------------------------------
Comment By: Martin v. Löwis (loewis)
Date: 2002-11-17 21:49
Message:
Logged In: YES
user_id=21627
Can you please explain the purpose of setting CFLAGS first
to MY_CPPFLAGS etc? Also, why are you seeting OLDFLAGS
between the if and the then?
Also, setting -Werror might have unintended site effects,
IMO, testing for a declaration is better done by checking
whether the address of a function can be taken.
Apart from that, the patch looks good.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=639371&group_id=5470