Closed Bug 1280789 Opened 8 years ago Closed 8 years ago

unbreak mingw builds after 1272887

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox50 fixed)

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: c, Unassigned)

References

Details

Attachments

(1 file)

Attached patch unbreak mingw builds (deleted) — Splinter Review
No description provided.
Attachment #8763351 - Flags: review?(till)
Blocks: 1272887
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)
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.
Comment on attachment 8763351 [details] [diff] [review] unbreak mingw builds Oops, forgot to r+ this before landing!
Attachment #8763351 - Flags: review?(sphink) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: