Project

General

Profile

Actions

Bug #5175

closed

[SDKs] Fix Python package versions so they can be uploaded to pypi

Added by Tom Clegg about 9 years ago. Updated about 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
SDKs
Target version:
Story points:
0.5

Subtasks 1 (0 open1 closed)

Task #5177: Review 5175-pep-440 (arvados + arvados-dev)ResolvedTom Clegg02/09/2015Actions

Related issues

Related to Arvados - Bug #4988: [SDKs] Fix incompatibility between Python packages and newer virtualenvClosedBrett Smith01/15/2015Actions
Actions #1

Updated by Brett Smith about 9 years ago

(12:14:23 PM) Me: tomclegg: As far as I can see, we should give up completely on ever trying to include the git commit hash in the version string.
(12:14:47 PM) Me: The local version identifier is the only part of the version string that would allow it under PEP 440, but we can't meet the other rules.
(12:14:55 PM) Me: Namely, "Local version identifiers SHOULD NOT be used for upstream projects."
(12:15:02 PM) Me: And, "As the Python Package Index is intended solely for indexing and hosting upstream projects, it MUST NOT allow the use of local version identifiers."
(12:16:03 PM) tomclegg: brett: possibly. It occurred to me that "local version" tags might be appropriate for packages built during development -- they shouldn't be confused with real packages that are published to pypi, etc...
(12:16:41 PM) Me: tomclegg: PEP 440 makes clear that it's meant to tag patched builds of an original version, e.g., in a distribution.
(12:17:20 PM) Me: If we want to mark our own development versions, we should use .devN. But N can only be an integer.
(12:19:21 PM) tomclegg: brett: Sure. Having an opinion about what to do with Python version numbers is a losing battle. PEP-440 has spoken.

Pushed to the branch to reflect this philosophy. Now at f32690a.

Actions #2

Updated by Tom Clegg about 9 years ago

  • Category set to SDKs
  • Status changed from New to Resolved
  • Assigned To set to Tom Clegg
Actions

Also available in: Atom PDF