kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #40981
Re: [PATCH] Set KiCad version in MacOS apps
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Tue, 11 Jun 2019 10:27:25 -0400
-
Autocrypt:
addr=stambaughw@xxxxxxxxx; prefer-encrypt=mutual; keydata= mQGiBEM0hxQRBAC2fNh3YOVLu1d5GZ0SbrTNldGiGnCJPLqzEnqFX9v6jmf33TMt6EmSLkl6 Wtfkoj0nVwKxcYmJkA8DX0QAokBkwNIzhSsBzQvthBLIk/5LnPVVKrEXOcL4mUyH1doKlkaE slgJozNa6Av+oavcvD02o1zJOloBbaHlNlyRt7fKswCgtIFlVjWggVH/15KfWk+Qo5JVPbME AIUBAQyL2OAx0n60AWec2WHnO9buHuG0ibtICgUMkE+2MRmYyKwYRdyVwGoIUemFuOyHp0AJ InX4T+vy2E7vkwODqjtMLfIoRkokW74Fi4nrvjlhOAw/vdq/twLbAmR9MOfPTpR4y7kQy1O2 /n+RkkRvh26vTzfbQmrH7cBJhk6aA/9Uwvu3E4zNJgHVZeS0HyWtmR1eOPPRbnkPgJTToX5O KMKzTJI/FX6kT7cFoCamitHrW3BJP4Dx+cMMsa47EGxqVTdbVJ4LjogsXTXxb+0Fn1u4zBdx x3Cer6O7+hqWy7zvpzeC6nSREjqDKa5CgHtv/GLm5uFPOmsjAsnHj2tlBrQmV2F5bmUgU3Rh bWJhdWdoIDxzdGFtYmF1Z2h3QGdtYWlsLmNvbT6IeAQTEQIAOBYhBOffs6CbblRzBkv33BtR cWlZ+CReBQJbFBS2AhsDBQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAAAoJEBtRcWlZ+CReMI8A nRbrLkzp7+c2f0vX7sfg4ICX8LAKAJ9uClo4uJajmZa5zZrL2nKdZlUwIrkCDQRDNIcxEAgA gCru+3/aOC6RCjpvYC72wY+d5SmHphC6yeiV2/mOumyt5MLo/Ps2GznZr11JspqFk5K/Zpvp MMLqqjDZ39+50a2iKRQFJ6NlK+hJWMmj6eJygQrCwYo3Gjc6CqfrqUv+8VSnf/i5sIZmtOVA 4ZjML18MuBvMSsNdVLFJd5HNnYb1iOECpvqdPVh/21LLCEw7MUUGGnHBhCrmk2aJe5hFmcSN g4ldBcXrgMQBwf7aMVoobXBMFDb/IENByXn0llB7Gr2IFMRmNS9/p8s/II1Yl2bTqyX4FSz8 cfn7C9KEz7faZ7wzAcpwHFC/zs3JoAjJ0IEKdNUpIwAlKMzT3CzctwADBQf/cxpG28MKyrqk nNmq/8LQLy+x6FSYXBLjxQz9BiBNYeesDZQ6J5UbL1mjpJzMa5tLZypPYo4bbGyR22hrbyDF K7m6AcVaMIJKl98g4ukMutFfAJyRDaREH5Zl/X1P4u1Z/yaAIy9mKaNbaK1/5djNJ5wCTFen TUgAp9xdc30kGkFDdLJFp5uxDY4P0vaZiZdjUCvDM3Zjv5IzpNOfxVqTUBQNUP/BnnKhkk0p DTD6s3X8S+D0rOtEBQ8K0cwERI/E8EFa8nj0TNw4e2MYGR8wg+SxqJ7z5f0zPY0bO6G9DDFB wYCqzzPWGqdAh9vA5971TAbPERtdFybhkurozp2SfYhJBBgRAgAJBQJDNIcxAhsMAAoJEBtR cWlZ+CResHUAniULLCWiT26ieRTl7N2vS6vBo/DuAJ4m7Ss/gyiW6ybTn1ctDXAUgm2QVQ==
-
In-reply-to:
<CAHBy37fG9tK9W2xBGEaVYRGXSo55vTy=L=6d-yQ13iE9JgbV+w@mail.gmail.com>
-
Openpgp:
preference=signencrypt
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0
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