All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-networking][PATCH] ntp: fix checksum
@ 2012-11-25 23:13 Eric Bénard
  2012-11-26 10:34 ` Paul Eggleton
  0 siblings, 1 reply; 7+ messages in thread
From: Eric Bénard @ 2012-11-25 23:13 UTC (permalink / raw)
  To: openembedded-devel

commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
a newer version but didn't update the checksum so fetch is failing.

Signed-off-by: Eric Bénard <eric@eukrea.com>
---
 meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
index f7c5b68..0fce5ce 100644
--- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
+++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
@@ -8,8 +8,8 @@ SRC_URI = "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
         file://ntpdate \
 "
 
-SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
-SRC_URI[sha256sum] = "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
+SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
+SRC_URI[sha256sum] = "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
 
 EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd" 
 
-- 
1.7.11.7




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [meta-networking][PATCH] ntp: fix checksum
  2012-11-25 23:13 Eric Bénard
@ 2012-11-26 10:34 ` Paul Eggleton
  2012-11-26 10:38   ` Eric Bénard
  0 siblings, 1 reply; 7+ messages in thread
From: Paul Eggleton @ 2012-11-26 10:34 UTC (permalink / raw)
  To: openembedded-devel

On Monday 26 November 2012 00:13:38 Eric Bénard wrote:
> commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
> a newer version but didn't update the checksum so fetch is failing.
> 
> Signed-off-by: Eric Bénard <eric@eukrea.com>
> ---
>  meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb index f7c5b68..0fce5ce
> 100644
> --- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> +++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> @@ -8,8 +8,8 @@ SRC_URI =
> "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
> file://ntpdate \
>  "
> 
> -SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
> -SRC_URI[sha256sum] =
> "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
> +SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
> +SRC_URI[sha256sum] =
> "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
> 
>  EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd"

Hmm, you're right - it did get missed. Strangely though if I do -c cleanall 
ntp then build ntp again I don't get an error with the wrong checksum (with 
danny) :/

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [meta-networking][PATCH] ntp: fix checksum
  2012-11-26 10:34 ` Paul Eggleton
@ 2012-11-26 10:38   ` Eric Bénard
  2012-11-26 10:47     ` Paul Eggleton
  0 siblings, 1 reply; 7+ messages in thread
From: Eric Bénard @ 2012-11-26 10:38 UTC (permalink / raw)
  To: Paul Eggleton; +Cc: openembedded-devel

Hi Paul,

Le Mon, 26 Nov 2012 10:34:23 +0000,
Paul Eggleton <paul.eggleton@linux.intel.com> a écrit :

> On Monday 26 November 2012 00:13:38 Eric Bénard wrote:
> > commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
> > a newer version but didn't update the checksum so fetch is failing.
> > 
> > Signed-off-by: Eric Bénard <eric@eukrea.com>
> > ---
> >  meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb index f7c5b68..0fce5ce
> > 100644
> > --- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > +++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > @@ -8,8 +8,8 @@ SRC_URI =
> > "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
> > file://ntpdate \
> >  "
> > 
> > -SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
> > -SRC_URI[sha256sum] =
> > "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
> > +SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
> > +SRC_URI[sha256sum] =
> > "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
> > 
> >  EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd"
> 
> Hmm, you're right - it did get missed. Strangely though if I do -c cleanall 
> ntp then build ntp again I don't get an error with the wrong checksum (with 
> danny) :/
> 
I also met this kind of behaviour but didn't look in depth at it : it
seems that once a package is fetched, further builds won't trigger
an error if the checksum in the recipe is wrong. 

Eric



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [meta-networking][PATCH] ntp: fix checksum
  2012-11-26 10:38   ` Eric Bénard
@ 2012-11-26 10:47     ` Paul Eggleton
  0 siblings, 0 replies; 7+ messages in thread
From: Paul Eggleton @ 2012-11-26 10:47 UTC (permalink / raw)
  To: Eric Bénard; +Cc: openembedded-devel

On Monday 26 November 2012 11:38:53 Eric Bénard wrote:
> Hi Paul,
> 
> Le Mon, 26 Nov 2012 10:34:23 +0000,
> 
> Paul Eggleton <paul.eggleton@linux.intel.com> a écrit :
> > On Monday 26 November 2012 00:13:38 Eric Bénard wrote:
> > > commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
> > > a newer version but didn't update the checksum so fetch is failing.
> > > 
> > > Signed-off-by: Eric Bénard <eric@eukrea.com>
> > > ---
> > > 
> > >  meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > > b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb index
> > > f7c5b68..0fce5ce
> > > 100644
> > > --- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > > +++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> > > @@ -8,8 +8,8 @@ SRC_URI =
> > > "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
> > > file://ntpdate \
> > > 
> > >  "
> > > 
> > > -SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
> > > -SRC_URI[sha256sum] =
> > > "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
> > > +SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
> > > +SRC_URI[sha256sum] =
> > > "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
> > > 
> > >  EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd"
> > 
> > Hmm, you're right - it did get missed. Strangely though if I do -c
> > cleanall
> > ntp then build ntp again I don't get an error with the wrong checksum
> > (with
> > danny) :/
> 
> I also met this kind of behaviour but didn't look in depth at it : it
> seems that once a package is fetched, further builds won't trigger
> an error if the checksum in the recipe is wrong.

The checksum is not verified if the .done file exists on the assumption that the 
fetch completed successfully and has already been verified; however when I did 
-c cleanall I checked - there was no .done file, so this is unexpected 
behaviour.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 7+ messages in thread

