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

uutils-coreutils: disable buildpaths error #865

Conversation

Ecordonnier
Copy link
Contributor

No description provided.

@shr-project
Copy link
Contributor

Don't just disable it, if it cannot be fixed, then lower it from ERROR_QA to WARN_QA (so that it's still shown until someone fixes it properly).

@Ecordonnier Ecordonnier force-pushed the eco/fix-uutils-coreutils-build branch from da52a32 to af60e2b Compare August 29, 2024 19:35
@Ecordonnier
Copy link
Contributor Author

@shr-project I force-pushed the PR with a new version implementing your feedback.

@kraj
Copy link
Contributor

kraj commented Aug 29, 2024

I would encourage you to root cause it, I know its rust and perhaps somewhere rust compiler not respecting debug-prefix mapping will be at the heart of the problem. I am trying to clean meta-openembedded for reproducibility and buildpaths is important to get rid of for that.

@shr-project
Copy link
Contributor

I didn't check uutils itself, but these:
https://github.com/rust-lang/rust/labels/A-reproducibility
rust-lang/rust#111540
might be useful here as well.

@Ecordonnier
Copy link
Contributor Author

I would also like to root-cause when I have some time. I thought in the meanwhile this would be useful to fix the immediate build failure. If you prefer to wait for a proper fix, it's fine to not merge this PR, however at the moment I can't commit to a timeline for a proper fix.

@Ecordonnier
Copy link
Contributor Author

Merged as deca934

@Ecordonnier Ecordonnier closed this Sep 7, 2024
@Ecordonnier Ecordonnier deleted the eco/fix-uutils-coreutils-build branch September 7, 2024 20:58
@Ecordonnier
Copy link
Contributor Author

Upstream issue: uutils/coreutils#6833

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.

3 participants