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 <[email protected]>
> ---
>  .../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


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


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14679): 
https://lists.yoctoproject.org/g/meta-arago/message/14679
Mute This Topic: https://lists.yoctoproject.org/mt/99694937/21656
Group Owner: [email protected]
Unsubscribe: 
https://lists.yoctoproject.org/g/meta-arago/leave/10763299/21656/89520264/xyzzy 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to