You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
zlib: Expose zlibVersion to query runtime version of zlib #56515
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee=Noneclosed_at=<Date2011-09-11.22:13:10.028>created_at=<Date2011-06-10.00:51:03.953>labels= ['type-feature', 'library']
title='zlib: Expose zlibVersion to query runtime version of zlib'updated_at=<Date2011-09-11.22:13:10.027>user='https://github.com/Bluehorn'
I am currently fighting a curious problem in using zlib from Python: Decompression completes but sometimes the resulting output does not match a sha224 hash of the expected output.
I deployed a zlib 1.2.5 with our PyInstaller output but the problem persists. I would like to check that the program is actually using the right zlib version.
zlib provides the function zlibVersion to access the actual run time version. The attached patch exposes this function from the zlib module. I also added a unit test to check that the function is there and the output matches ZLIB_VERSION (which should match on the build machine).
Good idea, and thanks for posting a patch! This is a new feature, so 3.3-only. Also, you'll need to update the docs (Doc/library/zlib.rst, presumably) to add the new module method/attribute (with a suitable "versionadded" tag).
semantically it should probably be a module-level attribute, rather than a method; after all it's kind of a constant; perhaps ZLIB_RUNTIME_VERSION?
Also, I see that ZLIB_VERSION itself isn't documented, it would be nice to have a separate patch to fix that.
Bluehorn commented on Jun 13, 2011
Bluehornmannequin
MannequinAuthor
More actions
Here is an updated patch:
Function zlibVersion() replaced by module-level constant ZLIB_RUNTIME_VERSION
Added documentation (with versionadded: 3.3)
Bluehorn commented on Jun 13, 2011
Bluehornmannequin
MannequinAuthor
More actions
Documentation for ZLIB_VERSION.
I just notice that it makes no sense to add the version info to that section. Feel free to move the two snippets.
I added a versionadded tag here as well (ZLIB_VERSION was already in Python 1.5). Is there any old release for that versionadded tags should be removed?
I've committed your patches. I took the liberty of removing the
"versionadded" tag for ZLIB_VERSION; I don't think many people will need
to worry about compatibility with Python 1.5 ;-)
Activity
Bluehorn commentedon Jun 10, 2011
I am currently fighting a curious problem in using zlib from Python: Decompression completes but sometimes the resulting output does not match a sha224 hash of the expected output.
I deployed a zlib 1.2.5 with our PyInstaller output but the problem persists. I would like to check that the program is actually using the right zlib version.
zlib provides the function zlibVersion to access the actual run time version. The attached patch exposes this function from the zlib module. I also added a unit test to check that the function is there and the output matches ZLIB_VERSION (which should match on the build machine).
pitrou commentedon Jun 10, 2011
Good idea, and thanks for posting a patch! This is a new feature, so 3.3-only. Also, you'll need to update the docs (Doc/library/zlib.rst, presumably) to add the new module method/attribute (with a suitable "versionadded" tag).
Nitpicking:
Also, I see that ZLIB_VERSION itself isn't documented, it would be nice to have a separate patch to fix that.
Bluehorn commentedon Jun 13, 2011
Here is an updated patch:
Bluehorn commentedon Jun 13, 2011
Documentation for ZLIB_VERSION.
I just notice that it makes no sense to add the version info to that section. Feel free to move the two snippets.
I added a versionadded tag here as well (ZLIB_VERSION was already in Python 1.5). Is there any old release for that versionadded tags should be removed?
python-dev commentedon Sep 11, 2011
New changeset b21d1de6d78e by Nadeem Vawda in branch 'default':
Issue bpo-12306: Add ZLIB_RUNTIME_VERSION to the zlib module.
http://hg.python.org/cpython/rev/b21d1de6d78e
nadeemvawda commentedon Sep 11, 2011
I've committed your patches. I took the liberty of removing the
"versionadded" tag for ZLIB_VERSION; I don't think many people will need
to worry about compatibility with Python 1.5 ;-)
Once again, thanks for the patches!