powerpc: Fix POWER9 implies

Message ID 1474058893-18161-1-git-send-email-tuliom@linux.vnet.ibm.com
State Committed
Headers

Commit Message

Tulio Magno Quites Machado Filho Sept. 16, 2016, 8:48 p.m. UTC
  Fix multiarch build for POWER9 by correcting the order of the
directories listed at sysnames configure variable.

2016-09-16  Tulio Magno Quites Machado Filho  <tuliom@linux.vnet.ibm.com>

	[BZ #20615]
	* sysdeps/powerpc/powerpc32/power9/multiarch/Implies: Replace
	fpu directory by non-fpu.
	* sysdeps/powerpc/powerpc64/power9/fpu/Implies: Remove dependency
	on non-fpu directory from a fpu directory.
---
 sysdeps/powerpc/powerpc32/power9/multiarch/Implies | 2 +-
 sysdeps/powerpc/powerpc64/power9/fpu/Implies       | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)
  

Comments

Carlos Eduardo Seo Sept. 16, 2016, 9:05 p.m. UTC | #1
On 9/16/16 5:48 PM, Tulio Magno Quites Machado Filho wrote:
> Fix multiarch build for POWER9 by correcting the order of the
> directories listed at sysnames configure variable.
>

Oops... oversight by my part when I did it. LGTM.
  
Paul E. Murphy Sept. 16, 2016, 9:10 p.m. UTC | #2
On 09/16/2016 03:48 PM, Tulio Magno Quites Machado Filho wrote:
> Fix multiarch build for POWER9 by correcting the order of the
> directories listed at sysnames configure variable.
> 
> 2016-09-16  Tulio Magno Quites Machado Filho  <tuliom@linux.vnet.ibm.com>
> 
> 	[BZ #20615]
> 	* sysdeps/powerpc/powerpc32/power9/multiarch/Implies: Replace
> 	fpu directory by non-fpu.
> 	* sysdeps/powerpc/powerpc64/power9/fpu/Implies: Remove dependency
> 	on non-fpu directory from a fpu directory.
> ---
>  sysdeps/powerpc/powerpc32/power9/multiarch/Implies | 2 +-
>  sysdeps/powerpc/powerpc64/power9/fpu/Implies       | 1 -
>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/sysdeps/powerpc/powerpc32/power9/multiarch/Implies b/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
> index 4393b56..1a46ef0 100644
> --- a/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
> +++ b/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
> @@ -1 +1 @@
> -powerpc/powerpc32/power8/fpu/multiarch
> +powerpc/powerpc32/power8/multiarch
> diff --git a/sysdeps/powerpc/powerpc64/power9/fpu/Implies b/sysdeps/powerpc/powerpc64/power9/fpu/Implies
> index fad2505..ae0dbaf 100644
> --- a/sysdeps/powerpc/powerpc64/power9/fpu/Implies
> +++ b/sysdeps/powerpc/powerpc64/power9/fpu/Implies
> @@ -1,2 +1 @@
>  powerpc/powerpc64/power8/fpu
> -powerpc/powerpc64/power8
> 

LGTM. I assume tested with an without multiarch enabled?
  
Tulio Magno Quites Machado Filho Sept. 19, 2016, 12:37 p.m. UTC | #3
"Paul E. Murphy" <murphyp@linux.vnet.ibm.com> writes:

> On 09/16/2016 03:48 PM, Tulio Magno Quites Machado Filho wrote:
>> Fix multiarch build for POWER9 by correcting the order of the
>> directories listed at sysnames configure variable.
>
> LGTM. I assume tested with an without multiarch enabled?

Yes.
Pushed as 1850ce5.
  

Patch

diff --git a/sysdeps/powerpc/powerpc32/power9/multiarch/Implies b/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
index 4393b56..1a46ef0 100644
--- a/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
+++ b/sysdeps/powerpc/powerpc32/power9/multiarch/Implies
@@ -1 +1 @@ 
-powerpc/powerpc32/power8/fpu/multiarch
+powerpc/powerpc32/power8/multiarch
diff --git a/sysdeps/powerpc/powerpc64/power9/fpu/Implies b/sysdeps/powerpc/powerpc64/power9/fpu/Implies
index fad2505..ae0dbaf 100644
--- a/sysdeps/powerpc/powerpc64/power9/fpu/Implies
+++ b/sysdeps/powerpc/powerpc64/power9/fpu/Implies
@@ -1,2 +1 @@ 
 powerpc/powerpc64/power8/fpu
-powerpc/powerpc64/power8