Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

should the Python module's version always match the library version? #150

Closed
anthrotype opened this issue Sep 8, 2015 · 6 comments
Closed

Comments

@anthrotype
Copy link
Member

v0.2.0 has been tagged, but the Python package version as stored in python/brotlimodule.cc still says "0.1.0" (see here).
This version string is read by the setup.py, and stored in the package metadata upon installing.

I wonder if the Python module's version should also increase every time a Brotli update is tagged, or if language bindings should have their own version numbers, independent from the core library.

WDYT?

@szabadka
Copy link
Collaborator

I think the Python module's version should refer to the version of the underlying Brotli library, but it seems difficult to keep them in sync automatically. Any ideas would be welcome.

If we would do it manually, we would have to remember to bump the version number right before we tag the release, right? Doing it right after would be too late since everything built from the tagged release would have the wrong version number (as with v0.2.0). I will try to remember doing this going forward.

@anthrotype
Copy link
Member Author

here they use a post-commit git hook which automatically adds a new tag whenever the version string is modified.

I don't know if it's worth doing it though.

@szabadka
Copy link
Collaborator

It is not clear if this would work if the version is updated through merging a pull request. Anyway, let's try to remember updating the version string before release for now and see if that works. The bro.cc does not have a --version flag yet, before we attempt to do this automation, I want to move the BROTLI_VERSION define to a common place and use it from the bro.cc as well.

@anthrotype
Copy link
Member Author

yes, that sounds good to me. Thanks.

@szabadka
Copy link
Collaborator

szabadka commented Oct 2, 2015

I created the version.h header in PR #193

Closing this now as the question in the issue was decided.

@szabadka szabadka closed this as completed Oct 2, 2015
@anthrotype
Copy link
Member Author

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants