Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Khem Raj 
I wonder if we should start using sha512sum along with sha256sum as
default with devtool upgrade and skip others.

On Wed, Dec 6, 2023 at 11:22 AM Peter Kjellerstedt
 wrote:
>
> Rather than only updating the sha256sum and removing the md5sum, update
> all existing checksums. If the only existing checksum is md5sum, then
> replace it with the default expected checksums.
>
> Signed-off-by: Peter Kjellerstedt 
> ---
>  .../devtool/devtool-upgrade-test3_1.5.3.bb| 16 ++
>  .../devtool-upgrade-test3_1.5.3.bb.upgraded   | 15 ++
>  .../devtool/devtool-upgrade-test4_1.5.3.bb| 22 
>  .../devtool-upgrade-test4_1.5.3.bb.upgraded   | 19 +++
>  meta/lib/oeqa/selftest/cases/devtool.py   | 48 +
>  scripts/lib/devtool/upgrade.py| 51 ++-
>  6 files changed, 148 insertions(+), 23 deletions(-)
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
>
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> new file mode 100644
> index 00..69c0d351ec
> --- /dev/null
> +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> @@ -0,0 +1,16 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> new file mode 100644
> index 00..3ce7e85e10
> --- /dev/null
> +++ 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> @@ -0,0 +1,15 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI[sha256sum] = 
> "9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> new file mode 100644
> index 00..9abf80e6ed
> --- /dev/null
> +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> @@ -0,0 +1,22 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> +SRC_URI[sha1sum] = "63a0801350e812541c7f8e9ad74e0d6b629d0b39"
> +SRC_URI[sha256sum] = 
> "681bcca9784bf3cb2207e68236d1f68e2aa7b80f999b5750dc77dcd756e81fbc"
> +SRC_URI[sha384sum] = 
> "5fff6390465ff23dbf573fcf39dfad3aed2f92074a35e6c02abe58b7678858d90fa6572ff4cb56df8b3e217c739cdbe3"
> +SRC_URI[sha512sum] = 
> "32efe7071a363f547afc74e96774f711795edda1d2702823a347d0f9953e859b7d8c45b3e63e18ffb9e0d5ed5910be652d7d727c8676e81b6cb3aed0b13aec00"
> +
> +PR = "r5"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> new file mode 100644
> index 00..cd2a0842f4
> --- /dev/null
> +++ 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> @@ -0,0 +1,19 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI

Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Alexander Kanavin 
On Wed, 6 Dec 2023 at 20:46, Peter Kjellerstedt
 wrote:
> Will changing that to:
>
> In addition to updating the sha256sum and removing the md5sum, update
> all other existing checksums. If the only existing checksum is md5sum, then
> replace it with the default expected checksums (currently only sha256sum).
>
> make it clearer?

Thanks, yes that is fine.

Alex

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

Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Peter Kjellerstedt 
> -Original Message-
> From: Alexander Kanavin 
> Sent: den 6 december 2023 20:26
> To: Peter Kjellerstedt 
> Cc: openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing 
> checksums for the SRC_URI
> 
> I mean, what possible reason there could be to keep them? None as far
> as I can see.
> 
> Alex
> 
> On Wed, 6 Dec 2023 at 20:25, Alexander Kanavin 
> wrote:
> >
> > We've been relying on this to actually remove md5sums on version
> > updates, so please do not regress that.

Sorry, if it was unclear in the commit message, but the code still of 
course removes md5sum.

> >
> > Alex
> >
> > On Wed, 6 Dec 2023 at 20:22, Peter Kjellerstedt 
> >  wrote:
> > >
> > > Rather than only updating the sha256sum and removing the md5sum, update
> > > all existing checksums. If the only existing checksum is md5sum, then
> > > replace it with the default expected checksums.

Will changing that to:

In addition to updating the sha256sum and removing the md5sum, update
all other existing checksums. If the only existing checksum is md5sum, then
replace it with the default expected checksums (currently only sha256sum).

make it clearer?

//Peter

