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

rust 1.69.0 #128865

Merged
merged 2 commits into from
May 1, 2023
Merged

rust 1.69.0 #128865

merged 2 commits into from
May 1, 2023

Conversation

ZhongRuoyu
Copy link
Member

@ZhongRuoyu ZhongRuoyu commented Apr 20, 2023

Created with brew bump-formula-pr.

  • resource blocks have been checked for updates.

@github-actions github-actions bot added bump-formula-pr PR was created using `brew bump-formula-pr` CI-build-dependents-from-source Pass --build-dependents-from-source to brew test-bot. long build Set a long timeout for formula testing labels Apr 20, 2023
@ZhongRuoyu ZhongRuoyu added CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. labels Apr 20, 2023
@carlocab
Copy link
Member

Seems to have failed fast despite CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. .

I guess this doesn't work as expected:

fail-fast: ${{ !contains(github.event.pull_request.labels.*.name, 'CI-no-fail-fast') }}

@ZhongRuoyu
Copy link
Member Author

I think it's because CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. was not added when github.event was captured.

@fxcoudert fxcoudert removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Apr 21, 2023
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added the stale No recent activity label Apr 24, 2023
@ZhongRuoyu ZhongRuoyu added in progress Stale bot should stay away CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. and removed stale No recent activity labels Apr 24, 2023
@ZhongRuoyu ZhongRuoyu mentioned this pull request Apr 24, 2023
@p-linnane p-linnane removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Apr 25, 2023
@github-actions github-actions bot added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Apr 25, 2023
@p-linnane p-linnane removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Apr 25, 2023
@github-actions github-actions bot added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Apr 25, 2023
carlocab added a commit to carlocab/homebrew-core that referenced this pull request Apr 25, 2023
…ts`"

This isn't working as expected. See, for example, Homebrew#128865.

This reverts commit 0f9481c.
carlocab added a commit to carlocab/homebrew-core that referenced this pull request Apr 25, 2023
The change from Homebrew#128809 doesn't work as expected, because `github.event`
doesn't seem to pick up changes in labels at some point after the PR is
opened. See Homebrew#128865 for an example.

In order to be able to change the labels and have them apply only to
dependent tests, let's check the labels again in a separate job for
dependent testing. In order to minimise code repitition, we extract the
bulk of the `check-labels` job into a separate script.
@Tenzer Tenzer mentioned this pull request Apr 25, 2023
@ZhongRuoyu
Copy link
Member Author

ZhongRuoyu commented Apr 25, 2023

Failures

Python-related failures

  • brew install --build-from-source buku (SetuptoolsDeprecationWarning: Invalid version: '0.2.2rc1.devNone'.; logs): buku 4.8 #129347
  • brew test --retry --verbose charm-tools (The 'pip<23,>=1.5.4' distribution was not found and is required by charm-tools; logs)
  • brew install --build-from-source duplicity (error in dropbox setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after version specifier); logs)
  • brew install --build-from-source dvc (error in kombu setup command: 'tests_require' must be a string or list of strings containing valid project/version requirement specifiers; Expected end or semicolon (after name and no valid version specifier); logs): superseded by dvc 2.55.0 #128229
  • (13-x86_64) brew install --build-from-source lexicon (Could not find a version that satisfies the requirement packaging>=20.0 (from versions: none); logs)
  • brew test --retry --verbose snapcraft (pkg_resources.ContextualVersionConflict: (setuptools 67.6.1 (/opt/homebrew/Cellar/snapcraft/7.3.1/libexec/lib/python3.11/site-packages), Requirement.parse('setuptools<66'), {'snapcraft'}); logs)
  • brew install --build-from-source trezor-agent (error in ledgerblue setup command: 'extras_require' must be a dictionary whose values are strings or lists of strings containing valid project/version requirement specifiers.; logs): trezor-agent: update resources #129345

Other platform-specific failures

Transient

  • (11-x86_64) brew test --retry --verbose boringtun (Timeout::Error: execution expired; logs): boringtun: test #125501
  • (11-arm64) brew install --build-from-source chatblade (failed to get successful HTTP response from `https://crates.io/api/v1/crates/pyo3-macros-backend/0.17.3/download`, got 503; logs)
  • (12-arm64, 11-arm64) brew test --retry --verbose drill (thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ValueOutOfRangeResizeDisabled'; logs): existing upstream issue: Benchmark panics if endpoint which assigns a value times out fcsonline/drill#72
  • (12-arm64) brew fetch --build-from-source --retry page (curl: (35) Recv failure: Connection reset by peer; logs)
  • (13-x86_64) brew test --retry --verbose sslyze (full output): looks like a network issue

This was referenced Apr 26, 2023
@ZhongRuoyu ZhongRuoyu added ready to merge PR can be merged once CI is green and removed help wanted Task(s) needing PRs from the community or maintainers labels May 1, 2023
@ZhongRuoyu
Copy link
Member Author

ZhongRuoyu commented May 1, 2023

We should be good to ship this now. All Rust-related issues are identified (see checklist) and are either fixed or to be fixed after the merge.

Many issues don't seem to be related to the Rust upgrade (many of them seem to be caused by recent Python package upgrades instead), so they shouldn't block us from shipping the Rust release.


A note on the grin failure (warning: variable 'run_count' set but not used "enum_(unnamed_at_CRoaring/roaring_h_7_1)" is not a valid Ident) which I didn't comment on in the checklist: it is due to an unused variable and the -Wall flag. A simple fix would be ENV.append_to_cflags "-Wno-unused-but-set-variable". Update: it's actually due to an issue with bindgen (logs) and can be fixed by switching to llvm@15 temporarily. But this error comes from the croaring-sys crate, and aside from croaring, it seems to be vendoring lmdb as well (see https://github.com/mimblewimble/grin/blob/v5.1.2/Cargo.lock). It's not a blocker so I will have a closer look at it later this week.

@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2023

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label May 1, 2023
@BrewTestBot BrewTestBot enabled auto-merge May 1, 2023 18:11
@BrewTestBot BrewTestBot added this pull request to the merge queue May 1, 2023
Merged via the queue into Homebrew:master with commit 14c3c62 May 1, 2023
@ZhongRuoyu ZhongRuoyu deleted the bump-rust-1.69.0 branch May 1, 2023 18:25
@ZhongRuoyu ZhongRuoyu mentioned this pull request May 9, 2023
6 tasks
ZhongRuoyu added a commit to ZhongRuoyu/homebrew-core that referenced this pull request May 9, 2023
Apply a patch for the error "`log_error` isn't a valid `#[macro_export]`
argument" seen in Homebrew#128865. Logs:
  https://github.com/Homebrew/homebrew-core/actions/runs/4784368511/jobs/8509780504?pr=128865#step:7:10764

In addition, use brewed dependencies instead of vendored ones, when
possible.
ZhongRuoyu added a commit to ZhongRuoyu/homebrew-core that referenced this pull request May 9, 2023
A follow-up to Homebrew#128865. The error '"enum_(unnamed_at_CRoaring/
roaring_h_7_1)" is not a valid Ident' was from `croaring`, caused by
rust-lang/rust-bindgen#2312. We could fix this by switching to
`llvm@15`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-build-dependents-from-source Pass --build-dependents-from-source to brew test-bot. CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. in progress Stale bot should stay away long build Set a long timeout for formula testing ready to merge PR can be merged once CI is green
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants