[PATCH 0/1] Minor fix for SANITY_TESTED_DISTROS feature

[PATCH 0/1] Minor fix for SANITY_TESTED_DISTROS feature

[Paul Eggleton]

The following changes since commit bf9e9961ec4e7b2d10f25b550b902df62b3939b1:

  Fix powerpc term and provide more template for different arches (2011-08-31 20:10:14 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib paule/tested-distro-fix
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/tested-distro-fix

Paul Eggleton (1):
  sanity.bbclass: fix splitting on newlines of SANITY_TESTED_DISTROS

 meta/classes/sanity.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
1.7.4.1

[PATCH 1/1] sanity.bbclass: fix splitting on newlines of SANITY_TESTED_DISTROS

[Paul Eggleton]

This variable should be split with \n sequences and these need to be
specified literally in the string. A corrected version of the example
given in the original commit (OE-core rev
75e3875341ddc8940e9ee2ccbbb2ec18194a68e6):

SANITY_TESTED_DISTROS = " \
        Ubuntu 11.04 \n \
        Fedora release 14 (Laughlin) \n \
        "

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta/classes/sanity.bbclass |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 37b9028..44b0688 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -149,7 +149,7 @@ def check_supported_distro(e):
                 finally:
                     f.close()
     if distro:
-        if distro not in [x.strip() for x in tested_distros.split('\n')]:
+        if distro not in [x.strip() for x in tested_distros.split('\\n')]:
             bb.warn('Host distribution "%s" has not been validated with this version of the build system; you may possibly experience unexpected failures. It is recommended that you use a tested distribution.' % distro)
     else:
         bb.warn('Host distribution could not be determined; you may possibly experience unexpected failures. It is recommended that you use a tested distribution.')
-- 
1.7.4.1

Re: [PATCH 1/1] sanity.bbclass: fix splitting on newlines of SANITY_TESTED_DISTROS

[Richard Purdie]

On Thu, 2011-09-01 at 12:36 +0100, Paul Eggleton wrote:
> This variable should be split with \n sequences and these need to be
> specified literally in the string. A corrected version of the example
> given in the original commit (OE-core rev
> 75e3875341ddc8940e9ee2ccbbb2ec18194a68e6):
> 
> SANITY_TESTED_DISTROS = " \
>         Ubuntu 11.04 \n \
>         Fedora release 14 (Laughlin) \n \
>         "
> 
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
>  meta/classes/sanity.bbclass |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Merged to master, thanks.

Richard