Skip to content

Conversation

@Flamefire
Copy link
Contributor

(created using eb --new-pr)

@Flamefire
Copy link
Contributor Author

Test report by @Flamefire
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in total)
taurusi6605.taurus.hrsk.tu-dresden.de - Linux RHEL 7.9, x86_64, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz (haswell), Python 2.7.5
See https://gist.github.com/b5fb1d5cfb53be52ab4d0aeeed3ec002 for a full test report.

Copy link
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket
Copy link
Contributor

Micket commented Sep 3, 2021

Test report by @Micket
SUCCESS
Build succeeded for 4 out of 4 (4 easyconfigs in total)
vera-c1 - Linux centos linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/1ca27c5245ac7fe2c31d4b3234655e56 for a full test report.

Copy link
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

(apparently i forgot to press approve)

@Micket
Copy link
Contributor

Micket commented Sep 3, 2021

Going in, thanks @Flamefire!

@Micket Micket merged commit 41da77f into easybuilders:develop Sep 3, 2021
@boegel boegel changed the title CLHEP: Remove superflous configopts and add sanity-checks/checksums CLHEP: Remove superflous configopts and add sanity checks/checksums in CLHEP easyconfigs Sep 3, 2021
@boegel boegel changed the title CLHEP: Remove superflous configopts and add sanity checks/checksums in CLHEP easyconfigs remove superflous configopts and add sanity checks/checksums in CLHEP easyconfigs Sep 3, 2021
@boegel boegel modified the milestones: 4.x, 4.4.2 Sep 3, 2021
@Flamefire Flamefire deleted the 20210805095203_new_pr_CLHEP2110 branch September 14, 2021 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants