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

Build: Version comes from git tag, ChangeLog, Jamulus.pro #2662

Open
hoffie opened this issue Jun 19, 2022 · 4 comments
Open

Build: Version comes from git tag, ChangeLog, Jamulus.pro #2662

hoffie opened this issue Jun 19, 2022 · 4 comments
Labels
bug Something isn't working tooling Changes to the automated build system

Comments

@hoffie
Copy link
Member

hoffie commented Jun 19, 2022

My concern is get_build_version() uses a different value. The tag, Jamulus.pro and ChangeLog could all have different versions.

Given we have three sources for the version, we really need automation to make sure they're aligned, at least: i.e. error out if things look wrong. But I agree, that's not in scope here. For that, it sounds like (and I don't fully understand how this works):

  • If the tag is set, we should use that to determine build type and version and that source should overrule others.
  • ChangeLog and Jamulus.pro "target" versions should match - and, if set, match the tag build type and version.
  • If the tag wasn't set, build type and version can then come from ChangeLog and Jamulus.pro, as needed.

Originally posted by @pljones in #2656 (comment)

@ann0see
Copy link
Member

ann0see commented Jun 19, 2022

Just a note, we shouldn't extract the version from ChangeLog.
We could automatically extract the version from the tag and put it into Jamulus.pro

@pljones
Copy link
Collaborator

pljones commented Jun 19, 2022

Just a note, we shouldn't extract the version from ChangeLog. We could automatically extract the version from the tag and put it into Jamulus.pro

I think the problem is that there might not be a useful tag when someone triggers the autobuild -- and it still has to work. So it has to get a version from somewhere. To me, Jamulus.pro should actually be the overruling version (regardless of tag). That must exist for a run to work. ChangeLog doesn't have to exist for qmake && make to work but Jamulus.pro must.

(So yes, I said let the tag overrule -- but I'm not fixated on that...)

@pljones pljones added this to the Release 3.10.0 milestone Oct 5, 2022
@ann0see
Copy link
Member

ann0see commented Nov 15, 2022

I think we should write a release script doing the tagging,...

@pljones
Copy link
Collaborator

pljones commented Apr 19, 2023

I don't think we know enough about what the solution will look like to target a release. Dropping the milestone and moving to Triage.

On another note:

I think we should write a release script doing the tagging,...

It could for the way we do it now by hand. But it depends what we want.

@pljones pljones removed this from the Release 3.10.0 milestone Apr 19, 2023
@pljones pljones added the tooling Changes to the automated build system label May 21, 2023
@ann0see ann0see added this to Tracking Jul 1, 2023
@github-project-automation github-project-automation bot moved this to Triage in Tracking Jul 1, 2023
@pljones pljones added the bug Something isn't working label May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooling Changes to the automated build system
Projects
Status: Triage
Development

No branches or pull requests

3 participants