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

Introduce local-pgi machine #169

Merged
merged 1 commit into from
Jun 19, 2017
Merged

Introduce local-pgi machine #169

merged 1 commit into from
Jun 19, 2017

Conversation

mnlevy1981
Copy link
Collaborator

PGI offers Community Edition of PGI 17.4, which can be installed locally and
then used to build / run the stand-alone MARBL tests. The default machine is
still 'local-gnu'; it might be time to consider adding a ~/.marbl/ directory
and letting users specify what compilers are available if the machine is not
recognized (so default machine would be 'local', and if ~/.marbl/ is not found
then only gnu would be supported; otherwise users could specify other compilers
that are installed)

** Compiler available at http://www.pgroup.com/products/community.htm

Verified that this does not change behavior on cheyenne or hobart

PGI offers Community Edition of PGI 17.4, which can be installed locally and
then used to build / run the stand-alone MARBL tests. The default machine is
still 'local-gnu'; it might be time to consider adding a ~/.marbl/ directory
and letting users specify what compilers are available if the machine is not
recognized (so default machine would be 'local', and if ~/.marbl/ is not found
then only gnu would be supported; otherwise users could specify other compilers
that are installed)
Copy link
Collaborator

@klindsay28 klindsay28 left a comment

Choose a reason for hiding this comment

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

looks good

@mnlevy1981 mnlevy1981 merged commit 809cad6 into marbl-ecosys:master Jun 19, 2017
@mnlevy1981 mnlevy1981 deleted the enhancement/new_machine_for_pgi branch June 19, 2017 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants