Closed
Bug 1169374
Opened 9 years ago
Closed 9 years ago
Use non-strict version dependencies for mozbase modules
Categories
(Testing :: Firefox UI Tests, defect)
Tracking
(firefox38 fixed, firefox39 fixed, firefox40 fixed, firefox41 fixed, firefox-esr38 fixed)
RESOLVED
FIXED
mozilla41
People
(Reporter: whimboo, Assigned: whimboo)
Details
Attachments
(1 file)
Today we had the situation that our tests were broken because we hard-depend on mozinfo 0.7, while marionette-client itself depends on >= 0.7. With the release of mozinfo 0.8 today, this version got installed with marionette. Once our deps were examined the setup step runs into a version conflict between 0.8 and 0.7. I think that for packages in our own control we should have loose version dependencies. That would allow us to pick up the latest changes without having to always bump versions in our setup.py file. There might be a risk for seeing breakage in case of an incompatible new release. But in those cases we could temporarily have strict version deps until the problem got fixed. I will come up with a proposed patch. Let me know what you think about.
Assignee | ||
Comment 1•9 years ago
|
||
PR for loose version deps. r+ taken over from Armen's reply on the Github PR.
Attachment #8612450 -
Flags: review+
Assignee | ||
Comment 2•9 years ago
|
||
We would need that patch on all branches.
status-firefox38:
--- → affected
status-firefox39:
--- → affected
status-firefox40:
--- → affected
status-firefox-esr38:
--- → affected
Assignee | ||
Comment 3•9 years ago
|
||
https://github.com/mozilla/firefox-ui-tests/commit/2a2fbd144411065cba5b746333e8c0af04744bd0 (master)
Target Milestone: --- → Firefox 41
Assignee | ||
Comment 4•9 years ago
|
||
https://github.com/mozilla/firefox-ui-tests/commit/1a01abaced4e8e44555b12a2243757a5442f377a (aurora) https://github.com/mozilla/firefox-ui-tests/commit/b3ef479472aa10b268605d0433bf11bedbbbf15e (beta, release, esr38)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Mozilla QA → Testing
You need to log in
before you can comment on or make changes to this bug.
Description
•