← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Update version string formatting after git migration

 

The hash tag is in the revision string as well, so as long as the
revision string is supported we know exactly which commit was
used to build.

- Cirilo

On Sun, Aug 28, 2016 at 2:02 AM, Bob Gustafson <bobgus@xxxxxxx> wrote:

> On 08/27/2016 10:55 AM, Chris Pavlina wrote:
>
> On Sat, Aug 27, 2016 at 05:48:45PM +0200, jp charras wrote:
>>
>>> Le 27/08/2016 à 17:14, Chris Pavlina a écrit :
>>>
>>>> Now that we've migrated from bzr, there isn't much reason to keep
>>>> attaching a (now fake) bzr revision number to the version string.
>>>> Additionally, we can choose a sensible default branch name if one isn't
>>>> specified on the cmake line, rather than "product". This patch reformats
>>>> the version strings to:
>>>>
>>>> (2016-08-26 revision 67230ac)-master
>>>>   |                   |        |
>>>>   |                   |        custom branch name if set. Otherwise,
>>>>   |                   |        branch name, "HEAD" if not on a branch,
>>>>   |                   |          or "unknown" if no .git present
>>>>   |                   |
>>>>   |                   abbreviated commit hash, or no-git if no .git
>>>>   |                     present
>>>>   |
>>>>   date of commit, or date of build if no .git present
>>>>
>>> I find the bzr revision number useful to easily know the order of
>>> revisions.
>>> the name bzr is now a bit strange, so the version string could be:
>>>
>>> (2016-08-26 rev 1234 git 67230ac)-master
>>>
>>> (users, many times, just give a rev number, no the full version string,
>>> so in a bug or mail, rev
>>> 1234 has meaning, but revision 67230ac has no meaning, at least for me).
>>>
>> Just use the date! That's why it's there. If a user told me he had an
>> issue with rev 1234 I have no idea how I'd go about converting that to a
>> git commit now that bzr is gone. It's meaningless now except as a
>> sequence, and...that's what the date is for!
>>
> This assumes that there is at most one revision per day..
>
> Bob G
>
>>
>> --
>>> Jean-Pierre CHARRAS
>>>
>>> _______________________________________________
>>>
>>
>
> _______________________________________________
> 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
>

References