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

rustfmt 1.5.2 #121441

Closed
wants to merge 2 commits into from
Closed

rustfmt 1.5.2 #121441

wants to merge 2 commits into from

Conversation

Porkepix
Copy link
Contributor

Created by brew bump


Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added bump-formula-pr PR was created using `brew bump-formula-pr` rust Rust use is a significant feature of the PR or issue labels Jan 25, 2023
@chenrui333
Copy link
Member

  ==> rustfmt --check ./src/main.rs
  dyld[81216]: Library not loaded: '@rpath/librustc_driver-f64ac9ee9a3f0eb4.dylib'
    Referenced from: '/opt/homebrew/Cellar/rustfmt/1.5.2/bin/rustfmt'
    Reason: tried: '/usr/local/lib/librustc_driver-f64ac9ee9a3f0eb4.dylib' (no such file), '/usr/lib/librustc_driver-f64ac9ee9a3f0eb4.dylib' (no such file)

@chenrui333 chenrui333 added test failure CI fails while running the test-do block help wanted Task(s) needing PRs from the community or maintainers labels Jan 25, 2023
@SMillerDev
Copy link
Member

Is this an issue with rustup usage?

@carlocab
Copy link
Member

Is this an issue with rustup usage?

Most likely, yes. We're using a pretty old nightly to build this, though. Maybe the rust formula works now? It may be a runtime dep.

@chenrui333
Copy link
Member

chenrui333 commented Jan 26, 2023

Is this an issue with rustup usage?

Most likely, yes. We're using a pretty old nightly to build this, though. Maybe the rust formula works now? It may be a runtime dep.

yeah, it depends on a nightly build, I tried the specified nightly build, but did not quite work out for me.
https://github.com/rust-lang/rustfmt/blob/master/CHANGELOG.md?plain=1#L26-L29

@chenrui333
Copy link
Member

the build went fine in my local, but in runtime it reference some librustc_driver

==> rustfmt --check ./src/main.rs
Last 15 lines from /Users/rui/Library/Logs/Homebrew/rustfmt/test.02.rustfmt:
2023-01-26 10:32:30 -0500

rustfmt
--check
./src/main.rs

dyld[52306]: Library not loaded: @rpath/librustc_driver-f64ac9ee9a3f0eb4.dylib
  Referenced from: <2CCD3F33-AAEE-3ADE-A78C-AC78D489E170> /opt/homebrew/Cellar/rustfmt/1.5.2/bin/rustfmt
  Reason: tried: '/System/Volumes/Preboot/Cryptexes/OS@rpath/librustc_driver-f64ac9ee9a3f0eb4.dylib' (no such file), '/usr/local/lib/librustc_driver-f64ac9ee9a3f0eb4.dylib' (no such file), '/usr/lib/librustc_driver-f64ac9ee9a3f0eb4.dylib' (no such file, not in dyld cache)
Error: rustfmt: failed

Comment on lines +28 to +29
# pinning to nightly-2023-01-24 to avoid inconstency
nightly_version = "nightly-2023-01-24"
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if we can just read this from the rust-toolchain file so that this doesn't have to be updated manually.

Copy link
Member

Choose a reason for hiding this comment

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

that makes sense.

Copy link
Member

Choose a reason for hiding this comment

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

note rustup's cargo already reads the version from that file, so unless we want to be explicit all we need is

    system "#{Formula["rustup-init"].bin}/rustup-init", "-qy", "--no-modify-path"
    ENV.prepend_path "PATH", HOMEBREW_CACHE/"cargo_cache/bin"
    system "cargo", "install", *std_cargo_args
Example

==> cargo install --locked --root /opt/homebrew/Cellar/rustfmt/1.5.2 --path .
info: syncing channel updates for 'nightly-2023-01-24-aarch64-apple-darwin'
info: latest update on 2023-01-24, rust version 1.69.0-nightly (c8e6a9e8b 2023-01-23)
info: downloading component 'cargo'
info: downloading component 'clippy'
info: downloading component 'llvm-tools'
info: downloading component 'rust-docs'
info: downloading component 'rust-std'
info: downloading component 'rustc'
info: downloading component 'rustc-dev'
info: downloading component 'rustfmt'
info: installing component 'cargo'
info: installing component 'clippy'
info: installing component 'llvm-tools'
info: installing component 'rust-docs'
info: installing component 'rust-std'
info: installing component 'rustc'
info: installing component 'rustc-dev'
info: installing component 'rustfmt'
  Installing rustfmt-nightly v1.5.2 (/private/tmp/rustfmt-20230126-48518-ucx7qv/rustfmt-1.5.2)
...

@p-linnane p-linnane removed the help wanted Task(s) needing PRs from the community or maintainers label Apr 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 3, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

@github-actions github-actions bot added the stale No recent activity label Apr 3, 2023
@github-actions github-actions bot closed this Apr 5, 2023
@henryhchchc henryhchchc mentioned this pull request Jul 19, 2023
6 tasks
@chenrui333
Copy link
Member

#137033

@chenrui333 chenrui333 added the superseded PR was replaced by another PR label Jul 20, 2023
@Porkepix Porkepix deleted the bump-rustfmt-1.5.2 branch February 5, 2024 16:13
@github-actions github-actions bot added the outdated PR was locked due to age label Mar 11, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` outdated PR was locked due to age rust Rust use is a significant feature of the PR or issue stale No recent activity superseded PR was replaced by another PR test failure CI fails while running the test-do block
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants