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

Make sure FairLogger has a proper version even when doing development #3945

Merged
merged 1 commit into from
Mar 16, 2022

Conversation

ktf
Copy link
Member

@ktf ktf commented Mar 15, 2022

No description provided.

@ktf ktf requested a review from a team as a code owner March 15, 2022 09:54
@ktf
Copy link
Member Author

ktf commented Mar 15, 2022

@dennisklein @rbx under certain circumstances (e.g. if you download the tarball, rather than doing a git checkout) the GIT_VERSION of FairLogger seems to be set to 0.0.0, which then prevents FairMQ to work because it requires 1.8.0. This solves the problem, basically forcing the version from the outside, is there a better way of doing it?

@ktf
Copy link
Member Author

ktf commented Mar 15, 2022

Maybe the tag should be embedded in the source as part of the release process?

@dennisklein
Copy link
Contributor

dennisklein commented Mar 15, 2022

Maybe the tag should be embedded in the source as part of the release process?

Ya, this is an old question. IMHO, the git repo is a first class quality of our source tree and tools that still model source tree as just a single version tarball need to adapt and learn how to include the git repo. I would hate it a lot to duplicate version numbers in two places. (Also automatic commits are a no go.)

However, what we can do, is create a dist build target, that generates such a source tarball for backwards compatibility (and then generate and attach those as release assets). - Would this work?


Just for my curiosity, in which environment/use case are you dropping the git history and why? (edit: aside from the example you already gave, in aliBuild I hope you have not changed the semantics to treat the git repo as first class entity, which would make me really sad, if you did ;) - imho it is one of the coolest features of aliBuild)

@dennisklein
Copy link
Contributor

btw, the solution you propose in this PR I like even better than a dist tarball. In fact I like it so much, that I will make it a documented feature for fair_get_git_version which FairLogger does not yet use, but it really should and will in one of the next releases.

@ktf
Copy link
Member Author

ktf commented Mar 15, 2022

I just had a spurious tag in fairlogger, which confused the current command. The "tarball" behaviour was just to make a more general example. If you are saying this is the way to go, I am also happy.

@ktf
Copy link
Member Author

ktf commented Mar 15, 2022

(also notice github provides you tarballs automatically)

@dennisklein
Copy link
Contributor

dennisklein commented Mar 15, 2022

(also notice github provides you tarballs automatically)

unfortunately i find, i think this is a terrible feature in its current form. they are not even deletable :(

@ktf ktf merged commit b3ae1aa into alisw:master Mar 16, 2022
@ktf ktf deleted the fix-fairlogger branch March 16, 2022 19:45
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