Closed
Bug 855465
Opened 11 years ago
Closed 11 years ago
Add emacs python mode comments to moz.build files
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla23
People
(Reporter: qdot, Assigned: qdot)
References
Details
Attachments
(1 file, 2 obsolete files)
615.16 KB,
patch
|
Details | Diff | Splinter Review |
moz.build files come with vi modesets. [insert religious war comment here]. We should add emacs also. I realize there's more than 2 python modes in emacs history, so I'm going to assume modern and go with fgallina's python-mode. Just need to add the following line to the top of all the moz.build files (assuming we're doing pep8 compatible python?): # -*- Mode: python; c-basic-offset: 4; indent-tabs-mode: nil; tab-width: 40 -*-
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Comment 2•11 years ago
|
||
Whoops, just wanted moz.build, not *.build.
Attachment #730358 -
Attachment is obsolete: true
Attachment #730360 -
Flags: review?(gps)
Comment 3•11 years ago
|
||
Comment on attachment 730358 [details] [diff] [review] Patch (v1) - Add emacs python-mode comments to moz.build files That makes me sad that the header now has to be 5 lines...
Assignee | ||
Comment 4•11 years ago
|
||
It does feel a little weird that the header will probably be longer than a lot of the files, but since it's a whitespace sensitive language, it'd be nice to get this locked down before inevitable untested-one-line-change burnage.
Comment 5•11 years ago
|
||
Comment on attachment 730360 [details] [diff] [review] Patch (v2) - Add emacs python-mode comments to moz.build files Review of attachment 730360 [details] [diff] [review]: ----------------------------------------------------------------- Maybe one day we'll rename all these files to .py so we won't have this problem. Until then, rs=gps.
Attachment #730360 -
Flags: review?(gps) → review+
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/03452b187c14
Assignee | ||
Comment 7•11 years ago
|
||
Bustaaaaaaaaaage. https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=03452b187c14
Assignee | ||
Comment 8•11 years ago
|
||
Fixed busted tests. https://tbpl.mozilla.org/?tree=Try&rev=cb1a988a2bb2
Assignee | ||
Comment 9•11 years ago
|
||
Needed to update test_reader line numbers
Attachment #730360 -
Attachment is obsolete: true
Comment 10•11 years ago
|
||
I granted IRC r+ post check-in.
Comment 11•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5f4f68f00419
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Comment 12•11 years ago
|
||
I see you ignored the # THIS FILE IS AUTOGENERATED BY importTestsuite.py - DO NOT EDIT comments.
Comment 13•6 years ago
|
||
Pushed by frgrahl@gmx.net: https://hg.mozilla.org/comm-central/rev/39dc6e2c2140 Add emacs python mode comments to moz.build files; r=gps https://hg.mozilla.org/comm-central/rev/2c4e3be38014 Backout for changeset 39dc6e2c2140 due to bustage on a CLOSED TREE; r=qdot https://hg.mozilla.org/comm-central/rev/51d07d8d4ebf Add emacs python mode comments to moz.build files; r=gps
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•