Closed
Bug 1280789
Opened 8 years ago
Closed 8 years ago
unbreak mingw builds after 1272887
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox50 fixed)
RESOLVED
FIXED
mozilla50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: c, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
sfink
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Updated•8 years ago
|
Attachment #8763351 -
Flags: review?(till)
Comment 1•8 years ago
|
||
Comment on attachment 8763351 [details] [diff] [review]
unbreak mingw builds
Thank you for the patch. To be landable, it'll need to be in the right format. It almost is, but the description needs to be of the format "Bug NNN - Description. r=reviewer". I'm forwarding the review to sfink, so with that in mind the description should be "Bug 1280789 - Unbreak mingw builds. r=sfink"
Attachment #8763351 -
Attachment is patch: true
Attachment #8763351 -
Flags: review?(till) → review?(sphink)
Comment 2•8 years ago
|
||
I didn't want to wait for the patch to be cleaned up, and it's simple enough, so I just landed it.
Thanks for the patch! You've made quite a few patches now, so it would be nice if you could be generating them in the correct format (proper user in the patch, with unix line endings, and with appropriate commit messages for this project.) I wasn't sure whether to use zhoubcfan@163.com or c@lightme.us as the author, but I saw you have a bunch of patches landed with the first so I used that. Please submit things in a way that doesn't require landers to guess at your preferences.
Pushed by sfink@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2a807ca4df8e
Unbreak mingw builds, r=sfink
Comment 4•8 years ago
|
||
Comment on attachment 8763351 [details] [diff] [review]
unbreak mingw builds
Oops, forgot to r+ this before landing!
Attachment #8763351 -
Flags: review?(sphink) → review+
Comment 5•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•