Re: [meta-arago] [master/kirkstone][PATCH v2 1/3] meta-arago-extras: sysrepo: Update checksum for nw-configurator

[Ravi Gunasekaran via lists.yoctoproject.org]

On 6/22/23 9:26 PM, Denys Dmytriyenko wrote:
> On Thu, Jun 22, 2023 at 04:04:43PM +0530, Ravi Gunasekaran wrote:
>> Update the checksum values for nw-configurator
> 
> Why?
> 
> It is always "why?" question you should answer in the commit log. Everyone 
> can see in the patch itself "what?" you are changing - license checksum - 
> and "where?" you are changing it - in nw-configurator.
> 
> Moreover, any license change when the checksum differs should be explicitly 
> explained to track any re-licensing due to legal reasons.
> 
> 
>> Signed-off-by: Ravi Gunasekaran <r-gunaseka...@ti.com>
>> ---
>>  .../recipes-sysrepo/nw-configurator/nw-configurator.bb          | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git 
>> a/meta-arago-extras/recipes-sysrepo/nw-configurator/nw-configurator.bb 
>> b/meta-arago-extras/recipes-sysrepo/nw-configurator/nw-configurator.bb
>> index 5a152017..b8a93f1c 100644
>> --- a/meta-arago-extras/recipes-sysrepo/nw-configurator/nw-configurator.bb
>> +++ b/meta-arago-extras/recipes-sysrepo/nw-configurator/nw-configurator.bb
>> @@ -1,6 +1,6 @@
>>  LICENSE="GPLv2"
>>  DESCRIPTION = "Sysrepo based repo to configure EST"
>> -LIC_FILES_CHKSUM = 
>> "file://nw-configurator.c;md5=a818a6cf4fbeeb21acc8b4e9956c08a4"
>> +LIC_FILES_CHKSUM = 
>> "file://nw-configurator.c;md5=f2ec672593c3707bb4fe68307dbf0506"
> 
> Since you are using a source file for license checksum, you might want to 
> limit it to the license text only, so code changes won't affect the checksum. 
> Use beginline=x and endline=y, e.g.:
> https://git.yoctoproject.org/meta-ti/tree/meta-ti-bsp/recipes-devtools/k3conf/k3conf_git.bb#n4

Thanks for sharing the reference. I will update accordingly

> 
> 
>>  SRC_URI = "file://nw-configurator.c"
>>  
>> -- 
>> 2.17.1

-- 
Regards,
Ravi


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14683): 
https://lists.yoctoproject.org/g/meta-arago/message/14683
Mute This Topic: https://lists.yoctoproject.org/mt/99694937/21656
Group Owner: meta-arago+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-