[PATCH yocto-autobuilder-helper]scripts/publish-artefacts: Fix for Bug 13947:Change checksum file extension from sha256sum to sha256

[PATCH yocto-autobuilder-helper]scripts/publish-artefacts: Fix for Bug 13947:Change checksum file extension from sha256sum to sha256

[Vineela]

From: Vineela Tummalapalli <vineela.tummalapalli@intel.com>


---
 scripts/publish-artefacts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/publish-artefacts b/scripts/publish-artefacts
index 6ed922a..3ef7206 100755
--- a/scripts/publish-artefacts
+++ b/scripts/publish-artefacts
@@ -7,10 +7,10 @@ sha256sums(){
     dest=$1
     for x in `find -L $dest -maxdepth 5 -type f`; do
         for w in $x;do
-            if [ ${w##*.} != sha256sum ]; then
+            if [ ${w##*.} != sha256 ]; then
                 shasum=`sha256sum $w | awk '{print $1}'`
                 filename=${w##*/}
-                echo $shasum $filename >> $w.sha256sum
+                echo $shasum $filename > $w.sha256
             fi
         done
     done
--
2.7.4

Re: [yocto] [PATCH yocto-autobuilder-helper]scripts/publish-artefacts: Fix for Bug 13947:Change checksum file extension from sha256sum to sha256

[Richard Purdie]

On Wed, 2020-06-17 at 20:12 +0000, Vineela wrote:
> From: Vineela Tummalapalli <vineela.tummalapalli@intel.com>
> 
> 
> ---
>  scripts/publish-artefacts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/publish-artefacts b/scripts/publish-artefacts
> index 6ed922a..3ef7206 100755
> --- a/scripts/publish-artefacts
> +++ b/scripts/publish-artefacts
> @@ -7,10 +7,10 @@ sha256sums(){
>      dest=$1
>      for x in `find -L $dest -maxdepth 5 -type f`; do
>          for w in $x;do
> -            if [ ${w##*.} != sha256sum ]; then
> +            if [ ${w##*.} != sha256 ]; then
>                  shasum=`sha256sum $w | awk '{print $1}'`
>                  filename=${w##*/}
> -                echo $shasum $filename >> $w.sha256sum
> +                echo $shasum $filename > $w.sha256
>              fi
>          done
>      done

This is turning into a bit of a mess. The preferred naming is
"X.sha256sum" since it says what they are and is the most clear. In 3.1
and mostly in 3.1.1 they were called "X.sha256".

We need to decide if we just switch to the naming most of us don't
like, or we switch to the one which we'd prefer. I'm leaning to
switching as there isn't much that depends on these (yet) apart from
the buildtools script which we'd need to patch with the new version url
anyway.

Any other strong opinions? There was a recent comment we should
probably have a manifest file containing the checksums, I'm wondering
if that should replace the individual checksum files anyway?

Cheers,

Richard