[meta-freescale] [3rdparty][PATCH 03/15] u-boot-toradex: update to 2016.11

[Max Krummenacher]

Hi Otavio, Stefan

That is what I changed it to on Friday.

+LOCALVERSION = "-${PR}"
+PR = "2.7.3"
+PV_append = "+git${SRCPV}"

I still have to test the resulting images which incorporates the changes and have to check the LOCALVERSION proposal from Otavio.

Max


________________________________
Von: Otavio Salvador <otavio.salvador at ossystems.com.br>
Gesendet: Montag, 11. September 2017 19:12:06
An: Stefan Agner
Cc: Max Krummenacher; meta-freescale at yoctoproject.org; Marcel Ziswiler
Betreff: Re: [meta-freescale] [3rdparty][PATCH 03/15] u-boot-toradex: update to 2016.11

On Mon, Sep 11, 2017 at 2:09 PM, Stefan Agner <stefan.agner at toradex.com> wrote:
>
> On 08.09.2017 05:15, Otavio Salvador wrote:
>> On Thu, Sep 7, 2017 at 2:50 PM, Max Krummenacher
>> <max.krummenacher at toradex.com> wrote:
>>> Hi Otavio
>>>
>>> On Wed, 2017-09-06 at 16:23 -0300, Otavio Salvador wrote:
>>>> On Wed, Sep 6, 2017 at 11:12 AM, Max Krummenacher <max.oss.09 at gmail.com> wrote:
>>>>> ...
>>>>> +
>>>>> +PV = "2016.11"
>>>>> +PR = "2.7.3_gitr${@d.getVar("SRCREV", False)[0:7]}"
>>>> Use ...git${SRCPV}; SRCREV alone is not sortable.
>>> Ok. I will rework that in v2.
>>> This probably requires me to move the SRCPV to be part of PV. Otherwise
>>> AUTOINC will not get replaced by a number in the buildprocess.
>> I'd put Toradex's version on the LOCALVERSION variable and would not  touch PR.
>>
> I also would prefer ~ or + to separate the git hash, this is more common in the distro world:
> https://lists.debian.org/debian-mentors/2011/03/msg00326.html

Please use + so it does imply it adds on top of this version.

--
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-freescale/attachments/20170911/e1c2bcf5/attachment.html>