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

[Backport release-24.11] aerogramme: unbroken #370395

Closed

Conversation

nix-backports[bot]
Copy link

@nix-backports nix-backports bot commented Jan 2, 2025

Bot-based backport to release-24.11, triggered by a label in #368128.

  • Before merging, ensure that this backport is acceptable for the release.
    • Even as a non-commiter, if you find that it is not acceptable, leave a comment.

(cherry picked from commit 40effac)
@nix-backports nix-backports bot mentioned this pull request Jan 2, 2025
13 tasks
@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 labels Jan 2, 2025
@lucasew
Copy link
Contributor

lucasew commented Jan 3, 2025

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 370395


x86_64-linux

❌ 1 package failed to build:
  • aerogramme

@lucasew
Copy link
Contributor

lucasew commented Jan 3, 2025

error: builder for '/nix/store/xs7zisyk03pyfmqb366n7liyz7jn315i-aerogramme-0.3.0.drv' failed with exit code 101;
       last 25 log lines:
       >    --> aero-proto/src/dav/resource.rs:933:36
       >     |
       > 933 |     fn supported_properties(&self, user: &ArcUser) -> dav::PropName<All> {
       >     |                                    ^^^^ help: if this is intentional, prefix it with an underscore: `_user`
       >
       > warning: unused variable: `prop`
       >    --> aero-proto/src/dav/resource.rs:937:43
       >     |
       > 937 |     fn properties(&self, _user: &ArcUser, prop: dav::PropName<All>) -> PropertyStream<'static> {
       >     |                                           ^^^^ help: if this is intentional, prefix it with an underscore: `_prop`
       >
       > error[E0391]: cycle detected when optimizing MIR for `sasl::<impl at aero-proto/src/sasl.rs:77:1: 77:13>::run::{closure#0}::{closure#2}::{closure#0}::{closure#0}`
       >    --> aero-proto/src/sasl.rs:112:68
       >     |
       > 112 |                     let login = async |user: String, pass: String| self.login.login(user.as_str(), pass.as_str()).await.is_ok();
       >     |                                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
       >     |
       >     = note: ...which immediately requires optimizing MIR for `sasl::<impl at aero-proto/src/sasl.rs:77:1: 77:13>::run::{closure#0}::{closure#2}::{closure#0}::{closure#0}` again
       >     = note: cycle used when computing layout of `{async closure body@aero-proto/src/sasl.rs:112:68: 112:128}`
       >     = note: see https://rustc-dev-guide.rust-lang.org/overview.html#queries and https://rustc-dev-guide.rust-lang.org/query.html for more information
       >
       > For more information about this error, try `rustc --explain E0391`.
       > warning: `aero-proto` (lib) generated 19 warnings
       > error: could not compile `aero-proto` (lib) due to 1 previous error; 19 warnings emitted
       > warning: build failed, waiting for other jobs to finish...
       For full logs, run 'nix log /nix/store/xs7zisyk03pyfmqb366n7liyz7jn315i-aerogramme-0.3.0.drv'.
error: 1 dependencies of derivation '/nix/store/ldj57acj61g6pv5r8vbvgw8f89dn76z3-review-shell.drv' failed to build

@wolfgangwalther wolfgangwalther marked this pull request as draft January 4, 2025 15:41
@Bot-wxt1221
Copy link
Member

This problem is fixed in the newest rustc that why we mark this as broken.

@Bot-wxt1221 Bot-wxt1221 closed this Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants