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

remove help2man build dependency for flex using system toolchain (because it requires Perl) #13922

Merged

Conversation

akesandgren
Copy link
Contributor

(created using eb --new-pr)

@akesandgren
Copy link
Contributor Author

Test report by @akesandgren
SUCCESS
Build succeeded for 2 out of 2 (2 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/44b678d4fec56c2466601a9050f1a2e4 for a full test report.

@Micket
Copy link
Contributor

Micket commented Sep 7, 2021

Old checksum needs updating to sha256

@akesandgren
Copy link
Contributor Author

@boegelbot Please test @ generoso

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. In fact, I think we should disable manpages almost everywhere when we can, since we are typically exceeding the max length for MANPATH. I think we should just refer users to the many online documentations for all these softwares instead /rant)

@Micket
Copy link
Contributor

Micket commented Sep 7, 2021

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

@boegelbot
Copy link
Collaborator

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

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

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

Test results coming soon (I hope)...

- notification for comment with ID 914290360 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 2 out of 2 (2 easyconfigs in total)
cns2 - Linux rocky linux 8.4, x86_64, Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/851c7c79c6f67d31751afc07844b355f for a full test report.

@Micket
Copy link
Contributor

Micket commented Sep 7, 2021

Going in, thanks @akesandgren!

@Micket Micket merged commit ddd7e65 into easybuilders:develop Sep 7, 2021
@akesandgren akesandgren deleted the 20210907142134_new_pr_flex264 branch September 7, 2021 13:49
@easybuilders easybuilders deleted a comment from boegelbot Sep 7, 2021
@boegel boegel modified the milestones: 4.x, next release (4.4.3?) Sep 7, 2021
@boegel boegel changed the title Drop help2man from flex at SYSTEM level. help2man depends on Perl and we do not want Perl at SYSTEM level. remove help2man build dependency for flex using system toolchain (because it requires Perl) Sep 7, 2021
@boegel boegel added the change label Sep 15, 2021
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.

None yet

4 participants