Skip to contentSkip to conversation

package/libiio: tests need threads

PATCH New Yann E. MORIN 2 participants 3 messages Aug 13, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/libiio: tests need threads

Fixes:

    http://autobuild.buildroot.org/results/4bb/4bbd2d6927ca75517c2bd37eca83a9d98f1cca01/
    http://autobuild.buildroot.org/results/949/949314b57f5a396b4bb03e422094b754dcf87e20/
    ...
---
 package/libiio/Config.in | 4 ++++
 1 file changed, 4 insertions(+)
package/libiio/Config.in +4 -0
index 21496ed..c77583d 100644
@@ -35,11 +35,15 @@ comment "IIO Daemon needs a toolchain w/ threads"
 
 config BR2_PACKAGE_LIBIIO_TESTS
 	bool "Install test programs"
+	depends on BR2_TOOLCHAIN_HAS_THREADS
 	select BR2_PACKAGE_LIBIIO_LOCAL_BACKEND
 	select BR2_PACKAGE_LIBIIO_NETWORK_BACKEND
 	help
 	  Install the test programs (iio_info, iio_genxml, iio_readdev).
 
+comment "IIO tests needs 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
1.9.1
#1 lore raw

PATCH 1/1 package/libiio: tests need threads

Hi Yann,

Quoted 6 lines · Yann E. MORIN

Fixes:

    http://autobuild.buildroot.org/results/4bb/4bbd2d6927ca75517c2bd37eca83a9d98f1cca01/
    http://autobuild.buildroot.org/results/949/949314b57f5a396b4bb03e422094b754dcf87e20/
    ...

I sent an almost identical patch two days ago:
http://patchwork.ozlabs.org/patch/505866/.

Quoted 14 lines
---
 package/libiio/Config.in | 4 ++++
 1 file changed, 4 insertions(+)
package/libiio/Config.in +1 -0
index 21496ed..c77583d 100644
@@ -35,11 +35,15 @@ comment "IIO Daemon needs a toolchain w/ threads"
 
 config BR2_PACKAGE_LIBIIO_TESTS
 	bool "Install test programs"
+	depends on BR2_TOOLCHAIN_HAS_THREADS
 	select BR2_PACKAGE_LIBIIO_LOCAL_BACKEND

package/libiio/Config.in comment "IIO Daemon needs a toolchain w/ threads" · line 49

Quoted 10 lines
package/libiio/Config.in +3 -0
 	select BR2_PACKAGE_LIBIIO_NETWORK_BACKEND
 	help
 	  Install the test programs (iio_info, iio_genxml, iio_readdev).
 
+comment "IIO tests needs 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

baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
#2 lore raw

[Buildroot] (no subject)

Baruch, All,

On 2015-08-13 22:02 +0300, Baruch Siach spake thusly:

Quoted 10 lines · Yann E. MORIN
Quoted 6 lines

Fixes:

    http://autobuild.buildroot.org/results/4bb/4bbd2d6927ca75517c2bd37eca83a9d98f1cca01/
    http://autobuild.buildroot.org/results/949/949314b57f5a396b4bb03e422094b754dcf87e20/
    ...

I sent an almost identical patch two days ago:
http://patchwork.ozlabs.org/patch/505866/.

Ah, OK, I missed it...

I'll mark my patch as Rejected and review yours.

Thanks for the heads up! ;-)

Regards,
Yann E. MORIN.

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.  |
'------------------------------^-------^------------------^--------------------'