Merge pull request #1806 from jimenezrick/fix-rustc-linter

Fix #1684: Use the correct flag name in rustc linter
This commit is contained in:
w0rp 2018-08-12 09:41:36 +01:00 committed by GitHub
commit a88a6d5285
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 5 deletions

View File

@ -1,7 +1,7 @@
" Author: Daniel Schemala <istjanichtzufassen@gmail.com>
" Description: rustc for rust files
call ale#Set('rust_rustc_options', '-Z no-trans')
call ale#Set('rust_rustc_options', '-Z no-codegen')
function! ale_linters#rust#rustc#RustcCommand(buffer) abort
" Try to guess the library search path. If the project is managed by cargo,

View File

@ -148,11 +148,11 @@ rustc *ale-rust-rustc*
g:ale_rust_rustc_options *g:ale_rust_rustc_options*
*b:ale_rust_rustc_options*
Type: |String|
Default: `'-Z no-trans'`
Default: `'-Z no-codegen'`
The variable can be used to change the options passed to `rustc`.
`-Z no-trans` should only work with nightly builds of Rust. Be careful when
`-Z no-codegen` should only work with nightly builds of Rust. Be careful when
setting the options, as running `rustc` could execute code or generate
binary files.

View File

@ -5,7 +5,7 @@ After:
call ale#assert#TearDownLinterTest()
Execute(The default command should be correct):
AssertLinter 'rustc', 'rustc --error-format=json -Z no-trans -'
AssertLinter 'rustc', 'rustc --error-format=json -Z no-codegen -'
Execute(The options should be configurable):
let b:ale_rust_rustc_options = '--foo'
@ -15,7 +15,7 @@ Execute(The options should be configurable):
Execute(Some default paths should be included when the project is a Cargo project):
call ale#test#SetFilename('cargo_paths/test.rs')
AssertLinter 'rustc', 'rustc --error-format=json -Z no-trans'
AssertLinter 'rustc', 'rustc --error-format=json -Z no-codegen'
\ . ' -L ' . ale#Escape(ale#path#GetAbsPath(g:dir, 'cargo_paths/target/debug/deps'))
\ . ' -L ' . ale#Escape(ale#path#GetAbsPath(g:dir, 'cargo_paths/target/release/deps'))
\ . ' -'