RLS and the compiler keep rebuilding after each other

[sbstp]

I'm working on a project with a fairly long compile time and it seems like the RLS and the compiler disagree on the artifacts, I'll compile from the command line with cargo build and open up VS Code only to see VS code rebuild everything. After VS Code has rebuilt, the next cargo build rebuilds everything again.

I'm using Rust nightly with the latest vscode and I have no settings set for the RLS, so everything is set to default.

The RLS extension says the version is 0.4.1, here's other environment info:

$ rustup show
Default host: x86_64-unknown-linux-gnu

installed toolchains
--------------------

stable-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu (default)

active toolchain
----------------

nightly-x86_64-unknown-linux-gnu (default)
rustc 1.26.0-nightly (517f24025 2018-03-31)

$ code --version
1.21.1
79b44aa704ce542d8ca4a3cc44cfca566e7720f1
x64

$ uname -a
Linux debbie 4.15.0-2-amd64 #1 SMP Debian 4.15.11-1 (2018-03-20) x86_64 GNU/Linux

[sbstp]

To clarify, it is not a short/incremental rebuild, it's a full blown first build as if cargo clean had been run between cargo build and opening vscode / opening vscode and then running cargo build, the order doesn't seem to matter.

[Bobo1239]

I believe #793 fixed this (rls is supposed to use a separate directory than a normal build) but the rls distributed with the nightlies doesn't include that change yet.

[alexheretic]

manual cargo builds use ./target/, Rls builds use ./target/rls so a manual build will not accelerate an Rls build. In this way both builds are independent.

On the other hand, cargo clean will simply remove ./target so will clear both manual and rls builds.

[Bobo1239]

Yeah that's what is supposed to happen but the rls included with the nightlies is currently using ./target/debug. I'll open a PR to update the submodule.
edit: PR is up.

[Xanewok]

Yep, I'm the one to blame for that, I'm afraid. #788 makes it so it uses whichever target/ directory Cargo points at, but #793 fixes that and uses previous target/rls directory (but still using target/ directory that Cargo uses)

[nrc]

Closing since this is fixed on master