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 #46

Merged
merged 3 commits into from
Oct 3, 2023
Merged

Version #46

merged 3 commits into from
Oct 3, 2023

Conversation

OndrejSladky
Copy link
Owner

Version is now automatically updated after every single commit.

@OndrejSladky OndrejSladky linked an issue Oct 3, 2023 that may be closed by this pull request
@karel-brinda
Copy link
Collaborator

Looks good, just the version of the hash is just a bit too long. I think 6 characters should be enough (already for this the prob of a collision should be very low).

btw. in the current implementation, which version will appear in "true" releases (I just remember this is what I was always struggling with a couple of ys ago)?

@OndrejSladky
Copy link
Owner Author

btw. in the current implementation, which version will appear in "true" releases (I just remember this is what I was always struggling with a couple of ys ago)?

I'm not entirely sure what is the question.
But if it is which versions will have separate tag on gh, then I was thinking I'd make it every 0.x version.
Or do you have any other sugestions?

@OndrejSladky OndrejSladky merged commit c4ebb0d into main Oct 3, 2023
3 checks passed
@OndrejSladky OndrejSladky deleted the version branch October 3, 2023 19:35
@karel-brinda
Copy link
Collaborator

karel-brinda commented Oct 3, 2023 via email

@OndrejSladky
Copy link
Owner Author

The way it is now, it'll display 0.3-348a841.
If you think this is not good enough, i'll file another issue and will think about how to elegantly fix this.

@karel-brinda
Copy link
Collaborator

In case it's a new release, it should be just 0.3. I'm not sure how it's doable, but it somehow should be as a I often see it in software packages.

@karel-brinda
Copy link
Collaborator

karel-brinda commented Oct 4, 2023

btw. see also:

  • git describe --tags --always
  • git describe --abbrev=4 --dirty --always --tags

@karel-brinda
Copy link
Collaborator

Actually $ git describe --abbrev=4 --dirty --always --tags v0.1.0 seems to be working really well!!!!! When it's a release, it prints just eg v0.1.0.

So it just needs to be included somehow in the Makefile – as the first step of compilation, this string should be generated.

One difficulty I see is that it make should be able to compile it even out of git without a failure.

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.

Versioning
2 participants