Skip to contentSkip to conversation

libiio: test programs require threads

PATCH New Baruch Siach 3 participants 3 messages Aug 11, 2015 – Aug 18, 2015

lore
Share & tools
lore

Baruch Siach

author
#0 lore raw

PATCH 1/1 libiio: test programs require threads

Fixes:
http://autobuild.buildroot.net/results/228/228a32167d05337acf17b0e3689aa91e8a67d8fb/
http://autobuild.buildroot.net/results/86a/86adb968659fec56684ba80e1c4230230fd7a1c8/

Signed-off-by Baruch Siach <[email protected]>
---
 package/libiio/Config.in | 4 ++++
 1 file changed, 4 insertions(+)
package/libiio/Config.in +4 -0
index 21496ed7c501..99da4c1c1000 100644
@@ -37,9 +37,13 @@ config BR2_PACKAGE_LIBIIO_TESTS
 	bool "Install test programs"
 	select BR2_PACKAGE_LIBIIO_LOCAL_BACKEND
 	select BR2_PACKAGE_LIBIIO_NETWORK_BACKEND
+	depends on BR2_TOOLCHAIN_HAS_THREADS
 	help
 	  Install the test programs (iio_info, iio_genxml, iio_readdev).
 
+comment "IIO test programs need a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
+
 config BR2_PACKAGE_LIBIIO_BINDINGS_CSHARP
 	bool "Install C# bindings"
 	depends on BR2_PACKAGE_MONO
Signature · 1 line
2.5.0
#1 lore raw

PATCH 1/1 libiio: test programs require threads

Baruch, All,

On 2015-08-11 07:54 +0300, Baruch Siach spake thusly:

Quoted 14 lines

Fixes:
http://autobuild.buildroot.net/results/228/228a32167d05337acf17b0e3689aa91e8a67d8fb/
http://autobuild.buildroot.net/results/86a/86adb968659fec56684ba80e1c4230230fd7a1c8/

Signed-off-by Baruch Siach <[email protected]>
---
 package/libiio/Config.in | 4 ++++
 1 file changed, 4 insertions(+)
package/libiio/Config.in +0 -0
index 21496ed7c501..99da4c1c1000 100644

package/libiio/Config.in config BR2_PACKAGE_LIBIIO_TESTS · line 40

Quoted 4 lines
package/libiio/Config.in +1 -0
@@ -37,9 +37,13 @@ config BR2_PACKAGE_LIBIIO_TESTS
 	bool "Install test programs"
 	select BR2_PACKAGE_LIBIIO_LOCAL_BACKEND
 	select BR2_PACKAGE_LIBIIO_NETWORK_BACKEND
+	depends on BR2_TOOLCHAIN_HAS_THREADS

Although I prefer we list dependencies before we do selects:

Acked-by "Yann E. MORIN" <[email protected]>

Regards,
Yann E. MORIN.

Quoted 16 lines
package/libiio/Config.in +3 -0
 	help
 	  Install the test programs (iio_info, iio_genxml, iio_readdev).
 
+comment "IIO test programs need a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
+
 config BR2_PACKAGE_LIBIIO_BINDINGS_CSHARP
 	bool "Install C# bindings"
 	depends on BR2_PACKAGE_MONO
Signature · 6 lines
2.5.0

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
Applied
#2 lore raw

PATCH 1/1 libiio: test programs require threads

In reply to Baruch Siach

Dear Baruch Siach,

Quoted 8 lines · Baruch Siach

Fixes:
http://autobuild.buildroot.net/results/228/228a32167d05337acf17b0e3689aa91e8a67d8fb/
http://autobuild.buildroot.net/results/86a/86adb968659fec56684ba80e1c4230230fd7a1c8/

Signed-off-by Baruch Siach <[email protected]>
---
 package/libiio/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

Applied, thanks.

Thomas

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