Closed
Bug 872285
Opened 12 years ago
Closed 12 years ago
bump packages and mirror to m-c for bug 790765
Categories
(Testing :: Mozbase, defect)
Testing
Mozbase
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: k0scist, Assigned: k0scist)
References
Details
Attachments
(1 file)
43.36 KB,
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
At least mozprocess, mozprofile, and mozfile will need to be version
bumped and released to pypi and m-c to unblock bug 790765.
│lsdiff https://bugzilla.mozilla.org/attachment.cgi?id=749059
a/testing/mozbase/mozprocess/tests/test_mozprocess.py
a/testing/mozbase/mozprofile/setup.py
a/testing/mozbase/mozprofile/tests/bug785146.py
a/testing/mozbase/mozprofile/tests/permissions.py
a/testing/mozbase/mozprofile/tests/server_locations.py
a/testing/mozbase/mozprofile/tests/test_preferences.py
the mozfile changes from bug 872138 are also needed
anything that uses mozfile.NamedTemporaryFile should be pegged to the
new version, just to be safe (IMHO)
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
The mozfile changes are both necessary and stable. Will bump that
Assignee | ||
Comment 2•12 years ago
|
||
(In reply to Jeff Hammel [:jhammel] from comment #1)
> The mozfile changes are both necessary and stable. Will bump that
done: https://bugzilla.mozilla.org/show_bug.cgi?id=872138#c7
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → jhammel
Assignee | ||
Comment 3•12 years ago
|
||
Looks like we're doin'
mozprofile -> 0.9
mozprocess -> 0.10
Assignee | ||
Comment 4•12 years ago
|
||
bumped, tagged, and pypied: https://github.com/mozilla/mozbase/commit/e431a0badeda05225fdba7e52bc516d935c37889
Assignee | ||
Comment 5•12 years ago
|
||
Attachment #753954 -
Flags: review?(ahalberstadt)
Comment 6•12 years ago
|
||
Comment on attachment 753954 [details] [diff] [review]
updates from github
Review of attachment 753954 [details] [diff] [review]:
-----------------------------------------------------------------
After cursory review, I give this the rubber stamp of approval.
Attachment #753954 -
Flags: review?(ahalberstadt) → review+
Assignee | ||
Comment 7•12 years ago
|
||
pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=f699d5ac87e7
Comment 8•12 years ago
|
||
Did this pick up the mozcrash changes in bug 761586? I'd like to get that to m-c.
Assignee | ||
Comment 9•12 years ago
|
||
While there are errors on the try run, nothing seems to be related to what is here as best I can tell. Going to push
Assignee | ||
Comment 10•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•12 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #8)
> Did this pick up the mozcrash changes in bug 761586? I'd like to get that to
> m-c.
Nope. I filed a follow-up for these issues: https://bugzilla.mozilla.org/show_bug.cgi?id=876761
Comment 12•12 years ago
|
||
Flags: in-testsuite+
Comment 13•12 years ago
|
||
Thanks for the followup!
You need to log in
before you can comment on or make changes to this bug.
Description
•