[Patches] [ python-Patches-805626 ] make test_fcntl 64bit clean

[SourceForge.net]

Patches item #805626, was opened at 2003-09-13 16:54
Message generated for change (Comment added) made by loewis
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=805626&group_id=5470

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Marc Recht (marc)
Assigned to: Nobody/Anonymous (nobody)
Summary: make test_fcntl 64bit clean

Initial Comment:
Patch to make test_fcntl.py 64bit clean (from NetBSD's

pkgsrc)



The patch is against release23-maint.



Note: I'm not the original autor of this patch. I have

permission to submit it, though.

----------------------------------------------------------------------

>Comment By: Martin v. Löwis (loewis)
Date: 2003-09-19 00:49

Message:
Logged In: YES 
user_id=21627

The Alpha version likely runs fine because it is a 64-bit

system, so they use all 8 bytes of the padding (even if in a

different order then the original layout - I assume the

structure definition is completely different from the one

for 32-bit systems?)



I'm mostly concerned about OS X, and perhaps other PPC

systems, as well as 32-bit SPARC. 



It would be good if you could report the *BSD definitions of

the structure on various architectures.

----------------------------------------------------------------------

Comment By: Marc Recht (marc)
Date: 2003-09-18 23:55

Message:
Logged In: YES 
user_id=205

To be honest, I'm not sure. It is tested on Alpha on NetBSD,

though. 

And it is in "pkgsrc" for quite some time now and noone

filed a PR. (But this could also mean that just nobody runs

the tests..)

----------------------------------------------------------------------

Comment By: Martin v. Löwis (loewis)
Date: 2003-09-18 20:48

Message:
Logged In: YES 
user_id=21627

Does the patch work on all systems? For both byte orders? On

both 32-bit and 64-bit systems?



I'm particularly concerned about big-endian 32-bit systems,

as the padding apparently is intended for little-endian

systems only.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=805626&group_id=5470