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.

Sign up for GitHub

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

Jump to bottom

[GCCcore/10.3.0] groff requires makeinfo as a build dep, which requires Perl, which requires groff; break cyclic dependency by adding Perl-minimal and makeinfo-minimal #13165

Merged
merged 3 commits into from
Jun 23, 2021
Merged

[GCCcore/10.3.0] groff requires makeinfo as a build dep, which requires Perl, which requires groff; break cyclic dependency by adding Perl-minimal and makeinfo-minimal #13165

merged 3 commits into from
Jun 23, 2021

Conversation

akesandgren
Copy link
Contributor

@akesandgren akesandgren commented Jun 15, 2021 •
edited
Loading

(created using eb --new-pr)

Depends on easybuilders/easybuild-easyblocks#2474

@akesandgren
[GCCcore/10.3.0] groff requires makeinfo as a build dep, which requir…
3578323 
…es Perl, which requires groff. Break cyclic redundancy by adding Perl-minimal and makeinfo-minimal.
@akesandgren akesandgren added this to the next release (4.4.1) milestone Jun 15, 2021
@akesandgren akesandgren mentioned this pull request Jun 15, 2021
Merged
@akesandgren
Copy link
Contributor Author

Test report by @akesandgren
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2474
SUCCESS
Build succeeded for 9 out of 9 (9 easyconfigs in total)
b-an02.hpc2n.umu.se - Linux Ubuntu 20.04, x86_64, Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz, Python 3.8.5
See https://gist.github.com/063577bdb5ef215754b332729fe3a142 for a full test report.

@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2474
SUCCESS
Build succeeded for 9 out of 9 (9 easyconfigs in total)
centos8-eb - Linux centos linux 8.3.2011, x86_64, Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (skylake) (skylake) (skylake), Python 3.6.8
See https://gist.github.com/6ac5fe3d4e976cca63b2de801f61a4ea for a full test report.

@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2474
SUCCESS
Build succeeded for 9 out of 9 (9 easyconfigs in total)
rocky8-eb - Linux rocky linux 8.4, x86_64, Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz (broadwell) (broadwell) (broadwell), Python 3.6.8
See https://gist.github.com/534213636f35cbdf639c2f841af6ce1f for a full test report.

SebastianAchilles
SebastianAchilles previously approved these changes Jun 22, 2021
View reviewed changes
Copy link
Member

@SebastianAchilles SebastianAchilles left a comment

Choose a reason for hiding this comment

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

lgtm

@SebastianAchilles SebastianAchilles mentioned this pull request Jun 23, 2021
Merged
@SebastianAchilles SebastianAchilles dismissed their stale review June 23, 2021 10:11

test_dep_versions_per_toolchain_generation was not executed for this PR

@SebastianAchilles
Copy link
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@SebastianAchilles: Request for testing this PR well received on generoso

PR test command 'EB_PR=13165 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_13165 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 866791162 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).

SebastianAchilles
SebastianAchilles approved these changes Jun 23, 2021
View reviewed changes
Copy link
Member

@SebastianAchilles SebastianAchilles left a comment

Choose a reason for hiding this comment

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

lgtm

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
generoso-x-2 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/9377d58fb5e1d53d22005bba9d3a79b0 for a full test report.

@SebastianAchilles
Copy link
Member

Going in, thanks @akesandgren!

@SebastianAchilles SebastianAchilles merged commit 086039f into easybuilders:develop Jun 23, 2021
@akesandgren akesandgren deleted the 20210615144647_new_pr_Perl5321 branch June 23, 2021 13:25
@boegel boegel changed the title [GCCcore/10.3.0] groff requires makeinfo as a build dep, which requires Perl, which requires groff. Break cyclic redundancy by adding Perl-minimal and makeinfo-minimal. [GCCcore/10.3.0] groff requires makeinfo as a build dep, which requires Perl, which requires groff; break cyclic dependency by adding Perl-minimal and makeinfo-minimal Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SebastianAchilles SebastianAchilles SebastianAchilles approved these changes

Assignees
No one assigned
Labels
bug fix update
Projects
None yet
Milestone
4.4.1
Development

Successfully merging this pull request may close these issues.
4 participants
@akesandgren @SebastianAchilles @boegelbot @migueldiascosta