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

python:: client.__version__:: erroneous 'v' #1285

Closed
adriansev opened this issue Sep 17, 2020 · 6 comments
Closed

python:: client.__version__:: erroneous 'v' #1285

adriansev opened this issue Sep 17, 2020 · 6 comments

Comments

@adriansev
Copy link
Contributor

Hi! Would it be possible to have (return to) strict decimal numbers in the module version?
Currently it is:

>>> print(client.__version__)
v5.0.1
@adriansev
Copy link
Contributor Author

adriansev commented Sep 17, 2020

and for 4.12.4 installed with pip3_install_user xrootd==4.12.4 i get

pip3_install_user xrootd==4.12.4
WARNING: Keyring is skipped due to an exception: Failed to unlock the collection!
Collecting xrootd==4.12.4
  Using cached xrootd-4.12.4.tar.gz (2.6 MB)
  WARNING: Requested xrootd==4.12.4 from https://files.pythonhosted.org/packages/87/06/705ab2671c08473f0175288080ffad73f01f17eec9eb5409e1965163869d/xrootd-4.12.4.tar.gz#sha256=86c5a6dc470dbb1ffafdecee26a5c125e9ecbede16c4b0a7c1104abb37cea518, but installing version 0.0.0
...

python3
Python 3.7.9 (default, Aug 19 2020, 17:05:11)
[GCC 9.3.1 20200408 (Red Hat 9.3.1-2)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from XRootD import client
>>> print(client.__version__)
v-

ping @simonmichal

@simonmichal
Copy link
Contributor

Hi @adriansev,

  • regarding the v- version, I just backported the respective commit to the stable branch (8f77fe3) my apologies it has not been done earlier

  • regarding the version format itself, my understanding is you would like to see 5.0.1 instead of v5.0.1? Yes, certainly that's doable.

@simonmichal
Copy link
Contributor

@adriansev : I'm stripping now the leading v from the __version__ -> 30350cd (also backported to stable branches).

@adriansev
Copy link
Contributor Author

@simonmichal Great!!! thanks a lot!! in principle the advantage of strictly numeric versions (also easily splittable) is that allow really easy version checking when a downstream application use the different signatures that can happen from version to version in upstream. (as i do for the checking hash when copy).

@simonmichal
Copy link
Contributor

@adriansev : is there anything else that needs to be sorted out, or can we close this one?

@adriansev
Copy link
Contributor Author

sorry, i forgot about this one, this can be closed (and of course i cannot test it until a release is out :) )

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