|
msg107398 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-06-09 14:20 |
2.5.1 version of distutils was able to correctly build wininst dists from linux. Right now there are errors which I'll describe in more details once I can get some Linux box.
|
|
msg107399 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-06-09 14:23 |
At least is was reported that compiled binaries contains wrong platform name, i.e. 'linux' and there are errors about inability to look up some compiler options in windows registry while building.
|
|
msg107400 - (view) |
Author: Brian Curtin (brian.curtin) *  |
Date: 2010-06-09 14:37 |
Where was that reported?
|
|
msg107432 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-06-09 22:39 |
In this SCons thread http://scons.tigris.org/ds/viewMessage.do?dsForumId=1268&dsMessageId=2617686
|
|
msg107533 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-06-11 07:51 |
$ python setup.py build bdist_wininst
running build
running build_py
running bdist_wininst
installing to build/bdist.linux-i686/wininst
running install_lib
creating build/bdist.linux-i686/wininst
creating build/bdist.linux-i686/wininst/PURELIB
copying build/lib.linux-i686-2.6/wget.py -> build/bdist.linux-i686/wininst/PURELIB
running install_egg_info
Writing build/bdist.linux-i686/wininst/PURELIB/wget-0.6.egg-info
creating '/tmp/tmpQQZI5O.zip' and adding '.' to it
adding 'PURELIB/wget-0.6.egg-info'
adding 'PURELIB/wget.py'
creating dist
Warning: Can't read registry to find the necessary compiler setting
Make sure that Python modules _winreg, win32api or win32con are installed.
removing 'build/bdist.linux-i686/wininst' (and everything under it)
$ ls dist
wget-0.6.linux-i686.exe
|
|
msg107535 - (view) |
Author: Éric Araujo (eric.araujo) *  |
Date: 2010-06-11 08:36 |
The error comes from the msvccompiler module, which shouldn’t get used on non-Windows platforms unless I’m mistaken. Reading the module, I’m not sure at all this command was meant to be run on not-Windows OSes.
|
|
msg107536 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-06-11 08:47 |
It did run ok on non-Windows OSes with previous Python versions.
|
|
msg108776 - (view) |
Author: Georg Brandl (georg.brandl) *  |
Date: 2010-06-27 10:04 |
It certainly should be possible to build these installers, and if I understand your output correctly, an installer *was* built. The warning is, after all, just a warning, and probably just the warning got added compared to 2.5, but the behavior is the same.
The platform name is a regression though.
|
|
msg109984 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-07-11 10:21 |
The regression is two-fold:
1. platform name regression is caused by r62197 by Mark Hammond - Issue #2513: enable 64bit cross compilation on windows
2. when wininst stub selection started to depend on MSVC compiler version is yet to be investigated, but I am sure that wininst stub selection should only be affected by compatibility with operating system only
More complete research notes are at https://wave.google.com/wave/waveref/googlewave.com/w+GnQ9tjc8A
|
|
msg110075 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-07-12 08:45 |
http://codereview.appspot.com/1774043
|
|
msg110076 - (view) |
Author: Georg Brandl (georg.brandl) *  |
Date: 2010-07-12 08:52 |
Did you test that patch? ISTM that it contains a syntax error:
+ if not self.distribution.has_ext_modules()
+ and not self.distribution.has_c_libraries():
|
|
msg110083 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2010-07-12 11:37 |
reuploaded correct version to rietveld.
|
|
msg110109 - (view) |
Author: Mark Hammond (mhammond) *  |
Date: 2010-07-12 16:35 |
With the caveat that I haven't tested it (I'm currently traveling), the patch looks good to me.
|
|
msg118800 - (view) |
Author: Georg Brandl (georg.brandl) *  |
Date: 2010-10-15 16:44 |
Raising priority.
|
|
msg119085 - (view) |
Author: Éric Araujo (eric.araujo) *  |
Date: 2010-10-18 21:31 |
I get an error trying to run the command because of the use of the mbcs codec.
Ezio made some comments on Rietveld, I addressed them in a local clone.
Question for Windows users/experts: Do the terms “binary extension” and “binary code” make sense? If not, I can change them to the clearer “extension modules” and “C code”.
|
|
msg125847 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2011-01-09 17:05 |
Fixed Ezio comments at http://codereview.appspot.com/1774043
Eric, if I understand you correctly, the mbcs issue you're experiencing is unrelated to the patch. What else do you need?
|
|
msg126523 - (view) |
Author: anatoly techtonik (techtonik) |
Date: 2011-01-19 11:38 |
Is this issue really critical?
Bump.
|
|
msg126529 - (view) |
Author: Éric Araujo (eric.araujo) *  |
Date: 2011-01-19 15:29 |
I opened another bug for the MBCS problem which prevents me from testing Anatoly’s patch.
|
|
msg135893 - (view) |
Author: Ralf Schlatterbeck (runtux) |
Date: 2011-05-13 10:29 |
2.6 already produces a .linux-i686.exe package instead of .win32.exe when running bdist_wininst. I've now used python2.5 for producing a binary windows installer for roundup.
|
|
msg135902 - (view) |
Author: Éric Araujo (eric.araujo) *  |
Date: 2011-05-13 13:09 |
2.6 only gets security fixes.
|
|
| Date |
User |
Action |
Args |
| 2011-05-13 13:09:23 | eric.araujo | set | messages:
+ msg135902 versions:
- Python 2.6 |
| 2011-05-13 10:29:15 | runtux | set | nosy:
+ runtux
messages:
+ msg135893 versions:
+ Python 2.6 |
| 2011-01-19 15:29:22 | eric.araujo | set | nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin dependencies:
+ bdist_wininst depends on MBCS codec, unavailable on non-Windows messages:
+ msg126529 |
| 2011-01-19 11:39:42 | techtonik | set | files:
+ issue1774043_7001.diff nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin |
| 2011-01-19 11:38:35 | techtonik | set | nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin messages:
+ msg126523 |
| 2011-01-09 17:41:01 | techtonik | set | files:
- suffix.patch nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin |
| 2011-01-09 17:40:53 | techtonik | set | files:
+ suffix.patch nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin |
| 2011-01-09 17:05:11 | techtonik | set | nosy:
mhammond, georg.brandl, techtonik, tarek, eric.araujo, brian.curtin messages:
+ msg125847 |
| 2010-10-18 21:31:03 | eric.araujo | set | assignee: tarek -> eric.araujo messages:
+ msg119085 versions:
- Python 2.6 |
| 2010-10-15 16:44:18 | georg.brandl | set | priority: normal -> critical
messages:
+ msg118800 |
| 2010-07-12 16:35:31 | mhammond | set | messages:
+ msg110109 |
| 2010-07-12 11:38:11 | techtonik | set | files:
- suffix.patch |
| 2010-07-12 11:37:57 | techtonik | set | files:
+ suffix.patch
messages:
+ msg110083 |
| 2010-07-12 08:52:07 | georg.brandl | set | messages:
+ msg110076 |
| 2010-07-12 08:46:31 | techtonik | set | files:
+ suffix.patch keywords:
+ patch |
| 2010-07-12 08:45:55 | techtonik | set | messages:
+ msg110075 |
| 2010-07-11 10:21:42 | techtonik | set | nosy:
+ mhammond messages:
+ msg109984
|
| 2010-06-27 10:04:24 | georg.brandl | set | nosy:
+ georg.brandl messages:
+ msg108776
|
| 2010-06-11 08:47:59 | techtonik | set | messages:
+ msg107536 |
| 2010-06-11 08:36:26 | eric.araujo | set | nosy:
+ eric.araujo messages:
+ msg107535
|
| 2010-06-11 07:51:29 | techtonik | set | messages:
+ msg107533 |
| 2010-06-09 22:39:10 | techtonik | set | messages:
+ msg107432 |
| 2010-06-09 14:37:00 | brian.curtin | set | nosy:
+ brian.curtin messages:
+ msg107400
type: behavior stage: test needed |
| 2010-06-09 14:23:54 | techtonik | set | messages:
+ msg107399 |
| 2010-06-09 14:20:42 | techtonik | create | |