Project

General

Profile

Actions

Bug #13324

closed

Python packaging broken

Added by Peter Amstutz over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Urgent
Assigned To:
Category:
-
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Story points:
-
Release:
Release relationship:
Auto

Description

(05:46:37 PM) tetron: we've been looking at it from the point of view of creating the packages
(05:46:50 PM) tetron: but setup.py also gets run to _install_ packages
(05:47:21 PM) tetron: it first looks for ARVADOS_BUILDING_VERSION
(05:47:27 PM) tetron: but that's not there, so version = None
(05:47:34 PM) tetron: then it goes into the if statement
(05:47:40 PM) tetron: it tries to import arvados_version
(05:47:53 PM) tetron: that's not included in the package, so it raises ImportError
(05:47:54 PM) tetron: which is caught
(05:47:57 PM) tetron: and passes
(05:48:06 PM) tetron: so it runs setup.py with version=None
(05:48:10 PM) tetron: which is why we get 0.0.0
(05:48:53 PM) tetron: this worked because before, we had a base case version="0.1" 
(05:49:21 PM) tetron: so the package doesn't actually know its own version
(05:49:37 PM) tetron: at least, not without using egg_info
(05:49:49 PM) Fernando: ok, but why you say that arvados_version is not included?
(05:50:50 PM) tetron: double checking, but I don't see it in MANIFEST.in
(05:51:10 PM) tetron: yea it isn't in the tarball
(05:51:20 PM) tetron: however
(05:51:23 PM) tetron: even if it was
(05:51:32 PM) tetron: it would have raised an error that it couldn't run git
(05:51:40 PM) tetron: which is the _other_ error we were seeing
Actions #1

Updated by Peter Amstutz over 6 years ago

  • Status changed from New to In Progress
Actions #2

Updated by Peter Amstutz over 6 years ago

  • Description updated (diff)
Actions #4

Updated by Peter Amstutz over 6 years ago

  • Status changed from In Progress to Resolved
  • Assigned To set to Peter Amstutz
  • Priority changed from Normal to Urgent
Actions #5

Updated by Tom Morris over 6 years ago

  • Release set to 13
Actions

Also available in: Atom PDF