Skip to content

Conversation

@chillum
Copy link

@chillum chillum commented Oct 14, 2014

This code:

(add-to-list 'auto-mode-alist
 '("nginx\.conf$" . nginx-mode)
 '("/etc/nginx/.*" . nginx-mode))

was broken in two ways:

  1. wrong regexps
  2. wrong add-to-list syntax: the third argument is append, not an additional item.

Changed the auto-mode-alist part so it works now, but only for */nginx/*.conf and nginx.conf.

Sites-enabled stuff should be disabled by default: clashes with apache-mode (mentioned this in docs).

Adding of all files under /etc/nginx is evil (they're not necessarily nginx configs actually), so disabled that as well.

ajc added a commit that referenced this pull request Aug 24, 2015
@ajc ajc merged commit 8a296e3 into ajc:master Aug 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants