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

Use git describe for Bio-Formats version (See #9639) #627

Merged
merged 7 commits into from Aug 12, 2013

Conversation

joshmoore
Copy link
Member

Current releases 2 PRs in the BF repository: 1 to bump the version numbers to the release value (4.5-DEV --> 5.0.0-beta1) and one to bump to either rollback or to bump to new DEV versions (5.0.0-beta1 --> 4.5-DEV or 5.0.0-beta2-DEV)

Tasks/features involved:

  • Add git describe commands and regexes to ant/common.xml
  • Remove hard-coded references to the version from build.properties files and source
  • Allow overriding by omero.version
  • Set version in maven (using perhaps mavanagaiata or jgit-buildnumber
  • Fix cppwrap header references (using string replacement?)

Two calls are made to git-describe in order to detect
whether this is an exact or an inexact match for a tag.
The release.version property is then built up from the
parsed tag and should be used as the basis for all versions.
Unclear why these were set to different values.
@melissalinkert
Copy link
Member

All seems to work as expected with Ant. Do you want Maven commits added as a PR against 9639-tag-describe so that everything goes in at once, or should we merge this PR first and I'll open a new one for the corresponding Maven changes?

@joshmoore
Copy link
Member Author

Either way, @melissalinkert.

@melissalinkert
Copy link
Member

Merging then, and I'll open a corresponding Maven PR hopefully later today.

melissalinkert added a commit that referenced this pull request Aug 12, 2013
Use `git describe` for Bio-Formats version (See #9639)
@melissalinkert melissalinkert merged commit 7d1c152 into ome:develop Aug 12, 2013
@jburel
Copy link
Member

jburel commented Sep 9, 2013

--rebased-to #636

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants