Add an emacs mode for gn files.

Add an emacs mode for gn files.

BUG=none

Committed: https://crrev.com/3074045136e6a83a08de513e847c68ccfb71e4dd
Cr-Commit-Position: refs/heads/master@{#323075}

[Elliot Glaysher, 2015-03-31 17:44:45]

erg@chromium.org changed reviewers:
+ thakis@chromium.org

[Elliot Glaysher, 2015-03-31 17:44:45]

Hi Nico, you appear to be one of the few people who have reviewed elisp patches.

[tfarina, 2015-03-31 17:46:27]

Could you add this in tools/gn/misc/emacs instead?

[Nico, 2015-03-31 18:01:35]

seems ok. i don't use emacs and don't know elisp well (i have some scheme
experience). it looks similar to ninja-mode.el – but I don't know anything about
smie.

lgtm once tfarina is happy with the location of this

https://codereview.chromium.org/1048103003/diff/20001/tools/gn/misc/emacs/gn.el
File tools/gn/misc/emacs/gn.el (right):

https://codereview.chromium.org/1048103003/diff/20001/tools/gn/misc/emacs/gn....
tools/gn/misc/emacs/gn.el:73: (define-derived-mode gn-mode prog-mode "GN"
does this have to be ###autoload too?

https://codereview.chromium.org/1048103003/diff/20001/tools/gn/misc/emacs/gn....
tools/gn/misc/emacs/gn.el:91: (add-to-list 'auto-mode-alist '("$BUILD.gn^" .
gn-mode))
Shouldn't the ^ be in the front and the $ in the back?

https://codereview.chromium.org/1048103003/diff/20001/tools/gn/misc/emacs/gn....
tools/gn/misc/emacs/gn.el:92: (add-to-list 'auto-mode-alist '("\\.gni\\'" .
gn-mode))
$ instead of \\ at the end?

[tfarina, 2015-03-31 18:05:02]

tfarina@chromium.org changed reviewers:
+ tfarina@chromium.org

[tfarina, 2015-03-31 18:05:03]

thanks, I'm happy with the new location.

lgtm

[Elliot Glaysher, 2015-03-31 18:08:58]

The CQ bit was checked by erg@chromium.org

[Elliot Glaysher, 2015-03-31 18:08:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
tfarina@chromium.org, thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1048103003/#ps40001
(title: "thakis comments")

[commit-bot: I haz the power, 2015-03-31 18:09:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1048103003/40001

[commit-bot: I haz the power, 2015-03-31 18:29:23]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-31 18:29:54]

Patchset 3 (id:??) landed as
https://crrev.com/3074045136e6a83a08de513e847c68ccfb71e4dd
Cr-Commit-Position: refs/heads/master@{#323075}