[google/4.6] Backport r184357 from trunk to fix validate_failures.py

[dougkwan, 2012-03-02 02:57:33]

Hi Diego,

    This is a backport from trunk to fix a problem in validate_failures.py
when testing a cross compiler.

-Doug

2012-03-01   Doug Kwan  <dougkwan@google.com>

	Backport r184357 from trunk
	2012-02-17   Doug Kwan  <dougkwan@google.com>

		* contrib/testsuite-management/validate_failures.py
		(GetMakefileValue): Check for cross compilers.

Index: contrib/testsuite-management/validate_failures.py
===================================================================
--- contrib/testsuite-management/validate_failures.py	(revision 184772)
+++ contrib/testsuite-management/validate_failures.py	(working copy)
@@ -146,7 +146,8 @@ def GetMakefileValue(makefile_name, value_name):
 def ValidBuildDirectory(builddir, target):
   if (not os.path.exists(builddir) or
       not os.path.exists('%s/Makefile' % builddir) or
-      not os.path.exists('%s/build-%s' % (builddir, target))):
+      (not os.path.exists('%s/build-%s' % (builddir, target)) and
+       not os.path.exists('%s/%s' % (builddir, target)))):
     return False
   return True
 

--
This patch is available for review at http://codereview.appspot.com/5720047

[dougkwan, 2012-03-02 03:00:23]

If approved, this will be applied after Diego's 4.6.3 release merge
into Google's gcc-4_6 branch.

-Doug

On Thu, Mar 1, 2012 at 6:57 PM, Doug Kwan <dougkwan@google.com> wrote:
> Hi Diego,
>
>    This is a backport from trunk to fix a problem in validate_failures.py
> when testing a cross compiler.
>
> -Doug
>
> 2012-03-01   Doug Kwan  <dougkwan@google.com>
>
>        Backport r184357 from trunk
>        2012-02-17   Doug Kwan  <dougkwan@google.com>
>
>                * contrib/testsuite-management/validate_failures.py
>                (GetMakefileValue): Check for cross compilers.
>
> Index: contrib/testsuite-management/validate_failures.py
> ===================================================================
> --- contrib/testsuite-management/validate_failures.py   (revision 184772)
> +++ contrib/testsuite-management/validate_failures.py   (working copy)
> @@ -146,7 +146,8 @@ def GetMakefileValue(makefile_name, value_name):
>  def ValidBuildDirectory(builddir, target):
>   if (not os.path.exists(builddir) or
>       not os.path.exists('%s/Makefile' % builddir) or
> -      not os.path.exists('%s/build-%s' % (builddir, target))):
> +      (not os.path.exists('%s/build-%s' % (builddir, target)) and
> +       not os.path.exists('%s/%s' % (builddir, target)))):
>     return False
>   return True
>
>
> --
> This patch is available for review at http://codereview.appspot.com/5720047

[Diego Novillo, 2012-03-02 13:31:19]

On 01/03/12 21:57 , Doug Kwan wrote:
> Hi Diego,
>
>      This is a backport from trunk to fix a problem in validate_failures.py
> when testing a cross compiler.
>
> -Doug
>
> 2012-03-01   Doug Kwan<dougkwan@google.com>
>
> 	Backport r184357 from trunk
> 	2012-02-17   Doug Kwan<dougkwan@google.com>
>
> 		* contrib/testsuite-management/validate_failures.py
> 		(GetMakefileValue): Check for cross compilers.

OK, thanks.

I'll ping you once the merge is committed.


Diego.