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

fix installer's Product Version inconsistency with older versions #257

Merged
merged 1 commit into from
Sep 23, 2019

Conversation

AudricGuerin
Copy link
Contributor

Why: Third-party applications or IT tools must be able to know
if the latest Git version is installed on the system
by means of version number comparisons.

How: Remove platform identifier from the Product Version.

Product Version now displays x.x.x.x instead of x.x.x.windows.x

See git-for-windows/git#2223

Signed-off-by: Audric Guerin 49029709+AudricGuerin@users.noreply.github.com

@PhilipOakley
Copy link
Contributor

I think this may also need a method for updating the git version --build-options so that when some user makes a report on the main Git mailing list folks can tell if they are talking about Linux version or Windows version , especially if a Linux maint version (.1) becomes confused with the Windows initial version (.1)

(I accept that some users don't know how to report bugs so are missing such info, but let's help those who make an effort..)

@PhilipOakley
Copy link
Contributor

I think this may also need a method for updating the git version --build-options

Maybe add a
OS platform: xxx line

@AudricGuerin
Copy link
Contributor Author

Only the Product Version of the installer was changed so git version still returns:
git version 2.23.0.windows.1

There is no problem with git-for-windows adding a platform identifier for its internal versioning purposes.
I totally understand why you would need this as I had to do such a thing this year for my company... However, I would not add an app-specific version identifier into the 'Product Version' or 'File Version' of any Windows installers or binaries of the project.

Instead I would write it in a .txt, .xml or .json file as suggested by @dscho here:
git-for-windows/git#2223 (comment)

For example, I would be totally willing to accept a PR that writes the current version in plain text (and in the expected, non-SemVer format) into the top-level directory.

And if you really want a Windows user to access Git internal version identifier through Properties > Details of the installer or Settings > Apps & features, then add it to 'Product Name' as @sendqueery said:
git-for-windows/git#2223 (comment)

If you really want it that badly, change the product name field (which also displays the version for some reason) to something like "Git version x.x.x for Windows". (Sidenote: don't do this, as it's still an unnecessary tag. We know it's Windows software by virtue of the fact that WE'RE INSTALLING A WIN32 BINARY ON WINDOWS.)

And by the way, thank you for redirecting me to the right place, Philip. ;)

@PhilipOakley
Copy link
Contributor

Do you fancy copying some of that clarifying explanation into the commit message so I don't fall into the same traps again... Especially if we have to explain it (our versions vs their single version) to the Linux upstream mail list etc etc.

Why: Third-party applications or IT tools must be able to know
if the latest Git version is installed on the system
by means of version number comparisons.

How: Remove platform identifier from the Product Version.

Product Version now displays x.x.x.x instead of x.x.x.windows.x

See git-for-windows/git#2223

Remarks:
Project's binaries Product Version are unchanged by this commit (only the installer's one)
Git internal versioning is left untouched ; 'git version' still returns x.x.x.windows.x (this commit addresses versioning issue from the OS point of view so the installer executable is the only file affected by this commit)

Signed-off-by: Audric Guerin <49029709+AudricGuerin@users.noreply.github.com>
@AudricGuerin
Copy link
Contributor Author

I added some remarks into the commit message to clarify that the Product Version of the installer file is the only version identifier affected by this commit.

@dscho dscho merged commit 9908e84 into git-for-windows:master Sep 23, 2019
dscho added a commit that referenced this pull request Oct 9, 2019
The installer's "ProductVersion" [is now
consistent with older Git for Windows
versions'](#257).

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants