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

Version is git hash for release? #1180

Closed
vsoch opened this issue Dec 8, 2022 · 6 comments
Closed

Version is git hash for release? #1180

vsoch opened this issue Dec 8, 2022 · 6 comments

Comments

@vsoch
Copy link
Contributor

vsoch commented Dec 8, 2022

Hiya! Am I building incorrectly if the release version is outputting what looks like a partial git hash?

make: Leaving directory '/home/runner/work/singularity-cli/singularity-cli/singularity-ce-3.10.4/builddir'
Run singularity version
  singularity version
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2d1c176

That's from a github action - so the output of singularity version is 2d1c176. I'll test other versions and see if they have similar behavior. Thanks!

@dtrudg
Copy link
Member

dtrudg commented Dec 8, 2022

I'm not sure what you are getting there...

For the 3.8.4 release tarball I get....

$ singularity version
3.8.4

For an unreleased git checkout, it should be a version plus a suffix, e.g.

$ singularity version
3.10.0+346-g8fbd83543

@dtrudg
Copy link
Member

dtrudg commented Dec 8, 2022

Doh... somehow I read 3.8.4 somewhere instead of 3.10.4. However, 3.10.4 installed from the release tarball here still works as I'd expect:

$ singularity version
3.10.4

@vsoch
Copy link
Contributor Author

vsoch commented Dec 8, 2022

I must be doing something silly then! Could you be an extra pair of eyes? Here are the steps: https://github.com/singularityhub/install-singularity/blob/main/action.yaml

And the output: https://github.com/singularityhub/install-singularity/actions/runs/3645364755/jobs/6155447743

image

@dtrudg
Copy link
Member

dtrudg commented Dec 8, 2022

Oh ... are you building the source inside another git repo? The scripts/get-version looks for a git repo to pull version info from. If you use the release tarball then there is no singularity-ce repo... but if you extract it inside another git repo then it'll find the git info from the repo further up.

I guess this is what is happening. You could wget and build singularity under a temp dir instead?

@vsoch
Copy link
Contributor Author

vsoch commented Dec 8, 2022

oh that's it! And that's exactly what I'll do - doh! Thanks @dtrudg !

@vsoch vsoch closed this as completed Dec 8, 2022
@vsoch
Copy link
Contributor Author

vsoch commented Dec 8, 2022

There we gooooo!! haha I feel... silly.
image

All is well <3 thank you again!

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

No branches or pull requests

2 participants