> > >
> > > Signed-off-by: Peter Kjellerstedt 
> > > ---
> > >  .../devtool/devtool-upgrade-test3_1.5.3.bb| 16 ++
> > >  .../devtool-upgrade-test3_1.5.3.bb.upgraded   | 15 ++
> > >  .../devtool/devtool-upgrade-test4_1.5.3.bb| 22 
> > >  .../devtool-upgrade-test4_1.5.3.bb.upgraded   | 19 +++
> > >  meta/lib/oeqa/selftest/cases/devtool.py   | 48 +
> > >  scripts/lib/devtool/upgrade.py| 51 ++
> -
> > >  6 files changed, 148 insertions(+), 23 deletions(-)
> > >  create mode 100644 
> > > meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> > >  create mode 100644 
> > > meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > >  create mode 100644 
> > > meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> > >  create mode 100644 
> > > meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> > >
> > > diff --git 
> > > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb 
> > > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> > > new file mode 100644
> > > index 00..69c0d351ec
> > > --- /dev/null
> > > +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> > > @@ -0,0 +1,16 @@
> > > +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> > > +LICENSE = "Artistic-2.0"
> > > +LIC_FILES_CHKSUM = 
> > > "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> > > +
> > > +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> > > +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> > > +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> > > feature"
> > > +
> > > +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> > > +
> > > +S = "${WORKDIR}/pv-${PV}"
> > > +
> > > +EXCLUDE_FROM_WORLD = "1"
> > > +
> > > +inherit autotools
> > > +
> > > diff --git 
> > > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > >  
> > > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > > new file mode 100644
> > > index 00..3ce7e85e10
> > > --- /dev/null
> > > +++ 
> > > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > > @@ -0,0 +1,15 @@
> > > +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> > > +LICENSE = "Artistic-2.0"
> > > +LIC_FILES_CHKSUM = 
> > > "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> > > +
> > > +SRC_URI[sha256sum] = 
> > > "9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
> > > +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> > > +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> > > +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> > > feature"
> > > +
> > > +S = &quo

Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Alexander Kanavin 
I mean, what possible reason there could be to keep them? None as far
as I can see.

Alex

On Wed, 6 Dec 2023 at 20:25, Alexander Kanavin  wrote:
>
> We've been relying on this to actually remove md5sums on version
> updates, so please do not regress that.
>
> Alex
>
> On Wed, 6 Dec 2023 at 20:22, Peter Kjellerstedt
>  wrote:
> >
> > Rather than only updating the sha256sum and removing the md5sum, update
> > all existing checksums. If the only existing checksum is md5sum, then
> > replace it with the default expected checksums.
> >
> > Signed-off-by: Peter Kjellerstedt 
> > ---
> >  .../devtool/devtool-upgrade-test3_1.5.3.bb| 16 ++
> >  .../devtool-upgrade-test3_1.5.3.bb.upgraded   | 15 ++
> >  .../devtool/devtool-upgrade-test4_1.5.3.bb| 22 
> >  .../devtool-upgrade-test4_1.5.3.bb.upgraded   | 19 +++
> >  meta/lib/oeqa/selftest/cases/devtool.py   | 48 +
> >  scripts/lib/devtool/upgrade.py| 51 ++-
> >  6 files changed, 148 insertions(+), 23 deletions(-)
> >  create mode 100644 
> > meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> >  create mode 100644 
> > meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> >  create mode 100644 
> > meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> >  create mode 100644 
> > meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> >
> > diff --git 
> > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb 
> > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> > new file mode 100644
> > index 00..69c0d351ec
> > --- /dev/null
> > +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> > @@ -0,0 +1,16 @@
> > +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> > +LICENSE = "Artistic-2.0"
> > +LIC_FILES_CHKSUM = 
> > "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> > +
> > +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> > +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> > +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> > feature"
> > +
> > +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> > +
> > +S = "${WORKDIR}/pv-${PV}"
> > +
> > +EXCLUDE_FROM_WORLD = "1"
> > +
> > +inherit autotools
> > +
> > diff --git 
> > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> >  
> > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > new file mode 100644
> > index 00..3ce7e85e10
> > --- /dev/null
> > +++ 
> > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> > @@ -0,0 +1,15 @@
> > +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> > +LICENSE = "Artistic-2.0"
> > +LIC_FILES_CHKSUM = 
> > "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> > +
> > +SRC_URI[sha256sum] = 
> > "9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
> > +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> > +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> > +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> > feature"
> > +
> > +S = "${WORKDIR}/pv-${PV}"
> > +
> > +EXCLUDE_FROM_WORLD = "1"
> > +
> > +inherit autotools
> > +
> > diff --git 
> > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb 
> > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> > new file mode 100644
> > index 00..9abf80e6ed
> > --- /dev/null
> > +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> > @@ -0,0 +1,22 @@
> > +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> > +LICENSE = "Artistic-2.0"
> > +LIC_FILES_CHKSUM = 
> > "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> > +
> > +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> > +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> > +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> > feature"
> > +
> > +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> > +SRC_URI[sha1sum] = "63a0801350e812541c7f8e9ad74e0d6b629d0b39"
> > +SRC_URI[sha256sum] = 
> > "681bcca9784bf3cb2207e68236d1f68e2aa7b80f999b5750dc77dcd756e81fbc"
> > +SRC_URI[sha384sum] = 
> > "5fff6390465ff23dbf573fcf39dfad3aed2f92074a35e6c02abe58b7678858d90fa6572ff4cb56df8b3e217c739cdbe3"
> > +SRC_URI[sha512sum] = 
> > "32efe7071a363f547afc74e96774f711795edda1d2702823a347d0f9953e859b7d8c45b3e63e18ffb9e0d5ed5910be652d7d727c8676e81b6cb3aed0b13aec00"
> > +
> > +PR = "r5"
> > +
> > +S = "${WORKDIR}/pv-${PV}"
> > +
> > +EXCLUDE_FROM_WORLD = "1"
> > +
> > +inherit autotools
> > +
> > diff --git 
> > a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> >  
> > b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> > new file mode 100644
> > index