* [meta-networking][PATCH] ntp: fix checksum
@ 2012-11-27 16:19 Eric Bénard
  2012-11-27 19:40 ` Joe MacDonald
  0 siblings, 1 reply; 7+ messages in thread
From: Eric Bénard @ 2012-11-27 16:19 UTC (permalink / raw)
  To: openembedded-devel; +Cc: joe.macdonald

commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
a newer version but didn't update the checksum so fetch is failing.

Signed-off-by: Eric Bénard <eric@eukrea.com>
---
 meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
index f7c5b68..0fce5ce 100644
--- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
+++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
@@ -8,8 +8,8 @@ SRC_URI = "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
         file://ntpdate \
 "
 
-SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
-SRC_URI[sha256sum] = "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
+SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
+SRC_URI[sha256sum] = "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
 
 EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd" 
 
-- 
1.7.10.4




^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [meta-networking][PATCH] ntp: fix checksum
  2012-11-27 16:19 Eric Bénard
@ 2012-11-27 19:40 ` Joe MacDonald
  0 siblings, 0 replies; 7+ messages in thread
From: Joe MacDonald @ 2012-11-27 19:40 UTC (permalink / raw)
  To: Eric Bénard; +Cc: openembedded-devel

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

[[meta-networking][PATCH] ntp: fix checksum] On 12.11.27 (Tue 17:19) Eric Bénard wrote:

> commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
> a newer version but didn't update the checksum so fetch is failing.
> 
> Signed-off-by: Eric Bénard <eric@eukrea.com>
> ---
>  meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> index f7c5b68..0fce5ce 100644
> --- a/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> +++ b/meta-networking/recipes-support/ntp/ntp_4.2.6p5.bb
> @@ -8,8 +8,8 @@ SRC_URI = "http://www.eecis.udel.edu/~ntp/ntp_spool/ntp4/ntp-4.2/ntp-${PV}.tar.g
>          file://ntpdate \
>  "
>  
> -SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
> -SRC_URI[sha256sum] = "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
> +SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
> +SRC_URI[sha256sum] = "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
>  
>  EXTRA_OECONF += " --with-net-snmp-config=no --without-ntpsnmpd" 
>  

I didn't merge the patch since the change is included in the updates I
merged from PaulE.

-- 
-Joe MacDonald.
:wq

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [meta-networking][PATCH] ntp: fix checksum
@ 2012-12-01 16:14 Peter A. Bigot
  0 siblings, 0 replies; 7+ messages in thread
From: Peter A. Bigot @ 2012-12-01 16:14 UTC (permalink / raw)
  To: openembedded-devel

I don't know how to reply to mail that was sent while I was unsubscribed, so:

http://patches.openembedded.org/patch/39697

was rejected per below, but still needs to be applied to danny.

Peter

[[meta-networking][PATCH]*ntp*:*fix*  *checksum*] On 12.11.27 (Tue 17:19) Eric Bénard wrote:
> commit e15543f05bb2c321cd649b90de50d1a3d8cd1199 upgrade the recipe to
> a newer version but didn't update the*checksum*  so fetch is failing.
>
> Signed-off-by: Eric Bénard <eric <at> eukrea.com>
> ---
>  meta-networking/recipes-support/*ntp*/*ntp*_4.2.6p5.bb |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta-networking/recipes-support/*ntp*/*ntp*_4.2.6p5.bb b/meta-networking/recipes-support/*ntp*/*ntp*_4.2.6p5.bb
> index f7c5b68..0fce5ce 100644
> --- a/meta-networking/recipes-support/*ntp*/*ntp*_4.2.6p5.bb
> +++ b/meta-networking/recipes-support/*ntp*/*ntp*_4.2.6p5.bb
> @@ -8,8 +8,8 @@ SRC_URI = "http://www.eecis.udel.edu/~*ntp*/*ntp*_spool/*ntp*4/*ntp*-4.2/*ntp*-${PV}.tar.g  <http://www.eecis.udel.edu/%7Entp/ntp_spool/ntp4/ntp-4.2/ntp-$%7BPV%7D.tar.g>
>          file://*ntp*date \
>  "
>
> -SRC_URI[md5sum] = "59876a9009b098ff59767ee45a88ebd2"
> -SRC_URI[sha256sum] = "6e84d4ddfa14b911c3ed88463af10867e1fa9b287e7b34d8a02e78be85a7c40e"
> +SRC_URI[md5sum] = "00df80a84ec9528fcfb09498075525bc"
> +SRC_URI[sha256sum] = "d6ab8371f9d31e594eb6922823d5ccd03dcc4e9d84b0e23ea25ac1405432f91c"
>
>  EXTRA_OECONF += " --with-net-snmp-config=no --without-*ntp*snmpd"
>

I didn't merge the patch since the change is included in the updates I
merged from PaulE.

--
-Joe MacDonald.
:wq



^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2012-12-01 16:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-01 16:14 [meta-networking][PATCH] ntp: fix checksum Peter A. Bigot
  -- strict thread matches above, loose matches on Subject: below --
2012-11-27 16:19 Eric Bénard
2012-11-27 19:40 ` Joe MacDonald
2012-11-25 23:13 Eric Bénard
2012-11-26 10:34 ` Paul Eggleton
2012-11-26 10:38   ` Eric Bénard
2012-11-26 10:47     ` Paul Eggleton

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.