Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

#1255 - Limits internal version string to 3 parts to fix patch releases #1256

Merged

Conversation

tst-ccamp
Copy link
Collaborator

Fixes #1255

Internally, Fusion will only track 3 parts of the version string in order to maintain compatibility with the asset root and with client/server messages.

Verification steps

  1. Install 5.2.5
  2. Do build
  3. Perform an upgrade to 5.2.5.2

@googlebot googlebot added the cla: yes Manual verification that all contributors have signed the CLA. label Apr 1, 2019
@tst-ccamp tst-ccamp requested a review from tst-rwildes April 1, 2019 20:10
@tst-rwildes
Copy link
Collaborator

Code looks good. I tested the regular expression and that looks good too. Assuming long version works correctly this should do what we want. Working on running over the verification steps now.

Copy link
Collaborator

@tst-rwildes tst-rwildes left a comment

Choose a reason for hiding this comment

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

I tested with these changes and after upgrading to 5.2.5.2 from 5.2.5 all the serves came up.

Copy link
Collaborator

@tst-blaird tst-blaird left a comment

Choose a reason for hiding this comment

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

Changes look good. Did not test, just code reviewed.

@tst-ccamp tst-ccamp merged commit 6a201d9 into google:release_5.2.5.2 Apr 3, 2019
husf-dsheremata added a commit to husf-dsheremata/earthenterprise that referenced this pull request Jun 4, 2019
* 'master' of https://github.com/google/earthenterprise:
  1270 Fix xml parsing in common.sh (google#1271)
  1192 update openjpeg to 2.3.1 (google#1269)
  Documented required boost dependency for Ubuntu 14 (google#1264)
  [WIP] Resolve issue google#798 - investigate Qt build performance (google#1245)
  Copy uninstall scripts (google#1232)
  refresh commit
  Issue google#1255 - Limits internal version string to 3 parts to fix various errors with patch releases (google#1256)
  Move build number to release rpm (google#1252)
  fix version
  link up 5.3.1 doc folder (google#1249)
  I1246 create 5.3.1 doc folder from copy of 5.3.0 doc folder (google#1248)
  change version files over to 5.3.1 (google#1247)
  start of 5.3.1
  Fixes google#279 - Admin Console password will be preserved on GEE server upgrade (google#1243)
  google#1201 - Fix issue with setting source date when using .kip directories (google#1234)
  I1228 Add y-scroll-bar to GLC Assembly webpage (google#1230)
  google#1236 Polygons constructor can accept KML as a string (google#1239)
  I1225 Re-add GROUPNAME to install_server.sh (google#1233)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Manual verification that all contributors have signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants