[bitbake-devel] [PATCH v2] fetcher: fix BB_STRICT_CHECKSUM datatype check
Christopher Larson
clarson at kergoth.com
Wed Oct 8 16:13:12 UTC 2014
On Tue, Sep 30, 2014 at 9:21 AM, Alejandro Hernandez <
alejandro.hernandez at linux.intel.com> wrote:
> Forcing strict to be a string, to avoid problems when performing
> comparisons
>
> [YOCTO #6762]
>
> Signed-off-by: Alejandro Hernandez <alejandro.hernandez at linux.intel.com>
> ---
> lib/bb/fetch2/__init__.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/bb/fetch2/__init__.py b/lib/bb/fetch2/__init__.py
> index df2f2b0..10c3346 100644
> --- a/lib/bb/fetch2/__init__.py
> +++ b/lib/bb/fetch2/__init__.py
> @@ -543,8 +543,8 @@ def verify_checksum(ud, d):
>
> if ud.method.recommends_checksum(ud):
> # If strict checking enabled and neither sum defined, raise error
> - strict = d.getVar("BB_STRICT_CHECKSUM", True) or None
> - if strict and not (ud.md5_expected or ud.sha256_expected):
> + strict = d.getVar("BB_STRICT_CHECKSUM", "1") or "0"
> + if (strict == "1") and not (ud.md5_expected or
> ud.sha256_expected):
>
Why was True (second argument of getVar) changed to "1" here? It violates
our conventions and doesn't actually change anything in the behavior.
--
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/bitbake-devel/attachments/20141008/db25bf53/attachment-0002.html>
More information about the bitbake-devel
mailing list