[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-13 Thread Tarek Ziadé

Changes by Tarek Ziadé ziade.ta...@gmail.com:


--
assignee: tarek - loewis

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-13 Thread Martin v . Löwis

Martin v. Löwis mar...@v.loewis.de added the comment:

For 2.6 and 3.0, this is now fixed in r73406 and r73407.

--
resolution:  - fixed
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-12 Thread Martin v . Löwis

Martin v. Löwis mar...@v.loewis.de added the comment:

For 2.7 and 3.1, this is now fixed with r73390 and r73391. The trick is
to set the Win64 bit in RegLocator also.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-12 Thread Jason R. Coombs

Jason R. Coombs jar...@jaraco.com added the comment:

That's great.  Thanks Martin!

What about Python 2.6?  If the fix is hard to port back to 2.6, perhaps
bdist_msi should raise an Unsupported error in 64-bit Python.

Let me know if I can help any further.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-11 Thread Tarek Ziadé

Tarek Ziadé ziade.ta...@gmail.com added the comment:

No that's fine Jason, I was about to do it ;)

I can work on the patch and commit it since it's distutils-related, but
Martin is the one that will validate it and give the green light on msi
matters.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-10 Thread Jason R. Coombs

New submission from Jason R. Coombs jar...@jaraco.com:

It appears as if bdist_msi isn't properly tagging 64-bit binary builds.

When launching an .msi built by Python 2.6.2 using bdist_msi, such as
numpy found here
(https://sourceforge.net/project/showfiles.php?group_id=1369package_id=175103),
it improperly detects the location of Python (which it gets from the
registry).  If the 32-bit Python 2.6 is also installed, it finds that
version.  If 32-bit Python 2.6 is not installed, it fails to find the
Python installation.  Even if the proper 64-bit Python 2.6 location is
selected, the files are not installed to the Python 2.6 site-packages. 
Furthermore, the registry Uninstall information is stored in the
Wow6432Node.

bdist_wininst executables appear to detect 64-bit Python and install
correctly.

I will attempt to recreate this problem with a minimal project and clean
64-bit Vista or Windows 7 machine at a later date.

--
assignee: tarek
components: Distutils
messages: 89221
nosy: jaraco, tarek
severity: normal
status: open
title: distributions built with bdist_msi on 64-bit Windows fail to install 
correctly
versions: Python 2.6

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-10 Thread Jason R. Coombs

Jason R. Coombs jar...@jaraco.com added the comment:

Indeed, I confirmed that using the simple example from the distutils manual 
(http://docs.python.org/distutils/introduction.html#a-simple-example) on a 
clean install of Python 2.6.2, bdist_msi exhibits the behavior previously 
described.

I suspect that the TargetPlatform property needs to be set (based on what I 
read here: http://msdn.microsoft.com/en-us/library/cd7a85k9(VS.80).aspx ).

--
title: distributions built with bdist_msi on 64-bit Windows fail to install 
correctly - distributions built with bdist_msi on 64-bit Windows fail  to 
install correctly

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-10 Thread Jason R. Coombs

Jason R. Coombs jar...@jaraco.com added the comment:

Based on the MSDN article and what I read in a blog entry
(http://blogs.msdn.com/heaths/archive/2005/10/24/windows-installer-on-64-bit-platforms.aspx),
I thought that the enclosed patch might work around the issue... and
while it does set the template property to x64, the undesirable behavior
persists.

Index: Lib/msilib/__init__.py
===
--- Lib/msilib/__init__.py  (revision 73295)
+++ Lib/msilib/__init__.py  (working copy)
@@ -3,8 +3,11 @@
 # Licensed to PSF under a Contributor Agreement.
 from _msi import *
 import os, string, re
+import sys

-Win64=0
+Intel64=0
+AMD64 = 'AMD64' in sys.version
+Win64 = Intel64 or AMD64

 # Partially taken from Wine
 datasizemask=  0x00ff
@@ -145,8 +148,10 @@
 si.SetProperty(PID_TITLE, Installation Database)
 si.SetProperty(PID_SUBJECT, ProductName)
 si.SetProperty(PID_AUTHOR, Manufacturer)
-if Win64:
+if Intel64:
 si.SetProperty(PID_TEMPLATE, Intel64;1033)
+elif AMD64:
+si.SetProperty(PID_TEMPLATE, x64;1033)
 else:
 si.SetProperty(PID_TEMPLATE, Intel;1033)
 si.SetProperty(PID_REVNUMBER, gen_uuid())

--
components: +Windows

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6258] distributions built with bdist_msi on 64-bit Windows fail to install correctly

2009-06-10 Thread Jason R. Coombs

Jason R. Coombs jar...@jaraco.com added the comment:

I'm adding Martin to this as he appears to be the author of msilib.  If
it's inappropriate for me to do this, I apologize.  Just let me know and
I won't do it again.

--
nosy: +loewis

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6258
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com