-
Notifications
You must be signed in to change notification settings - Fork 706
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
switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 #15636
switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 #15636
Conversation
Test report by @pberthier |
@pberthier Do you have some more details here, the exact error that you're running into with Rust 1.52.1? Because it definitely used to work, see #14391 |
Does the build logfile helps? |
First error produced by Rust compiler:
That looks very similar to the problem that led me to use a more recent Rust version for This is probably a result of pulling in Rust packages during the installation of Rust, it looks like we may be pulling in package versions that are not actually compatible with Rust 1.52.1 (while we weren't before). @zao Any thoughts on this, since you're a bit more familiar with the Rust ecosystem? |
Actually, it's probably because whatever Rust packages are required for So we're now pulling in So bumping the Rust version we're using as a build dep to something more recent is a short term solution, but we're bound to run into the same problem again later, and we need to properly fix this by making sure we install the same dependency versions... |
@zao Any thoughts on this (since you're more familiar with the Rust ecosystem)? |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1163484033 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Going in, thanks @pberthier! |
I've opened a separate issue on the broader problem here for follow-up, so we can get this PR merged as a temporary workaround for this specific problem; see #15763 |
(created using
eb --new-pr
)