← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Set KiCad version in MacOS apps

 

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>
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>
> 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
> >>
> >
> >
> > _______________________________________________
> > 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
> >
>
> _______________________________________________
> 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
>

Attachment: 0001-Set-KiCad-version-in-MacOS-apps.patch
Description: Binary data


Follow ups

References