Skip to content

force use of bash for Anaconda install script#2692

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Micket:anaconda_bash
Mar 2, 2022
Merged

force use of bash for Anaconda install script#2692
boegel merged 1 commit intoeasybuilders:developfrom
Micket:anaconda_bash

Conversation

@Micket
Copy link
Copy Markdown
Contributor

@Micket Micket commented Mar 2, 2022 •
edited by boegel
Loading

This was broken on Ubuntu and other OS's that doesn't alias /bin/sh to /bin/bash

edit (by @boegel): fixes this issue (reproduced on Ubuntu 20.04):

done
installation finished.
./Anaconda3-2021.11-Linux-x86_64.sh: 516: Syntax error: "(" unexpected (expecting ")")

@Micket Micket added the bug fix label Mar 2, 2022
@Micket Micket added this to the next release (4.5.4?) milestone Mar 2, 2022
@boegel boegel changed the title Force bash for Anaconda install script force use of bash for Anaconda install script Mar 2, 2022
@Micket
Copy link
Copy Markdown
Contributor Author

Micket commented Mar 2, 2022

Test report by @Micket

Overview of tested easyconfigs (in order)

  • SUCCESS Anaconda3-2021.11.eb
  • SUCCESS Anaconda3-2018.12.eb
  • SUCCESS Anaconda3-5.1.0.eb
  • SUCCESS Anaconda2-2019.10.eb
  • SUCCESS Miniconda2-4.7.10.eb
  • SUCCESS Miniconda3-4.9.2.eb

Build succeeded for 6 out of 6 (6 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/43a79b8fcc463e0f7228811c07707211 for a full test report.

boegel
boegel approved these changes Mar 2, 2022
View reviewed changes
Copy link
Copy Markdown
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
Copy Markdown
Member

boegel commented Mar 2, 2022

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Anaconda3-2021.11.eb
  • SUCCESS Anaconda3-2020.07.eb
  • SUCCESS Anaconda3-2018.12.eb
  • SUCCESS Anaconda3-4.0.0.eb
  • SUCCESS Anaconda3-5.3.0.eb
  • SUCCESS Anaconda2-2019.10.eb
  • SUCCESS Anaconda2-2018.12.eb
  • SUCCESS Anaconda2-5.3.0.eb
  • SUCCESS Anaconda2-4.0.0.eb

Build succeeded for 9 out of 9 (9 easyconfigs in total)
node3515.doduo.os - Linux RHEL 8.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/754a98d1244d2b7007b11aee86d5eacb for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 2, 2022

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Anaconda3-2020.07.eb
  • SUCCESS Anaconda3-5.3.0.eb
  • SUCCESS Anaconda3-2021.11.eb
  • SUCCESS Anaconda3-2018.12.eb
  • SUCCESS Anaconda2-5.3.0.eb
  • SUCCESS Anaconda2-2019.10.eb
  • SUCCESS Anaconda2-2018.12.eb

Build succeeded for 7 out of 7 (7 easyconfigs in total)
node2692.swalot.os - Linux Ubuntu 20.04.3 LTS (Focal Fossa), x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.8.10
See https://gist.github.com/5bdc486f157dc303c4c6164829366407 for a full test report.

@boegel boegel merged commit dad6683 into easybuilders:develop Mar 2, 2022
@Micket Micket deleted the anaconda_bash branch April 3, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.4

Development

Successfully merging this pull request may close these issues.

2 participants

@Micket @boegel