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

Updated config to be compatible with Gromacs 5.1.1 #53

Merged
merged 1 commit into from
Apr 20, 2016

Conversation

dotsdl
Copy link
Collaborator

@dotsdl dotsdl commented Apr 20, 2016

Since Gromacs 5.1.x removed the use of g_* methods, instead opting only
for the new gmx <command> style, this config features the necessary
bits to use the commands this way. This configuration will not work
for Gromacs 4.x or earlier. We will hopefully replace this config style
with one that works for multiple versions more flexibly with #49.

Removed extras, as these tools are no longer maintained and it's not
clear that they will work as expected under this config.

Since Gromacs 5.1.x removed the use of g_* methods, instead opting only
for the new ``gmx <command>`` style, this config features the necessary
bits to use the commands this way. This configuration will *not* work
for Gromacs 4.x or earlier. We will hopefully replace this config style
with one that works for multiple versions more flexibly with #49.

Removed ``extras``, as these tools are no longer maintained and it's not
clear that they will work as expected under this config.
@dotsdl dotsdl added this to the 0.5 milestone Apr 20, 2016
@dotsdl
Copy link
Collaborator Author

dotsdl commented Apr 20, 2016

With this merge, we should got proper doc builds (#32), and it will go towards getting CI up and running under travis (#39).

@orbeckst orbeckst merged commit a310058 into develop Apr 20, 2016
@orbeckst orbeckst deleted the gmx-5.1.1-config branch April 20, 2016 18:29
@orbeckst
Copy link
Member

fastest merge ever ;-)

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.

2 participants