Use non-strict version dependencies for mozbase modules

RESOLVED FIXED in Firefox 38

Status

Testing
Firefox UI Tests
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

41 Branch
mozilla41
Points:
---

Firefox Tracking Flags

(firefox38 fixed, firefox39 fixed, firefox40 fixed, firefox41 fixed, firefox-esr38 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
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

3 years ago
Created attachment 8612450 [details] [review]
github_pull_request.txt

PR for loose version deps. r+ taken over from Armen's reply on the Github PR.
Attachment #8612450 - Flags: review+
(Assignee)

Comment 2

3 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

3 years ago
https://github.com/mozilla/firefox-ui-tests/commit/2a2fbd144411065cba5b746333e8c0af04744bd0 (master)
status-firefox41: affected → fixed
Target Milestone: --- → Firefox 41
(Assignee)

Comment 4

3 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
Last Resolved: 3 years ago
status-firefox38: affected → fixed
status-firefox39: affected → fixed
status-firefox40: affected → fixed
status-firefox-esr38: affected → fixed
Resolution: --- → FIXED
Product: Mozilla QA → Testing
You need to log in before you can comment on or make changes to this bug.