diff options
| author | Josua Mayer <josua.mayer97@gmail.com> | 2014-02-14 16:02:24 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-17 15:27:55 +0000 |
| commit | 39737e00e85bd4a6053f63f0c959fd424aa009be (patch) | |
| tree | d6ed0332826b35fb10d3132c94a3bcda1aa8782a /meta/classes | |
| parent | fd39da0a6134c4e14776428f7fd6d712a7ed96ce (diff) | |
| download | openembedded-core-39737e00e85bd4a6053f63f0c959fd424aa009be.tar.gz openembedded-core-39737e00e85bd4a6053f63f0c959fd424aa009be.tar.bz2 openembedded-core-39737e00e85bd4a6053f63f0c959fd424aa009be.zip | |
package.bbclass: do_split_packages should always return something.
So far, when do_split_packages is passed a non-existing folder, it will return nothing.
While building Mesa with PACKAGECONFIG="egl gles" for qemux86, do_package threw a nice exception on a line reading "pkgs += do_split_packages":
"Exception: TypeError: 'NoneType' object is not iterable"
I think a function should always return something, and in this specific case, returning an empty list should be right.
This patch simply fixes the return statement to do just that.
Signed-off-by: Josua Mayer <josua.mayer97@gmail.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Diffstat (limited to 'meta/classes')
| -rw-r--r-- | meta/classes/package.bbclass | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index 042d74c17b..297d962266 100644 --- a/meta/classes/package.bbclass +++ b/meta/classes/package.bbclass @@ -126,7 +126,7 @@ def do_split_packages(d, root, file_regex, output_pattern, description, postinst # If the root directory doesn't exist, don't error out later but silently do # no splitting. if not os.path.exists(dvar + root): - return + return [] ml = d.getVar("MLPREFIX", True) if ml: |