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

switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 #15636

Merged

Conversation

pberthier
Copy link
Contributor

(created using eb --new-pr)

@boegel boegel added the bug fix label Jun 8, 2022
@boegel boegel added this to the next release (4.5.6?) milestone Jun 8, 2022
@pberthier
Copy link
Contributor Author

Test report by @pberthier
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
softdev.binf.unibe.ch - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6240 CPU @ 2.60GHz, Python 3.6.8
See https://gist.github.com/db975c82458378acd0b5389e17a68d46 for a full test report.

@boegel boegel changed the title build fails with Rust 1.52.1 (E0658), works with newer Rust switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 (E0658) Jun 8, 2022
@boegel
Copy link
Member

boegel commented Jun 8, 2022

@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

@pberthier
Copy link
Contributor Author

Does the build logfile helps?
easybuild-bamtofastq-1.4.0-20220609.121200.kBTnh.log

@boegel
Copy link
Member

boegel commented Jun 9, 2022

First error produced by Rust compiler:

error[E0658]: arbitrary expressions in key-value attributes are unstable
   --> /home/softdev/.cargo/registry/src/github.com-1ecc6299db9ec823/libz-sys-1.1.8/src/lib.rs:115:19
    |
115 |     #[link_name = zng_prefix!(adler32)]
    |                   ^^^^^^^^^^^^^^^^^^^^
    |
    = note: see issue #78835 <https://github.com/rust-lang/rust/issues/78835> for more information
    = help: add `#![feature(extended_key_value_attributes)]` to the crate attributes to enable

That looks very similar to the problem that led me to use a more recent Rust version for tokenizers in #15587...

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?

@boegel
Copy link
Member

boegel commented Jun 9, 2022

Actually, it's probably because whatever Rust packages are required for bamtofastq are being downloaded on the fly during the installation of bamtofastq itself, without proper version locking...

So we're now pulling in libz-sys-1.1.8, which requires a Rust version newer than 1.52.1, while before we were pulling down an older version of libz-sys that worked fine with Rust 1.52.1.

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...

@boegel
Copy link
Member

boegel commented Jun 10, 2022

@zao Any thoughts on this (since you're more familiar with the Rust ecosystem)?

@verdurin
Copy link
Member

Ping @zao @boegel

@boegel
Copy link
Member

boegel commented Jun 22, 2022

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=15636 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_15636 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8760

Test results coming soon (I hope)...

- notification for comment with ID 1163484033 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/bfb6fc58e86d146906627711f4dc4191 for a full test report.

@boegel
Copy link
Member

boegel commented Jun 24, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3138.skitty.os - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/6c2a90646eb12ee8b27e6e8491d960bc for a full test report.

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member

boegel commented Jun 24, 2022

Going in, thanks @pberthier!

@boegel boegel merged commit c0e72ac into easybuilders:develop Jun 24, 2022
@boegel
Copy link
Member

boegel commented Jun 24, 2022

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

@pberthier pberthier deleted the 20220608102130_new_pr_bamtofastq140 branch June 30, 2022 07:50
@boegel boegel changed the title switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 (E0658) switch to Rust 1.60.0 build dependency for bamtofastq, since build of fails with Rust 1.52.1 Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants