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

Use spack to install gpg 2.4 for amazon-linux-2 #18

Merged
merged 3 commits into from Mar 9, 2023

Conversation

zackgalbreath
Copy link
Collaborator

No description provided.

@zackgalbreath
Copy link
Collaborator Author

FYI @alalazo @eugeneswalker

gpg.yaml Outdated Show resolved Hide resolved
gpg.yaml Outdated Show resolved Hide resolved
gpg.yaml Outdated Show resolved Hide resolved
@zackgalbreath
Copy link
Collaborator Author

Thanks for the feedback @haampie . I'm testing out your proposed changes locally. If all goes well I'll push an update shortly.

gpg.yaml Outdated Show resolved Hide resolved
zackgalbreath and others added 2 commits March 9, 2023 10:32
Co-authored-by: eugeneswalker <38933153+eugeneswalker@users.noreply.github.com>
Co-authored-by: Harmen Stoppels <harmenstoppels@gmail.com>
gpg.yaml Outdated Show resolved Hide resolved
@haampie
Copy link
Member

haampie commented Mar 9, 2023

Bit confused about SPACK_ROOT... should really not be necessary

@zackgalbreath
Copy link
Collaborator Author

Bit confused about SPACK_ROOT... should really not be necessary

me too 😅 but I was following what CI told me to do here:

https://github.com/spack/gitlab-runners/actions/runs/4369563677/jobs/7643487182#step:7:3084

@haampie haampie merged commit d476c20 into spack:main Mar 9, 2023
@haampie
Copy link
Member

haampie commented Mar 9, 2023

Apparently CI is not a requirement to merge things here, lol. In any case, if it builds on the main branch that saves some CPU cycles

@haampie
Copy link
Member

haampie commented Mar 9, 2023

You don't have to set up shell support to use spack, so that simplifies things. The problem is that /proc/1/exe symlinks to the host system's qemu, not the docker image's shell.

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.

None yet

2 participants