Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
This mostly a documentation concern - correct? ctypes.wintypes is sparsely
documented - in 2.7 the help for it seems to be:
The ctypes.wintypes module provides quite some other Windows specific data
types, for example HWND, WPARAM
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
The following two error conditions that can be returned by MsiOpenDatabase do
not have specific handling:
ERROR_CREATE_FAILED (1631)
The database could not be created.
ERROR_OPEN_FAILED (110)
The database could not be opened
Changes by Mark Mc Mahon mtnbikingm...@gmail.com:
--
keywords: +patch
Added file:
http://bugs.python.org/file22332/support_vt_empty_in_summary_getproperty.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12239
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I was looking at this - and see that (at least as far as GetFileAttributes is
concerned) that a mount and a linked directory are seen the same...
Here are some tests using ctypes
# mounted drive
hex(windll.kernel32.GetFileAttributesW
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
Using some.msi where the first property is VT_EMPTY
Using COM:
from win32com.client import gencache
com_lib = gencache.EnsureModule('{000C1092---C000-0046}',
409, 1, 0)
com_msi = com_lib.Installer()
db
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
In _msi.c, summary_getproperty() and summary_setproperty() raise an exception
if the type being set/got is VT_FILETIME.
This issue is to track adding support for (or declining to support) FILETIMEs
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
Per Eric V. Smith's comment issue1104 (msg134976) the return value of the call
MsiSummaryInfoGetProperty() in summary_getproperty() (pc\_msi.c) is only
checked for the error return ERROR_MORE_DATA.
Other error values should be checked
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
Responding to Eric's comments
1. There are only three valid property types returned by MsiInteger, String
FILETIME. (http://msdn.microsoft.com/en-us/library/aa372045%28v=VS.85%29.aspx)
2. That comment makes sense - I have entered a new
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I have added tests to the patch - but it's not easy to know what would cause an
error in MsiSummaryInfoGetProperty which would trigger the new code.
--
keywords: +patch
Added file:
http://bugs.python.org/file22165
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I am not sure that I fully understand the issue - but it seems that trunk still
has this issue.
As stated by Amaury - this is on DEBUG builds only.
c:\pcbuild\python_d.exe
Python 3.3a0 (default, May 28 2011, 20:22:11) [MSC v.1500 32 bit
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I wasn't so happy trawling through \windows\installer either :)
Creating an MSI to test is very simple, and actually quicker than I had
originally thought. The latest patch (support_dir_for_msi_objs.patch) creates
the one and just uses
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
You can pass it to any function in the MSI SDK through ctypes.
e.g.
def ReadStream(record, field):
buf = (ctypes.c_char * 2048)()
orig_size = ctypes.sizeof(buf)
status = 0
res = []
while
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
Background:
My main use case for msilib is for working with/editing existing MSI files and
not creating MSI files.
As such I find much of the MSI API that I need missing.
While not difficult to re-create _msi.c with ctypes, I thought
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
Reviewing the patch (issue133572.py33.patch):
You have used forward slashes for the first change - but back slashes for the
others. I see that other places in the existing docs use back slashes when
referring to windows paths.
I have
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I have updated the patch for current trunk (though no real changes required). I
also include a testcase.
One thing to review is that I added functionality to the tests to create the
MSI to be tested. (felt this was safer than touching
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
And fix the typo... (thanks Ezio)
--
Added file: http://bugs.python.org/file21525/issue1104_msi_3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1104
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
That fixed it - but it seems we need that for the other Types defined in the
module.
Regarding testing - would it be a good idea to add an MSI to the test suite -
or better to create one during testing (using msilib) and then use
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
This issue has been fixed by changes made in issue7639 and issue11696
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2694
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
This issue has been fixed when issue11696 was fixed.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8554
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
Running the following code:
import msilib
db = msilib.OpenDatabase('c:/windows/installer/1c7a1.msi', 1)
db
_msi.Database object at 0x01E230A0
dir(db)
(Python crashes - tested on current Trunk and Python 2.7.1).
I tried tracking
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
Following up from my comment in issue1128, please find the patch
msilib.make_id_fix_and_tests.patch
This improves the make_id() function in the following way:
- ensures that NO invalid identifier characters make it through (it does
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
For Directory.make_short() the only things which are left could be considered
splitting hairs.
1. Do we need to remove '\/:' from the file name - if these characters are
there then the file name is not valid in the first place.
2
New submission from Mark Mc Mahon mtnbikingm...@gmail.com:
msilib.make_id() currently ensure that any of the following characters are not
in the resulting ID: -+~;
Per the Microsoft documentation the following list of characters are allowed.
http://msdn.microsoft.com/en-us/library/aa369212(v
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
Per: http://msdn.microsoft.com/en-us/library/aa369212(v=vs.85).aspx
The Identifier data type is a text string. Identifiers may contain the
ASCII characters A-Z (a-z), digits, underscores (_), or periods (.). However,
every identifier must
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
How about the following patch and tests...
Per: http://msdn.microsoft.com/en-us/library/aa369212(v=vs.85).aspx
The Identifier data type is a text string. Identifiers may contain the
ASCII characters A-Z (a-z), digits, underscores
Mark Mc Mahon mtnbikingm...@gmail.com added the comment:
I looked at the existing patches - and noted that they went closer to how
Windows does short files - but still left out some cases.
I believe the latest patch catches all cases.
from http://msdn.microsoft.com/en-us/library/aa368590(v=vs
26 matches
Mail list logo
