Re: [PATCH 0/1] SRC_URI checksum support v3

[Joshua Lock <josh@openedhand.com>]

On Tue, 2010-12-21 at 13:32 +0800, Yu Ke wrote:
> On Dec 20, 16:10, Richard Purdie wrote:
> > On Fri, 2010-12-17 at 14:33 +0800, Yu Ke wrote:
> > > This patch add SRC_URI checksum support. With this patch,fetcher
> > > can verify the MD5 and SHA256 checksum of download src with the
> > > value defined in recipes SRC_URI.
> > > 
> > > This is the v3 patch with following changes compared with v2:
> > > - add configurable variable BB_STRICT_CHECKSUM to handle checksum missing case
> > >   if checksum is missing and BB_STRICT_CHECKSUM = "1", bitbake will fatal
> > > - add check to only verify checksum for protocol http/https/ftp/ftps, not
> > >   verify checksum for local file and other SCM
> > > 
> > > Pull URL: git://git.pokylinux.org/poky-contrib.git
> > >   Branch: kyu3/srcuri-v3
> > >   Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=kyu3/srcuri-v3
> > 
> > I merged this, then realised there was a problem with subsequent builds
> > proceeding with corrupt files. I've pushed a fix for at least part of
> > that.
> 
> Thanks, it is indeed necessary fix.
> 
> BTW, I am also thinking if we could merge Fetch.write_md5sum and
> Fetch.verify_md5sum into verify_checksum, since they are all checksum
> related, puting them together would be more clean logically.

That makes the assumption that you only want to verify a checksum when
fetching one, which I don't think is correct?

Cheers,
Joshua
-- 
Joshua Lock
        Intel Open Source Technology Centre