[OE-core] [PATCH 1/1] combo-layer: handle unset dest_dir in sanity_check()

[Paul Eggleton]

On Wednesday 20 May 2015 13:48:20 Patrick Ohly wrote:
> The previous "clean up dest_dir checking" patch (f8cdbe7497) improved
> handling of empty dest_dir but made handling of unset dest_dir worse:
> instead showing the "Option dest_dir is not defined for component ..."
> error, it fails with a Python exception.
> 
> Avoid that by providing a sane fallback for the unset case. With that
> change, dest_dir is no longer strictly required, but the check for it
> is kept to ensure that a combo-layer.conf also works with older
> combo-layer versions.
> 
> [Yocto #7773]
> 
> Signed-off-by: Patrick Ohly <patrick.ohly at intel.com>
> ---
>  scripts/combo-layer | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/combo-layer b/scripts/combo-layer
> index b0b7c28..698d3e3 100755
> --- a/scripts/combo-layer
> +++ b/scripts/combo-layer
> @@ -145,8 +145,10 @@ class Configuration(object):
>                      msg = "%s\nOption %s is not defined for component %s"
> %(msg, option, name) missing_options.append(option)
>              # Sanitize dest_dir so that we do not have to deal with edge
> cases -            # (empty string, double slashes) in the rest of the
> code. -            dest_dir =
> os.path.normpath(self.repos[name]["dest_dir"]) +            # (unset, empty
> string, double slashes) in the rest of the code. +            # It not
> being set will still be flagged as error because it is +            #
> listed as required option above; that could be changed now. +           
> dest_dir = os.path.normpath(self.repos[name].get("dest_dir", "."))
> self.repos[name]["dest_dir"] = "." if not dest_dir else dest_dir if msg !=
> "":
>              logger.error("configuration file %s has the following error:
> %s" % (self.conffile,msg))

I suppose we might now tidy up the line that follows, but that's less critical 
than fixing the behaviour that this patch does. Therefore:

Acked-by: Paul Eggleton <paul.eggleton at linux.intel.com>

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre