Re: [PATCH] lib/bb/fetch2: don't skip checksums when local mirror enabled

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Thu, 2012-04-05 at 18:07 +0100, Paul Eggleton wrote:
> If you have PREMIRRORS set up to point to a local directory, the local
> download function does nothing and always succeeds and the mirror code
> was writing a .done stamp regardless, thus the checksum verification was
> being skipped even if the file wasn't actually copied from the mirror.
> 
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
>  lib/bb/fetch2/__init__.py |    7 ++++---
>  1 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/bb/fetch2/__init__.py b/lib/bb/fetch2/__init__.py
> index e5c4b34..414cc2b 100644
> --- a/lib/bb/fetch2/__init__.py
> +++ b/lib/bb/fetch2/__init__.py
> @@ -476,9 +476,10 @@ def try_mirrors(d, origud, mirrors, check = False):
>  
>              if not os.path.exists(ud.donestamp) or ud.method.need_update(newuri, ud, ld):
>                  ud.method.download(newuri, ud, ld)
> -                open(ud.donestamp, 'w').close()
> -                if hasattr(ud.method,"build_mirror_data"):
> -                    ud.method.build_mirror_data(newuri, ud, ld)
> +                if os.path.exists(ud.localpath):
> +                    open(ud.donestamp, 'w').close()
> +                    if hasattr(ud.method,"build_mirror_data"):
> +                        ud.method.build_mirror_data(newuri, ud, ld)
>  
>              if not ud.localpath or not os.path.exists(ud.localpath):
>                  continue

I messed up some commands and this ended up merged with the following
commit I manipulated:

http://git.openembedded.org/bitbake/commit/?id=c9eed04c6275ef2c694f89e047f85c7de76f89b6

Since its pushed at this point I'm not sure what options I have to undo
it. Sorry :(

Richard