[PATCH 0/4] Make missing checksums an error v2 + clutter-gst update

[PATCH 0/4] Make missing checksums an error v2 + clutter-gst update

[Joshua Lock]

NOTE: this requires a patch I sent to the BitBake list[1] to error cleanly,
otherwise you'll see a Python backtrace and the build fail...

This is v2 of a series to add set BB_STRICT_CHECKSUM in default-distrovars.inc
so that missing checksums in recipes raises an error.

Since v1 I've run a test over OE-core to ensure to ensure I can do world
fetches for each of the qemu machines and discovered that omap3-sgx-modules
and puzzles recipes needed checksums adding.

I've snuck an extra patch lying on my disk into the queue too, an update for
clutter-gst.

Regards,
Joshua


The following changes since commit 044324465bd54d53ae768f3c1e7468ae0e0c6200:

  dpkg-native: Fix perl path (2011-12-02 15:31:03 +0000)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib josh/checksums
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/checksums

Joshua Lock (4):
  omap3-sgx-modules: set SRC_URI checksums
  puzzles: set SRC_URI checksums
  default-distrovars: missing checksums should raise an error
  clutter-gst: upgrade to 1.4.4

 meta/conf/distro/include/default-distrovars.inc    |    3 +++
 .../omap3-sgx-modules_1.3.13.1397.bb               |    2 ++
 ...r-gst-1.8_1.4.2.bb => clutter-gst-1.8_1.4.4.bb} |    4 ++--
 meta/recipes-sato/puzzles/puzzles_r9306.bb         |    8 ++++----
 4 files changed, 11 insertions(+), 6 deletions(-)
 rename meta/recipes-graphics/clutter/{clutter-gst-1.8_1.4.2.bb => clutter-gst-1.8_1.4.4.bb} (80%)

-- 
1.7.7.3

[PATCH 1/4] omap3-sgx-modules: set SRC_URI checksums

[Joshua Lock]

We'd like to make missing SRC_URI checksums an error, so all
appropriate recipes need to have a checksum value set.

Signed-off-by: Joshua Lock <josh@linux.intel.com>
---
 .../omap3-sgx-modules_1.3.13.1397.bb               |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-bsp/powervr-drivers/omap3-sgx-modules_1.3.13.1397.bb b/meta/recipes-bsp/powervr-drivers/omap3-sgx-modules_1.3.13.1397.bb
index 0e850bc..fba0e18 100644
--- a/meta/recipes-bsp/powervr-drivers/omap3-sgx-modules_1.3.13.1397.bb
+++ b/meta/recipes-bsp/powervr-drivers/omap3-sgx-modules_1.3.13.1397.bb
@@ -4,6 +4,8 @@ LICENSE = "GPLv2"
 inherit module
 
 SRC_URI = "http://dominion.thruhere.net/koen/OE/omap3-sgx-modules-1.3.13.1397.tar.bz2"
+SRC_URI[md5sum] = "1b48bbb5df6b4c12ebb42b92ae1bf1f9"
+SRC_URI[sha256sum] = "908a69340b020f7d865acf70724e37837016a81a0cb499fb83441759cbef7616"
 
 S = ${WORKDIR}/${BPN}-${PV}/eurasiacon/build/linux/omap3430_linux/kbuild/
 
-- 
1.7.7.3

[PATCH 2/4] puzzles: set SRC_URI checksums

[Joshua Lock]

In order to set missing checksums as an error we need to ensure
all appropriate recipes have a checksum value set.

Signed-off-by: Joshua Lock <josh@linux.intel.com>
---
 meta/recipes-sato/puzzles/puzzles_r9306.bb |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-sato/puzzles/puzzles_r9306.bb b/meta/recipes-sato/puzzles/puzzles_r9306.bb
index d9b8e3e..14b0461 100644
--- a/meta/recipes-sato/puzzles/puzzles_r9306.bb
+++ b/meta/recipes-sato/puzzles/puzzles_r9306.bb
@@ -8,12 +8,12 @@ MOD_PV = "${@d.getVar('PV',1)[1:]}"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://LICENCE;md5=9928b60f3b78be315b7ab699c1b03ff5"
 
-SRC_URI = "http://www.chiark.greenend.org.uk/~sgtatham/puzzles/puzzles-${PV}.tar.gz"
-
 # Upstream updates puzzles.tar.gz for the new release, so checksums seem to be changing regularly right now
-#SRC_URI[md5sum] = "e8b4c7e2341264789f3cd6733857bf1c"
-#SRC_URI[sha256sum] = "69f37e497e63b2f7ab44e7bbd50aee078269c16dbc2d78ec1b00257678f2cbd2"
 #SRC_URI = "svn://ixion.tartarus.org/main;module=puzzles;rev=${MOD_PV}"
