Skip to contentSkip to conversation

package/librtas: only show comment on supported architectures

PATCH New Yann E. MORIN 3 participants 3 messages Aug 19, 2015 – Aug 20, 2015

lore
Share & tools
lore

Yann E. MORIN

author
#0 lore raw

PATCH 1/1 package/librtas: only show comment on supported architectures

librtas is only available for PPC and PPC64, so only show the comment about the (e)glibc dependency for those archs.

Signed-off-by "Yann E. MORIN" <[email protected]>
Cc Sam Bobroff <[email protected]>
---
 package/librtas/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/librtas/Config.in +1 -0
index e437fa1..2e6063f 100644
@@ -10,4 +10,5 @@ config BR2_PACKAGE_LIBRTAS
 	  http://librtas.sourceforge.net/
 
 comment "librtas needs an (e)glibc toolchain"
+	depends on BR2_powerpc || BR2_powerpc64 || BR2_powerpc64le
 	depends on !BR2_TOOLCHAIN_USES_GLIBC
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 package/librtas: only show comment on supported architectures

Quoted 21 lines · Yann E. MORIN

librtas is only available for PPC and PPC64, so only show the comment about the (e)glibc dependency for those archs.

Signed-off-by "Yann E. MORIN" <[email protected]>
Cc Sam Bobroff <[email protected]>
---
 package/librtas/Config.in | 1 +
 1 file changed, 1 insertion(+)
package/librtas/Config.in +1 -0
index e437fa1..2e6063f 100644
@@ -10,4 +10,5 @@ config BR2_PACKAGE_LIBRTAS
 	  http://librtas.sourceforge.net/
 
 comment "librtas needs an (e)glibc toolchain"
+	depends on BR2_powerpc || BR2_powerpc64 || BR2_powerpc64le
 	depends on !BR2_TOOLCHAIN_USES_GLIBC
Signature · 1 line
1.9.1

Yann,

Looks good to me :-)

Reviewed-by Sam Bobroff <[email protected]>

Thanks,
Sam.

Applied
#2 lore raw

PATCH 1/1 package/librtas: only show comment on supported architectures

In reply to Yann E. MORIN

Dear Yann E. MORIN,

Quoted 8 lines · Yann E. MORIN

librtas is only available for PPC and PPC64, so only show the comment about the (e)glibc dependency for those archs.

Signed-off-by "Yann E. MORIN" <[email protected]>
Cc Sam Bobroff <[email protected]>
---
 package/librtas/Config.in | 1 +
 1 file changed, 1 insertion(+)

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com