← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Set KiCad version in MacOS apps

 

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
> 


Follow ups

References