Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

[rls::build::cargo] error deleting file #1474

Closed
schomatis opened this issue Jun 4, 2019 · 2 comments
Closed

[rls::build::cargo] error deleting file #1474

schomatis opened this issue Jun 4, 2019 · 2 comments

Comments

@schomatis
Copy link
Contributor

I'm getting this spurious error frequently in the logs, doesn't seem important but I was wondering if (following 801cef4) it could be demoted to a trace (if not altogether made silent).

debug!("error deleting file, {}: {}", name, e);

It seems that exec (at least in my local setup) takes long enough to have running concurrently the check with and without --test, with the two threads competing to remove the same file.

@schomatis
Copy link
Contributor Author

I'm also not sure why do we explicitly remove the .json file, it seems the handler just overwrites it in

https://github.com/rust-lang/rust/blob/817d2feb13dd1faad47ec699cd473b8be2093ec9/src/librustc_save_analysis/lib.rs#L1052-L1056

mati865 pushed a commit to mati865/rls that referenced this issue Jun 7, 2019
…7, r=Xanewok

Bump serde from 1.0.85 to 1.0.87

Bumps [serde](https://github.com/serde-rs/serde) from 1.0.85 to 1.0.87.
<details>
<summary>Release notes</summary>

*Sourced from [serde's releases](https://github.com/serde-rs/serde/releases).*

> ## v1.0.87
> - Fix inclusion of tag in struct containing flattened fields ([rust-lang#1468](https://github-redirect.dependabot.com/serde-rs/serde/issues/1468), thanks [**jwillbold**](https://github.com/jwillbold))
>
> ## v1.0.86
> - Implement Serialize and Deserialize for core::ops::Bound\<T\> ([rust-lang#1466](https://github-redirect.dependabot.com/serde-rs/serde/issues/1466), thanks [**0nkery**](https://github.com/0nkery))
</details>
<details>
<summary>Commits</summary>

- [`134f268`](serde-rs/serde@134f268) Release 1.0.87
- [`c473633`](serde-rs/serde@c473633) Format with rustfmt 2018-12-10
- [`6a3a820`](serde-rs/serde@6a3a820) Merge pull request [rust-lang#1474](https://github-redirect.dependabot.com/serde-rs/serde/issues/1474) from jwillbold/master
- [`1d6ef76`](serde-rs/serde@1d6ef76) Fixed [rust-lang#1468](https://github-redirect.dependabot.com/serde-rs/serde/issues/1468), flattened struct fields made structs ignore their tag
- [`c8e3959`](serde-rs/serde@c8e3959) Release 1.0.86
- [`796f412`](serde-rs/serde@796f412) Document that Bound<T> impls exist
- [`fa854a2`](serde-rs/serde@fa854a2) Format with rustfmt 2018-12-10
- [`3a097ff`](serde-rs/serde@3a097ff) Deserialize Bound::Unbounded as unit variant
- [`8463bfc`](serde-rs/serde@8463bfc) Remove as yet unrequested range impls
- [`7a72b4c`](serde-rs/serde@7a72b4c) Merge pull request [rust-lang#1466](https://github-redirect.dependabot.com/serde-rs/serde/issues/1466) from 0nkery/master
- Additional commits viewable in [compare view](serde-rs/serde@v1.0.85...v1.0.87)
</details>
<br />

[![Dependabot compatibility score](https://api.dependabot.com/badges/compatibility_score?dependency-name=serde&package-manager=cargo&previous-version=1.0.85&new-version=1.0.87)](https://dependabot.com/compatibility-score.html?dependency-name=serde&package-manager=cargo&previous-version=1.0.85&new-version=1.0.87)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
- `@dependabot badge me` will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in the `.dependabot/config.yml` file in this repo:
- Update frequency (including time of day and day of week)
- Automerge options (never/patch/minor, and dev/runtime dependencies)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

</details>
mati865 pushed a commit to mati865/rls that referenced this issue Jun 7, 2019
…e-1.0.87, r=Xanewok

Bump serde_derive from 1.0.85 to 1.0.87

Bumps [serde_derive](https://github.com/serde-rs/serde) from 1.0.85 to 1.0.87.
<details>
<summary>Release notes</summary>

*Sourced from [serde_derive's releases](https://github.com/serde-rs/serde/releases).*

> ## v1.0.87
> - Fix inclusion of tag in struct containing flattened fields ([rust-lang#1468](https://github-redirect.dependabot.com/serde-rs/serde/issues/1468), thanks [**jwillbold**](https://github.com/jwillbold))
>
> ## v1.0.86
> - Implement Serialize and Deserialize for core::ops::Bound\<T\> ([rust-lang#1466](https://github-redirect.dependabot.com/serde-rs/serde/issues/1466), thanks [**0nkery**](https://github.com/0nkery))
</details>
<details>
<summary>Commits</summary>

- [`134f268`](serde-rs/serde@134f268) Release 1.0.87
- [`c473633`](serde-rs/serde@c473633) Format with rustfmt 2018-12-10
- [`6a3a820`](serde-rs/serde@6a3a820) Merge pull request [rust-lang#1474](https://github-redirect.dependabot.com/serde-rs/serde/issues/1474) from jwillbold/master
- [`1d6ef76`](serde-rs/serde@1d6ef76) Fixed [rust-lang#1468](https://github-redirect.dependabot.com/serde-rs/serde/issues/1468), flattened struct fields made structs ignore their tag
- [`c8e3959`](serde-rs/serde@c8e3959) Release 1.0.86
- [`796f412`](serde-rs/serde@796f412) Document that Bound<T> impls exist
- [`fa854a2`](serde-rs/serde@fa854a2) Format with rustfmt 2018-12-10
- [`3a097ff`](serde-rs/serde@3a097ff) Deserialize Bound::Unbounded as unit variant
- [`8463bfc`](serde-rs/serde@8463bfc) Remove as yet unrequested range impls
- [`7a72b4c`](serde-rs/serde@7a72b4c) Merge pull request [rust-lang#1466](https://github-redirect.dependabot.com/serde-rs/serde/issues/1466) from 0nkery/master
- Additional commits viewable in [compare view](serde-rs/serde@v1.0.85...v1.0.87)
</details>
<br />

[![Dependabot compatibility score](https://api.dependabot.com/badges/compatibility_score?dependency-name=serde_derive&package-manager=cargo&previous-version=1.0.85&new-version=1.0.87)](https://dependabot.com/compatibility-score.html?dependency-name=serde_derive&package-manager=cargo&previous-version=1.0.85&new-version=1.0.87)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
- `@dependabot badge me` will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in the `.dependabot/config.yml` file in this repo:
- Update frequency (including time of day and day of week)
- Automerge options (never/patch/minor, and dev/runtime dependencies)
- Pull request limits (per update run and/or open at any time)
- Out-of-range updates (receive only lockfile updates, if desired)
- Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

</details>
@schomatis
Copy link
Contributor Author

Closing in favor of #1485.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant