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

version numbers missing on macOS #372

Closed
core-code opened this issue Jun 10, 2019 · 9 comments
Closed

version numbers missing on macOS #372

core-code opened this issue Jun 10, 2019 · 9 comments

Comments

@core-code
Copy link
Contributor

core-code commented Jun 10, 2019

i've downloaded version '2.3.14' but when selecting it in the macOS Finder, the Version Number entry is missing.

the CFBundleVersion and CFBundleShortVersionString entries in your Info.plist file are missing

@devernay
Copy link
Member

can see the version number in the get info panel:
image

	<key>CFBundleGetInfoString</key>
	<string>2.3</string>

The Info.plist file is generated by qmake, we don't do anything custom beyond what qmake does

@core-code
Copy link
Contributor Author

@devernay thanks for your reply. sorry, yes, i can confirm something shows up in the finder

two problems
• obviously 2.3 is not the version number. it should read 2.3.13
• actually the 'CFBundleGetInfoString' entry is deprecated for more than 10 years and needs to be removed. you need to use the 'CFBundleVersion' and 'CFBundleShortVersionString' entries

all of this should be easy with stock QMake things or just postprocessing the app with '/usr/libexec/PlistBuddy' while generating the DMG

@core-code
Copy link
Contributor Author

try adding this to your Info.plist

	<key>CFBundleShortVersionString</key>
	<string>@FULL_VERSION@</string>
	<key>CFBundleVersion</key>
	<string>@FULL_VERSION@</string>

@devernay
Copy link
Member

Feel free to do a pull request to change that. Frankly there are more important things to fix in Natron, that affect all platforms.
2.3 looks ok to me, as it is the real version number. 13 or 14 is a micro version number.

@core-code
Copy link
Contributor Author

all Mac apps should have their real version number embedded. its also important in case you have multiple versions installed. without monotonically increasing numerical CFBundleVersion that won't work, as described in the Apple documentations.

i'll try to send a PR.

@devernay
Copy link
Member

fixed by #374

@devernay
Copy link
Member

This doesn't work.
FULL_VERSION is only available in Qt5.
I'm reverting to SHORT_VERSION for the moment, we'll have to find another solution, maybe based on QMAKE_SUBSTITUTES

@devernay devernay reopened this Jul 10, 2019
@devernay
Copy link
Member

image

@devernay
Copy link
Member

devernay added a commit that referenced this issue Jul 10, 2019
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

No branches or pull requests

2 participants