+SRC_URI = "http://www.chiark.greenend.org.uk/~sgtatham/puzzles/puzzles-${PV}.tar.gz"
+SRC_URI[md5sum] = "fd1b05c0e0cfb752c6136ce2e4562416"
+SRC_URI[sha256sum] = "512aa26a0d11867a815030449bb903861f75512725886a70eece387bd8240ddd"
+
 
 S = "${WORKDIR}/${BPN}-${PV}"
 
-- 
1.7.7.3

[PATCH 3/4] default-distrovars: missing checksums should raise an error

[Joshua Lock]

Set BB_STRICT_CHECKSUM in default-distrovars so that an error is raised
if no checksum is set.

Signed-off-by: Joshua Lock <josh@linux.intel.com>
---
 meta/conf/distro/include/default-distrovars.inc |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/meta/conf/distro/include/default-distrovars.inc b/meta/conf/distro/include/default-distrovars.inc
index 6f5f1c0..e1594f3 100644
--- a/meta/conf/distro/include/default-distrovars.inc
+++ b/meta/conf/distro/include/default-distrovars.inc
@@ -48,3 +48,6 @@ NO32LIBS ??= "1"
 BBINCLUDELOGS ??= "yes"
 SDK_VERSION ??= "oe-core.0"
 DISTRO_VERSION ??= "oe-core.0"
+
+# Missing checksums should raise an error
+BB_STRICT_CHECKSUM = "1"
-- 
1.7.7.3

[PATCH 4/4] clutter-gst: upgrade to 1.4.4

[Joshua Lock]

Signed-off-by: Joshua Lock <josh@linux.intel.com>
---
 ...r-gst-1.8_1.4.2.bb => clutter-gst-1.8_1.4.4.bb} |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-graphics/clutter/{clutter-gst-1.8_1.4.2.bb => clutter-gst-1.8_1.4.4.bb} (80%)

diff --git a/meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.2.bb b/meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.4.bb
similarity index 80%
rename from meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.2.bb
rename to meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.4.bb
index 54b1fc1..a0616c7 100644
--- a/meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.2.bb
+++ b/meta/recipes-graphics/clutter/clutter-gst-1.8_1.4.4.bb
@@ -13,8 +13,8 @@ SRC_URI = "http://source.clutter-project.org/sources/clutter-gst/1.4/clutter-gst
 
 S = "${WORKDIR}/clutter-gst-${PV}"
 
-SRC_URI[md5sum] = "fd6b2a54f43d04382748e9e4d8a335c5"
-SRC_URI[sha256sum] = "b0e7ff76ee14307b6d26856972ec9e718f62aec2bc8807fb3f2960e508e578b5"
+SRC_URI[md5sum] = "c660649907785ee96f6e84d97f19d574"
+SRC_URI[sha256sum] = "23cb10e7629696a696ca8619614c517ff0a341e89a701075f2a123a2e26999e9"
 
 do_configure_prepend () {
        # Disable DOLT
-- 
1.7.7.3

Re: [PATCH 1/4] omap3-sgx-modules: set SRC_URI checksums

[Koen Kooi]

[-- Attachment #1: Type: text/plain, Size: 400 bytes --]


Op 6 dec. 2011, om 03:17 heeft Joshua Lock het volgende geschreven:

> We'd like to make missing SRC_URI checksums an error, so all
> appropriate recipes need to have a checksum value set.
> 
> Signed-off-by: Joshua Lock <josh@linux.intel.com>

NAK, this recipe needs to get deleted from OE, it belongs in a BSP. FWIW, meta-ti already has a more up to date version of it.

regards,

Koe 

[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 169 bytes --]

Re: [PATCH 0/4] Make missing checksums an error v2 + clutter-gst update

[Richard Purdie]

On Mon, 2011-12-05 at 18:17 -0800, Joshua Lock wrote:
> NOTE: this requires a patch I sent to the BitBake list[1] to error cleanly,
> otherwise you'll see a Python backtrace and the build fail...
> 
> This is v2 of a series to add set BB_STRICT_CHECKSUM in default-distrovars.inc
> so that missing checksums in recipes raises an error.
> 
> Since v1 I've run a test over OE-core to ensure to ensure I can do world
> fetches for each of the qemu machines and discovered that omap3-sgx-modules
> and puzzles recipes needed checksums adding.
> 
> I've snuck an extra patch lying on my disk into the queue too, an update for
> clutter-gst.
> 
> Regards,
> Joshua
> 
> 
> The following changes since commit 044324465bd54d53ae768f3c1e7468ae0e0c6200:
> 
>   dpkg-native: Fix perl path (2011-12-02 15:31:03 +0000)
> 
> are available in the git repository at:
>   git://git.openembedded.org/openembedded-core-contrib josh/checksums
>   http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=josh/checksums
> 
> Joshua Lock (4):
>   omap3-sgx-modules: set SRC_URI checksums
>   puzzles: set SRC_URI checksums
>   default-distrovars: missing checksums should raise an error
>   clutter-gst: upgrade to 1.4.4

Merged to master apart from the omap3 patch which was superseded,
thanks.

Richard