[issue10898] posixmodule.c redefines FSTAT

2020-11-15 Thread Irit Katriel
Change by Irit Katriel : -- versions: +Python 3.10, Python 3.8, Python 3.9 -Python 2.7, Python 3.2, Python 3.3, Python 3.4 ___ Python tracker ___ _

[issue10898] posixmodule.c redefines FSTAT

2013-07-07 Thread Christian Heimes
Changes by Christian Heimes : -- nosy: +christian.heimes versions: +Python 3.3, Python 3.4 -Python 3.1 ___ Python tracker ___ ___ Pyth

[issue10898] posixmodule.c redefines FSTAT

2011-09-12 Thread Alan Hourihane
Changes by Alan Hourihane : -- resolution: accepted -> remind ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue10898] posixmodule.c redefines FSTAT

2011-07-03 Thread Alan Hourihane
Alan Hourihane added the comment: Well, I'd probably prefer something akin to my first patch then. There's no need to #undef things at all if we just prefix the usage with PYTHON_ -- ___ Python tracker __

[issue10898] posixmodule.c redefines FSTAT

2011-07-02 Thread Antoine Pitrou
Antoine Pitrou added the comment: > I've had to split the three #undef's up to just before they're used. I would really prefer to move up the offending #include rather than sprinkle those #undef's all over the place. -- ___ Python tracker

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Alan Hourihane
Alan Hourihane added the comment: I've had to split the three #undef's up to just before they're used. -- ___ Python tracker ___ ___

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Antoine Pitrou
Antoine Pitrou added the comment: Hum, can you propose something? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubsc

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Alan Hourihane
Alan Hourihane added the comment: No, that patch still doesn't work. -- resolution: fixed -> accepted ___ Python tracker ___ ___ Pyth

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ah, sorry. Can you check with the changeset I've just pushed (it's on branch 2.7)? If ok, I'll port it to 3.2/3.x and close again. -- resolution: accepted -> fixed ___ Python tracker

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Roundup Robot
Roundup Robot added the comment: New changeset 45b27448f95c by Antoine Pitrou in branch '2.7': Really fix issue #10898: posixmodule.c redefines FSTAT http://hg.python.org/cpython/rev/45b27448f95c -- nosy: +python-dev ___ Python tracker

[issue10898] posixmodule.c redefines FSTAT

2011-07-01 Thread Alan Hourihane
Alan Hourihane added the comment: Hi Antoine, Unfortunately the #undef is too early and later #includes redefine it. We should move the #undef closer to the code that actually uses them. -- resolution: fixed -> accepted status: closed -> open ___ P

[issue10898] posixmodule.c redefines FSTAT

2011-01-19 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ok, fixed in r88111 (3.2), r88112 (3.1) and r88113 (2.7). -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker _

[issue10898] posixmodule.c redefines FSTAT

2011-01-18 Thread Alan Hourihane
Alan Hourihane added the comment: On Tue, 2011-01-18 at 19:59 +, Antoine Pitrou wrote: > Antoine Pitrou added the comment: > > How about adding "#undef FSTAT" instead? Would it work for you? Sure. Alan. -- ___ Python tracker

[issue10898] posixmodule.c redefines FSTAT

2011-01-18 Thread Antoine Pitrou
Antoine Pitrou added the comment: How about adding "#undef FSTAT" instead? Would it work for you? -- components: +Extension Modules nosy: +pitrou stage: -> needs patch versions: +Python 2.7, Python 3.1, Python 3.2 ___ Python tracker

[issue10898] posixmodule.c redefines FSTAT

2011-01-12 Thread Alan Hourihane
New submission from Alan Hourihane : Python 2.7.1 redefines FSTAT, but the problem is my libc already defines FSTAT in sys/ioctl.h. I've worked around this by prefixing the FSTAT define with PYTHON. It should probably be done with STAT too. --- Modules/posixmodule.c.old 2011-01-12 01:46:45.