Re: [OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Alexander Kanavin 
We've been relying on this to actually remove md5sums on version
updates, so please do not regress that.

Alex

On Wed, 6 Dec 2023 at 20:22, Peter Kjellerstedt
 wrote:
>
> Rather than only updating the sha256sum and removing the md5sum, update
> all existing checksums. If the only existing checksum is md5sum, then
> replace it with the default expected checksums.
>
> Signed-off-by: Peter Kjellerstedt 
> ---
>  .../devtool/devtool-upgrade-test3_1.5.3.bb| 16 ++
>  .../devtool-upgrade-test3_1.5.3.bb.upgraded   | 15 ++
>  .../devtool/devtool-upgrade-test4_1.5.3.bb| 22 
>  .../devtool-upgrade-test4_1.5.3.bb.upgraded   | 19 +++
>  meta/lib/oeqa/selftest/cases/devtool.py   | 48 +
>  scripts/lib/devtool/upgrade.py| 51 ++-
>  6 files changed, 148 insertions(+), 23 deletions(-)
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
>  create mode 100644 
> meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
>
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> new file mode 100644
> index 00..69c0d351ec
> --- /dev/null
> +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
> @@ -0,0 +1,16 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> new file mode 100644
> index 00..3ce7e85e10
> --- /dev/null
> +++ 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
> @@ -0,0 +1,15 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI[sha256sum] = 
> "9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> new file mode 100644
> index 00..9abf80e6ed
> --- /dev/null
> +++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
> @@ -0,0 +1,22 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
> +RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade 
> feature"
> +
> +SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
> +SRC_URI[sha1sum] = "63a0801350e812541c7f8e9ad74e0d6b629d0b39"
> +SRC_URI[sha256sum] = 
> "681bcca9784bf3cb2207e68236d1f68e2aa7b80f999b5750dc77dcd756e81fbc"
> +SRC_URI[sha384sum] = 
> "5fff6390465ff23dbf573fcf39dfad3aed2f92074a35e6c02abe58b7678858d90fa6572ff4cb56df8b3e217c739cdbe3"
> +SRC_URI[sha512sum] = 
> "32efe7071a363f547afc74e96774f711795edda1d2702823a347d0f9953e859b7d8c45b3e63e18ffb9e0d5ed5910be652d7d727c8676e81b6cb3aed0b13aec00"
> +
> +PR = "r5"
> +
> +S = "${WORKDIR}/pv-${PV}"
> +
> +EXCLUDE_FROM_WORLD = "1"
> +
> +inherit autotools
> +
> diff --git 
> a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> new file mode 100644
> index 00..cd2a0842f4
> --- /dev/null
> +++ 
> b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
> @@ -0,0 +1,19 @@
> +SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
> +LICENSE = "Artistic-2.0"
> +LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
> +
> +SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
> +UPSTREAM_CHECK_URI =

[OE-core] [PATCH 8/9] devtool: upgrade: Update all existing checksums for the SRC_URI

2023-12-06 Thread Peter Kjellerstedt 
Rather than only updating the sha256sum and removing the md5sum, update
all existing checksums. If the only existing checksum is md5sum, then
replace it with the default expected checksums.

Signed-off-by: Peter Kjellerstedt 
---
 .../devtool/devtool-upgrade-test3_1.5.3.bb| 16 ++
 .../devtool-upgrade-test3_1.5.3.bb.upgraded   | 15 ++
 .../devtool/devtool-upgrade-test4_1.5.3.bb| 22 
 .../devtool-upgrade-test4_1.5.3.bb.upgraded   | 19 +++
 meta/lib/oeqa/selftest/cases/devtool.py   | 48 +
 scripts/lib/devtool/upgrade.py| 51 ++-
 6 files changed, 148 insertions(+), 23 deletions(-)
 create mode 100644 
meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
 create mode 100644 
meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
 create mode 100644 
meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
 create mode 100644 
meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded

diff --git a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb 
b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
new file mode 100644
index 00..69c0d351ec
--- /dev/null
+++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb
@@ -0,0 +1,16 @@
+SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
+LICENSE = "Artistic-2.0"
+LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
+
+SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
+UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
+RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade feature"
+
+SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
+
+S = "${WORKDIR}/pv-${PV}"
+
+EXCLUDE_FROM_WORLD = "1"
+
+inherit autotools
+
diff --git 
a/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded 
b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
new file mode 100644
index 00..3ce7e85e10
--- /dev/null
+++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test3_1.5.3.bb.upgraded
@@ -0,0 +1,15 @@
+SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
+LICENSE = "Artistic-2.0"
+LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
+
+SRC_URI[sha256sum] = 
"9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
+SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
+UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
+RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade feature"
+
+S = "${WORKDIR}/pv-${PV}"
+
+EXCLUDE_FROM_WORLD = "1"
+
+inherit autotools
+
diff --git a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb 
b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
new file mode 100644
index 00..9abf80e6ed
--- /dev/null
+++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb
@@ -0,0 +1,22 @@
+SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
+LICENSE = "Artistic-2.0"
+LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
+
+SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
+UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
+RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade feature"
+
+SRC_URI[md5sum] = "9365d86bd884222b4bf1039b5a9ed1bd"
+SRC_URI[sha1sum] = "63a0801350e812541c7f8e9ad74e0d6b629d0b39"
+SRC_URI[sha256sum] = 
"681bcca9784bf3cb2207e68236d1f68e2aa7b80f999b5750dc77dcd756e81fbc"
+SRC_URI[sha384sum] = 
"5fff6390465ff23dbf573fcf39dfad3aed2f92074a35e6c02abe58b7678858d90fa6572ff4cb56df8b3e217c739cdbe3"
+SRC_URI[sha512sum] = 
"32efe7071a363f547afc74e96774f711795edda1d2702823a347d0f9953e859b7d8c45b3e63e18ffb9e0d5ed5910be652d7d727c8676e81b6cb3aed0b13aec00"
+
+PR = "r5"
+
+S = "${WORKDIR}/pv-${PV}"
+
+EXCLUDE_FROM_WORLD = "1"
+
+inherit autotools
+
diff --git 
a/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded 
b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
new file mode 100644
index 00..cd2a0842f4
--- /dev/null
+++ b/meta-selftest/recipes-test/devtool/devtool-upgrade-test4_1.5.3.bb.upgraded
@@ -0,0 +1,19 @@
+SUMMARY = "Pipe viewer test recipe for devtool upgrade test"
+LICENSE = "Artistic-2.0"
+LIC_FILES_CHKSUM = "file://doc/COPYING;md5=9c50db2589ee3ef10a9b7b2e50ce1d02"
+
+SRC_URI = "http://www.ivarch.com/programs/sources/pv-${PV}.tar.gz;
+UPSTREAM_CHECK_URI = "http://www.ivarch.com/programs/pv.shtml;
+RECIPE_NO_UPDATE_REASON = "This recipe is used to test devtool upgrade feature"
+
+SRC_URI[sha1sum] = "395ce62f4f3e035b86c77038f04b96c5aa233595"
+SRC_URI[sha256sum] = 
"9dd45391806b0ed215abee4c5ac1597d018c386fe9c1f5afd2f6bc3b07fd82c3"
+SRC_URI[sha384sum] = 
"218c8d2d097aeba5310be759bc20573f18ffa0b11701eac6dd2e7e14ddf13c6e0e094ca7ca026eaa05ef92a056402e36"
+SRC_URI[sha512sum] =