Skip to content

Fix configure step for GROMACS version < 4.6#1742

Merged
boegel merged 1 commit intoeasybuilders:developfrom
victorusu:patch-2
Jun 5, 2019
Merged

Fix configure step for GROMACS version < 4.6#1742
boegel merged 1 commit intoeasybuilders:developfrom
victorusu:patch-2

Conversation

@victorusu
Copy link
Copy Markdown
Contributor

@victorusu victorusu commented Jun 5, 2019

When installing an older version of GROMACS that still makes use of Autotools, one gets

running command "./bootstrap &&  cmake --prefix=/opt/easybuild/osuse/software/GROMACS/4.5.7-foss-2019a    --enable-static  --with-external-blas --with-external-lapack  --without-x  --enable-threads  --without-gsl"

This fixes it by setting it back to

running command "./bootstrap &&  ./configure --prefix=/opt/easybuild/osuse/software/GROMACS/4.5.7-foss-2019a    --enable-static  --with-external-blas --with-external-lapack  --without-x  --enable-threads  --without-gsl"

When installing an older version of GROMACS that still makes use of Autotools, one gets
```
running command "./bootstrap &&  make --prefix=/opt/easybuild/osuse/software/GROMACS/4.5.7-foss-2019a    --enable-static  --with-external-blas --with-external-lapack  --without-x  --enable-threads  --without-gsl"
```
This fixes it by setting it back to 
```
running command "./bootstrap &&  ./configure --prefix=/opt/easybuild/osuse/software/GROMACS/4.5.7-foss-2019a    --enable-static  --with-external-blas --with-external-lapack  --without-x  --enable-threads  --without-gsl"
```
@boegel boegel added the bug fix label Jun 5, 2019
@boegel boegel added this to the next release (3.9.2) milestone Jun 5, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Jun 5, 2019

This clearly escaped our attention in the regression test, thanks for the fix @victorusu!

@boegel boegel changed the base branch from master to develop June 5, 2019 20:41
@boegel boegel merged commit 252a645 into easybuilders:develop Jun 5, 2019
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.

2 participants