← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Set KiCad version in MacOS apps

 

Hey

I closed the merge request on Launchpad and re-attached the patch and
before and after screenshots (the after being built from a dirty master
branch) to this mail:

Before:
[image: Screenshot 2019-06-05 at 22.46.43.png]
After:
[image: Screenshot 2019-06-05 at 22.46.54.png]

As you can see, having this version information displayed in Spotlight
makes choosing the correct KiCad version a lot easier. It works for the
other apps (EEschema, PCBNew, ...) as well.

Greetings
Seppe

On Tue, Jun 11, 2019 at 2:50 PM Seth Hillbrand <seth@xxxxxxxxxxxxx> wrote:

> 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