Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added logging to crate_universe checksum validation #2657

Merged
merged 3 commits into from
May 16, 2024

Conversation

UebelAndre
Copy link
Collaborator

When CARGO_BAZEL_DEBUG=1 is enabled, the following log lines will print when determining repins.

Query 2024-05-16T15:17:26.592139Z DEBUG cargo_bazel::lockfile: cargo-bazel version hash: ef009193262ad048e8890de3f6c4c66f8f26584396e9b6f544894b337ab951ce
Query 2024-05-16T15:17:26.592402Z DEBUG cargo_bazel::lockfile: lockfile context hash: a89c68580b8ac127d1c3d3a700c3c895d107d0826a84c6594c0b4d6c55459af8
Query 2024-05-16T15:17:26.592417Z DEBUG cargo_bazel::lockfile: workspace config hash: baf129ecaaf01eda9020a45991fe804ce7def5e3dc600e8f4f9b99c814ab5635
Query 2024-05-16T15:17:26.592453Z DEBUG cargo_bazel::lockfile: splicing manifest hash: dceb142aa8d5be9cec3bf5d6318150b17c874a7ac84ca3fd9a608b3df279435c
Query 2024-05-16T15:17:26.592456Z DEBUG cargo_bazel::lockfile: Cargo version hash: 573f7bac3e83c5bd86e2fc369f5ecdafc3db984a9641f9f72b6bf68ab060f9d6
Query 2024-05-16T15:17:26.592458Z DEBUG cargo_bazel::lockfile: Rustc version hash: 25b2309140d57d18083da87e1597ef66785a693f2c12dbd7c6912728a2fbce1c
Query 2024-05-16T15:17:26.592461Z DEBUG cargo_bazel::lockfile: Digest hash: 81997a51e91981729d56b9118839163206d491451a10bf9611c6037d1ae2f665
Error: Digests do not match: Current Digest("4cdd2a75d814f49c49d7c02588accb2bc780d3cb1c0f718a83d7edffe4bbebe0") != Expected Digest("81997a51e91981729d56b9118839163206d491451a10bf9611c6037d1ae2f665")

Copy link
Collaborator

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super useful, thanks!

@UebelAndre UebelAndre enabled auto-merge May 16, 2024 15:27
@UebelAndre UebelAndre mentioned this pull request May 16, 2024
@UebelAndre UebelAndre added this pull request to the merge queue May 16, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 16, 2024
@UebelAndre UebelAndre enabled auto-merge May 16, 2024 15:45
@UebelAndre UebelAndre added this pull request to the merge queue May 16, 2024
Merged via the queue into bazelbuild:main with commit 4023d94 May 16, 2024
3 checks passed
rrbutani pushed a commit to bazel-contrib/toolchains_llvm that referenced this pull request May 16, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [rules_rust](https://togithub.com/bazelbuild/rules_rust) |
http_archive | minor | `0.44.0` -> `0.45.0` |

---

### Release Notes

<details>
<summary>bazelbuild/rules_rust (rules_rust)</summary>

###
[`v0.45.0`](https://togithub.com/bazelbuild/rules_rust/releases/tag/0.45.0)

[Compare
Source](https://togithub.com/bazelbuild/rules_rust/compare/0.44.0...0.45.0)

### 0.45.0

```python
load("@&#8203;bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
http_archive(
    name = "rules_rust",
    integrity = "sha256-0IoZbO7cp5obgv01ucsnAjrWdGpBasntBusUXtdj1aY=",
    urls = ["https://github.com/bazelbuild/rules_rust/releases/download/0.45.0/rules_rust-v0.45.0.tar.gz"],
)
```

Additional documentation can be found at:
https://bazelbuild.github.io/rules_rust/#setup

#### What's Changed

- Fix cc_common_link when using sibling repository layout by
[@&#8203;yuzhy8701](https://togithub.com/yuzhy8701) in
[bazelbuild/rules_rust#2643
- Allow runfiles object to be an arbitrary expression. by
[@&#8203;matts1](https://togithub.com/matts1) in
[bazelbuild/rules_rust#2651
- Allow searching for the rlocation of a module. by
[@&#8203;matts1](https://togithub.com/matts1) in
[bazelbuild/rules_rust#2652
- Add support for generated sources to rust-analyzer. by
[@&#8203;matts1](https://togithub.com/matts1) in
[bazelbuild/rules_rust#2654
- Added logging to crate_universe checksum validation by
[@&#8203;UebelAndre](https://togithub.com/UebelAndre) in
[bazelbuild/rules_rust#2657
- Release 0.45.0 by
[@&#8203;UebelAndre](https://togithub.com/UebelAndre) in
[bazelbuild/rules_rust#2658

#### New Contributors

- [@&#8203;yuzhy8701](https://togithub.com/yuzhy8701) made their first
contribution in
[bazelbuild/rules_rust#2643

**Full Changelog**:
bazelbuild/rules_rust@0.44.0...0.45.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/bazel-contrib/toolchains_llvm).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zNjMuNSIsInVwZGF0ZWRJblZlciI6IjM3LjM2My41IiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIiwibGFiZWxzIjpbXX0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Copy link
Collaborator Author

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Crap!

if result.stdout.strip().lower() == "repin":

This should be
if result.return_code:

@illicitonion would you be able to do me a huge favor and put up a fix for this and cut a new release? I won’t be available to author a change for some time and this is pretty breaking :(

illicitonion added a commit to illicitonion/rules_rust that referenced this pull request May 16, 2024
github-merge-queue bot pushed a commit that referenced this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants