Skip to content

Bump versioning in repo#84

Open
DavidStirling wants to merge 4 commits intoglencoesoftware:mainfrom
DavidStirling:bumper
Open

Bump versioning in repo#84
DavidStirling wants to merge 4 commits intoglencoesoftware:mainfrom
DavidStirling:bumper

Conversation

@DavidStirling
Copy link
Contributor

Per internal discussion, bumps the build version ready for future releases and replaces the old changelog with a link to the releases log in the main readme.

}

version = '2.1.0'
version = '2.1.1'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there some reference discussion that explains why SNAPSHOT builds are not an option for this project? And outside the scope of this PR, is that something that could be relaxed in the future and would you have an idea of the workload?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if snapshots are actually a problem, I think we just haven't tried them. I would have anticipated that it might cause problems with the update checker, but from a quick review it looks like snapshot tags are already handled. We could have a go with 2.1.1-SNAPSHOT?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understood. Maybe only the signing workflow works with release candidate. If using SNAPSHOT is possible, that would be my preference as it's consistent with the other Java projects

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a quick test it looks like when using a snapshot version the initial build works fine but jpackage then struggles to bundle it. Probably needs investigating as it's own issue.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The error is something like:

> Task :jpackageImage FAILED
java.lang.IllegalArgumentException: "Version [2.1.0-SNAPSHOT] contains invalid component [0-SNAPSHOT]"

Looking at how QuPath handles this problem:

https://github.com/qupath/qupath/blob/main/buildSrc/src/main/kotlin/qupath.jpackage-conventions.gradle.kts#L96

That looks like replacing the -SNAPSHOT just when running jpackage is a way forward.

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.

3 participants

Comments