← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Set KiCad version in MacOS apps

 

Hi Seppe-

I see this e-mail.  Perhaps it was a launchpad hiccup.

I've added Adam to the code review at [1]. Would you mind re-sending the images to the list?

Thanks-
Seth

[1] https://code.launchpad.net/~seppestas/kicad/+git/kicad/+merge/368644

On 2019-06-11 05:23, Seppe Stas wrote:
Hey

I'm not sure if this email got ignored or if it got rejected by some
mailing system, but it does not seem to show up in the mailing list archive
<https://lists.launchpad.net/kicad-developers/date.html>.

Maybe now it works?

Seppe

On Wed, Jun 5, 2019 at 10:55 PM Seppe Stas <seppestas@xxxxxxxxx> wrote:

Hey guys and girls (probably mostly Adam in particular)

Attached is a patch that sets the version in all MacOS apps to the value of KICAD_VERSION, i.e the value of git describe. See commit message for more technical details. This version show up when e.g launching the app using spotlight, and during the migration period from KiCad 4 to KiCad 5 (that is still going on) I really miss(ed) this feature, since I have 3
different versions of KiCad on my system (4, 5 and master).

See attached screenshots:
[image: Screenshot 2019-06-05 at 22.46.54.png]
[image: Screenshot 2019-06-05 at 22.46.43.png]

I tested this patch on both the latest master (6f8a0a4ee) and the 5.1
branch (cd6da987c). I hope you consider adding it to a KiCad 5 release so I can use it when KiCad 6 comes out. (I hope I finished migrating to KiCad 5
by then).

Note that in order to update this value, CMake has to be re-run, but I
would not expect this to be a big problem since I assume the releases are
always built from scratch.

Greeting
Seppe Stas


_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Follow ups

References