summaryrefslogtreecommitdiff
path: root/ale_linters
diff options
context:
space:
mode:
authorKeith Pinson <kazark@zoho.com>2017-10-06 17:26:50 -0400
committerKeith Pinson <kazark@zoho.com>2017-10-06 17:26:50 -0400
commitf21f52343db942bd39ed6a2bb196d01aaf1f741f (patch)
tree7a5770a570657a321087c0dcc7f8d0d0d524d0e9 /ale_linters
parent47577564a2294668d7997b261e17fd97f0dd1fe6 (diff)
downloadale-f21f52343db942bd39ed6a2bb196d01aaf1f741f.zip
Fix Elm linter for Windows (resolves #980)
Looks like elm-make only respects /dev/null, even on Windows. The person who wrote this linter maybe did not test it on Windows, and wrote the code in the way you would expect to be solid by using NUL on Windows. However it seems elm-make is not actually making use of /dev/null but rather using it as a form of flag. Ironically this seems to be what is already described in the comments; I added some clarification.
Diffstat (limited to 'ale_linters')
-rw-r--r--ale_linters/elm/make.vim4
1 files changed, 2 insertions, 2 deletions
diff --git a/ale_linters/elm/make.vim b/ale_linters/elm/make.vim
index 4038e3b4..3783b5e3 100644
--- a/ale_linters/elm/make.vim
+++ b/ale_linters/elm/make.vim
@@ -71,11 +71,11 @@ function! ale_linters#elm#make#GetCommand(buffer) abort
" The elm-make compiler, at the time of this writing, uses '/dev/null' as
" a sort of flag to tell the compiler not to generate an output file,
- " which is why this is hard coded here.
+ " which is why this is hard coded here. It does not use NUL on Windows.
" Source: https://github.com/elm-lang/elm-make/blob/master/src/Flags.hs
let l:elm_cmd = ale#Escape(l:elm_exe)
\ . ' --report=json'
- \ . ' --output=' . ale#Escape(g:ale#util#nul_file)
+ \ . ' --output=/dev/null'
return l:dir_set_cmd . ' ' . l:elm_cmd . ' %t'
endfunction