Closed Bug 855465 Opened 8 years ago Closed 8 years ago
Add emacs python mode comments to moz
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 -*-
Whoops, just wanted moz.build, not *.build.
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...
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 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+
Fixed busted tests. https://tbpl.mozilla.org/?tree=Try&rev=cb1a988a2bb2
Needed to update test_reader line numbers
Attachment #730360 - Attachment is obsolete: true
I granted IRC r+ post check-in.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
I see you ignored the # THIS FILE IS AUTOGENERATED BY importTestsuite.py - DO NOT EDIT comments.
Pushed by firstname.lastname@example.org: 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
You need to log in before you can comment on or make changes to this bug.