[GrafX2] #167: Recent releases for macOS do not have version information

1 view
Skip to first unread message

GrafX2

unread,
Sep 12, 2021, 8:06:24 AMSep 12
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------------+------------------------
Reporter: vendomoveis2010@… | Owner: pulkomandy
Type: defect | Status: new
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Keywords: |
-------------------------------+------------------------
macOS apps need the CFBundleVersion and CFBundleShortVersionString keys
set in Info.plist to display version information.

Older versions of Grafx2 always set CFBundleVersion as 1.0, but
CFBundleShortVersionString had version information. In recent releases,
CFBundleShortVersionString is empty.

Having accurate version information is useful for macOS to behave
correctly with the app, and for third-party apps to interact with it.

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167>
GrafX2 <http://pulkomandy.tk/projects/GrafX2>
The ultimate 256-color painting program

GrafX2

unread,
Sep 13, 2021, 6:21:21 AMSep 13
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+------------------------
Reporter: Carl Frank | Owner: pulkomandy
Type: defect | Status: new
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+------------------------

Comment (by Thomas Bernard):

I can see
{{{
<key>CFBundleVersion</key>
<string>1.0</string>
}}}
in the Info.plist file, and
{{{
CFBundleShortVersionString = "Grafx2 version 2.7.2942";
}}}
in English.lproj/InfoPlist.string.

@Carl Frank : could you provide correct Info.plist and InfoPList.string
files so I update them ?
Regards.

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:1>

GrafX2

unread,
Sep 13, 2021, 9:29:40 AMSep 13
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+------------------------
Reporter: Carl Frank | Owner: pulkomandy
Type: defect | Status: new
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+------------------------
Changes (by Carl Frank):

* Attachment "Info.plist" added.

GrafX2

unread,
Sep 13, 2021, 9:30:00 AMSep 13
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+------------------------
Reporter: Carl Frank | Owner: pulkomandy
Type: defect | Status: new
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+------------------------

Comment (by Carl Frank):

Technically, even those old versions are wrong. CFBundleVersion should not
remain static and CFBundleShortVersionString should just be the version,
not have "Grafx2 version" in there. The correct values would be
"CFBundleVersion" as 3104 and CFBundleShortVersionString as 2.8.

But if we're going for _consistency_, the attached Info.plist should be
enough to fix it. Only change I made was to add to it:

<key>CFBundleShortVersionString</key>
<string>Grafx2 version 2.8.3104</string>

The key did not exist in the file. You cannot localize (with is what
InfoPlist.string is for) a non-existent key.

I really don’t get why you even have that localization file, though. It’s
only one language, why not just add the keys directly to Info.plist?

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:2>

GrafX2

unread,
Sep 14, 2021, 5:21:02 PMSep 14
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+------------------------
Reporter: Carl Frank | Owner: pulkomandy
Type: defect | Status: new
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+------------------------

Comment (by Thomas Bernard):

https://developer.apple.com/documentation/bundleresources/information_property_list/cfbundleversion
says CFBundleVersion should be x.y.z

Same for CFBundleShortVersion :
https://developer.apple.com/documentation/bundleresources/information_property_list/cfbundleshortversionstring

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:3>

GrafX2

unread,
Sep 14, 2021, 5:45:27 PMSep 14
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+----------------------------
Reporter: Carl Frank | Owner: Thomas Bernard
Type: defect | Status: accepted
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+----------------------------
Changes (by Thomas Bernard):

* owner: pulkomandy => Thomas Bernard
* status: new => accepted


Comment:

can you try to download https://pulkomandy.tk/projects/GrafX2/downloads/69
and tell me if it is better ?

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:4>

GrafX2

unread,
Sep 14, 2021, 5:49:01 PMSep 14
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+----------------------------
Reporter: Carl Frank | Owner: Thomas Bernard
Type: defect | Status: accepted
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+----------------------------

Comment (by Thomas Bernard):

see https://gitlab.com/GrafX2/grafX2/-/merge_requests/322

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:5>

GrafX2

unread,
Sep 14, 2021, 5:53:01 PMSep 14
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+----------------------------
Reporter: Carl Frank | Owner: Thomas Bernard
Type: defect | Status: accepted
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+----------------------------

Comment (by Thomas Bernard):

The things are the way they are for a long time ;)
it is never too late to fix them, but I think you are the first to
complain
https://gitlab.com/GrafX2/grafX2/-/blame/master/src/Info.plist
https://gitlab.com/GrafX2/grafX2/-/commits/master/src/English.lproj/InfoPlist.strings

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:6>

GrafX2

unread,
Sep 15, 2021, 7:17:14 AMSep 15
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+----------------------------
Reporter: Carl Frank | Owner: Thomas Bernard
Type: defect | Status: accepted
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: | Keywords:
-------------------------+----------------------------

Comment (by Carl Frank):

I confirm that build is good! Thank you for being so responsive and fast
to fix!

--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:7>

GrafX2

unread,
Sep 16, 2021, 3:17:38 PMSep 16
to gra...@googlegroups.com
#167: Recent releases for macOS do not have version information
-------------------------+----------------------------
Reporter: Carl Frank | Owner: Thomas Bernard
Type: defect | Status: closed
Priority: minor | Milestone:
Component: GrafX2 | Version: 2.9WIP
Resolution: fixed | Keywords:
-------------------------+----------------------------
Changes (by Thomas Bernard):

* status: accepted => closed
* resolution: => fixed


--
Ticket URL: <http://pulkomandy.tk/projects/GrafX2/ticket/167#comment:8>
Reply all
Reply to author
Forward
0 new messages