Check on save is generating a debug directory instead of a release directory about rust-enhanced HOT 3 CLOSED

Because rust_syntax_checking_include_tests is set to true, the plugin is hard-coded to use --profile=test which will override your --release setting. This is a limitation of cargo, and not something we can work around.

I'm curious why you are using release mode for clippy? There shouldn't be any significant differences between that and the dev profile.

from rust-enhanced.

Ah, this is my first time "debugging" a sublime package, but I was able to print out how Rust Enhanced is invoking cargo:

$ cargo clippy --lib --release --message-format=json --profile=test
warning: the `--release` flag should not be specified with the `--profile` flag
The `--release` flag will be ignored.
This was historically accepted, but will become an error in a future release.

@ehuss you beat me to it :)

I'm curious why you are using release mode for clippy?

I change pretty often between the terminal and sublime text, running cargo clippy and cargo run in both.

For some projects which have a hefty build step, I'd like to re-use all the cached compilation artifacts in one profile instead of dancing between two of them.

Indeed setting rust_syntax_checking_include_tests to false returns the behavior to what I would expect.

Thanks for the quick response!

from rust-enhanced.

For anyone following along, I worked around this by using the following RustEnhanced.sublime-settings:

{
    "rust_syntax_checking_method": "clippy",
    // Set this to false or else it will run cargo without --release
    "rust_syntax_checking_include_tests": false,
    "cargo_build":
    {
        "defaults":
        {
            "release": true,
        },
        "variants":
        {
            "clippy":
            {
                "extra_cargo_args": "--all-targets",
                "extra_run_args": "-D warnings",
            },
        },
    },
}

The --all-targets flag will also run clippy on your tests. The -D warning just denies warnings, not necessary for a workaround but figured I'd post my settings in full.

from rust-enhanced.