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

Rename arolla to tsa #236

Merged
merged 4 commits into from
Jan 23, 2020
Merged

Rename arolla to tsa #236

merged 4 commits into from
Jan 23, 2020

Conversation

yboetz
Copy link
Contributor

@yboetz yboetz commented Jan 23, 2020

We changed all paths and option files from arolla to tsa. With these changes serialbox does not install on tsa anymore, since the build script checks only for host arolla and then fails to load the correct env file.

I also disabled cray compiler on tsa, since we don't support it anymore.

@yboetz yboetz requested review from havogt and lxavier January 23, 2020 08:11
@havogt
Copy link
Collaborator

havogt commented Jan 23, 2020

launch jenkins

@havogt
Copy link
Collaborator

havogt commented Jan 23, 2020

launch jenkins

@havogt
Copy link
Collaborator

havogt commented Jan 23, 2020

@yboetz I think for all you cases you should actually not use this script to install, but use the package builder from the buildenv. Are you aware of this?

@havogt
Copy link
Collaborator

havogt commented Jan 23, 2020

launch jenkins

@havogt havogt merged commit f6faa58 into GridTools:master Jan 23, 2020
@yboetz
Copy link
Contributor Author

yboetz commented Jan 23, 2020

@yboetz I think for all you cases you should actually not use this script to install, but use the package builder from the buildenv. Are you aware of this?

I didn't know this. I only saw that the serialbox2 plan is failing, and that one is using this script. But @lxavier now told me that serialbox2_release is the plan we actually use and that one is using the package builder.

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