Patches item #1339673, was opened at 2005-10-27 17:18 Message generated for change (Comment added) made by hanwen You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1339673&group_id=5470
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Jan Nieuwenhuizen (janneke-sf) Assigned to: Nobody/Anonymous (nobody) Summary: cross compile and mingw support Initial Comment: The cross compile patch is similar to what I did for Guile and has still some bits from Daniel Goertzen's https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1006238&group_id=5470 The mingw patch is based on Matthias Kramm's https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1053879&group_id=5470 but adds autoconf tests and does have his C modifications. -- Jan Nieuwenhuizen <[EMAIL PROTECTED]> | GNU LilyPond - The music typesetter http://www.xs4all.nl/~jantien | http://www.lilypond.org ---------------------------------------------------------------------- Comment By: Han-Wen Nienhuys (hanwen) Date: 2006-12-05 23:52 Message: Logged In: YES user_id=161998 Originator: NO superseded by #1597850 ---------------------------------------------------------------------- Comment By: Han-Wen Nienhuys (hanwen) Date: 2006-12-05 23:51 Message: Logged In: YES user_id=161998 Originator: NO superseded by #1597850 ---------------------------------------------------------------------- Comment By: Richard Tew (rmt38) Date: 2006-01-06 05:46 Message: Logged In: YES user_id=1417949 > Why do you think that? You will find that no autotooled > package will build or install in directories with spaces Perhaps I was not clear enough. The problem is not that I am building in those directories, it is that the patch sets the PATH environment variable as a prefix to the compilation command (in the same line). And on Windows, it is not uncommon for directories in Program Files and other directories with spaces in their names to be in the path. This breaks the correctness of the command line and the execution of it. Take another look at the erroneous command: PATH=/usr/bin:.:/usr/local/bin:/mingw/bin:/bin:/c/WINDOWS/s ystem32:/c/WINDOWS:/c/WINDOWS/System32/Wbem:/c/Program Files/ATI Technologies/ATI Control Panel:/c/PROGRA~1/COMMON~1/SONICS~1/:/c/Program Files/Common Files/Adobe/AGL:/usr/bin:/c/devkitpro/devkitarm/bin:. cc - c -I -I./Include -o Parser/acceler.x Parser/acceler.c And the error: /bin/sh: Files/ATI: No such file or directory make: *** [Parser/acceler.x] Error 127 Hope this helps. ---------------------------------------------------------------------- Comment By: Jan Nieuwenhuizen (janneke-sf) Date: 2006-01-06 00:31 Message: Logged In: YES user_id=1368960 > Looks like the patch needs to handle paths with spaces in, > in the PATH. Why do you think that? You will find that no autotooled package will build or install in directories with spaces. If this would be possible at all, it would be a separate feature. I do not see a dependency with cross compilation or mingw building. ---------------------------------------------------------------------- Comment By: Richard Tew (rmt38) Date: 2006-01-05 21:20 Message: Logged In: YES user_id=1417949 Hi, I patched Python 2.4.2 source code download with this (can't get CVS as bandwidth is limited). context-cross.patch fails on one chunk of Makefile.pre.in, although hand application fixed it. Otherwise, context- mingw-2.patch applied cleanly. With both applied, I did the following: export BASECFLAGS="-mcpu=arm9tdmi -mcpu=arm9tdmi -ffast- math -mthumb -mthumb-interwork -DARM9 -D__NDS__" export CFLAGS="-mcpu=arm9tdmi -mcpu=arm9tdmi -ffast-math - mthumb -mthumb-interwork -DARM9 -D__NDS__" export LDFLAGS="-specs=ds_arm9.specs -g -mthumb -mthumb- interwork -Wl,-Map,libpython2.4.map" sh configure --host=arm-elf This is what eventually happened: PATH=/usr/bin:.:/usr/local/bin:/mingw/bin:/bin:/c/WINDOWS/s ystem32:/c/WINDOWS:/c/WINDOWS/System32/Wbem:/c/Program Files/ATI Technologies/ATI Control Panel:/c/PROGRA~1/COMMON~1/SONICS~1/:/c/Program Files/Common Files/Adobe/AGL:/usr/bin:/c/devkitpro/devkitarm/bin:. cc - c -I -I./Include -o Parser/acceler.x Parser/acceler.c /bin/sh: Files/ATI: No such file or directory make: *** [Parser/acceler.x] Error 127 Looks like the patch needs to handle paths with spaces in, in the PATH. ---------------------------------------------------------------------- Comment By: Jan Nieuwenhuizen (janneke-sf) Date: 2005-11-04 13:59 Message: Logged In: YES user_id=1368960 The second mingw patch actually produces a working python, fixing * loadable dll modules * argv0 relocation * posixflavour nt -- Jan. ---------------------------------------------------------------------- Comment By: Jan Nieuwenhuizen (janneke-sf) Date: 2005-10-27 17:20 Message: Logged In: YES user_id=1368960 .... and does NOT have his C modifications ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1339673&group_id=5470 _______________________________________________ Patches mailing list Patches@python.org http://mail.python.org/mailman/listinfo/patches
