kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #40997
Re: [PATCH] Set KiCad version in MacOS apps
I will review today.
Thanks for your help, Seppe!
On Tue, Jun 11, 2019, 10:11 AM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> Seppe,
>
> Your patch looks good to me. Any MacOS devs care to comment?
>
> Cheers,
>
> Wayne
>
> On 6/11/19 10:46 AM, Seppe Stas wrote:
> > Hey Wayne
> >
> > I attached my patch (generated with `git format-patch --attach
> > origin/master`) to my last email as
> > per http://www.kicad-pcb.org/contribute/developers/#_submitting_patches.
> > I have a feeling Gmail might not like the mail headers in the patch.
> >
> > I created a new patch without the --attach option and added it to this
> > email (I am more used to this patch format and I believe it worked in
> > the past).
> >
> > Greetings
> > Seppe
> >
> > On Tue, Jun 11, 2019 at 4:27 PM Wayne Stambaugh <stambaughw@xxxxxxxxx
> > <mailto:stambaughw@xxxxxxxxx>> wrote:
> >
> > Seppe,
> >
> > I don't understand why your emails keep ending up on the moderated
> list
> > but something strange is going on. I had to moderate this one as
> well.
> > Please attach your patch (created using `git format-patch`) so it
> can
> > be reviewed and commented on.
> >
> > Cheers,
> >
> > Wayne
> >
> > On 6/11/19 10:17 AM, Seppe Stas wrote:
> > > 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
> > <mailto: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
> > <mailto: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
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > >>> Unsubscribe : https://launchpad.net/~kicad-developers
> > >>> More help : https://help.launchpad.net/ListHelp
> > >>
> > >
> > >
> > > _______________________________________________
> > > Mailing list: https://launchpad.net/~kicad-developers
> > > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > > Unsubscribe : https://launchpad.net/~kicad-developers
> > > More help : https://help.launchpad.net/ListHelp
> > >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help : https://help.launchpad.net/ListHelp
> >
>
> _______________________________________________
> 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