Skip to contentSkip to conversation

gnuradio: sys/types.h is needed for mode_t

PATCH New Gwenhael Goavec-Merou 2 participants 4 messages Aug 29, 2015 – Sep 2, 2015

lore
Share & tools
lore

Gwenhael Goavec-Merou

author
#0 lore raw

PATCH 1/1 gnuradio: sys/types.h is needed for mode_t

From: Gwenhael Goavec-Merou <[email protected]>

fix :
http://autobuild.buildroot.net/results/8e3/8e3afd5f22312f0660f631fdb82bbb64eda913fb/build-end.log

error: 'mode_t' has not been declared

Signed-off-by Gwenhael Goavec-Merou <[email protected]>
---
 .../gnuradio/0002-types_t-is-need-for-mode_t.patch    | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 package/gnuradio/0002-types_t-is-need-for-mode_t.patch
package/gnuradio/0002-types_t-is-need-for-mode_t.patch new file +19 -0
new file mode 100644
index 0000000..7730d2d
@@ -0,0 +1,19 @@
+To avoid :
+error: 'mode_t' has not been declared
+sys/types.h must be included 
+
+Signed-off-by: Gwenhael Goavec-Merou <[email protected]>
+---
+Index: gnuradio-3.7.5/gnuradio-runtime/include/gnuradio/logger.h.in
+===================================================================
+--- gnuradio-3.7.5.orig/gnuradio-runtime/include/gnuradio/logger.h.in
++++ gnuradio-3.7.5/gnuradio-runtime/include/gnuradio/logger.h.in
+@@ -44,6 +44,8 @@
+ 
+ #ifdef _MSC_VER
+ typedef unsigned short mode_t;
++#else
++#include <sys/types.h>
+ #endif
+ 
+ #include <gnuradio/api.h>
Signature · 1 line
2.4.6
Applied
#1 lore raw

PATCH 1/1 gnuradio: sys/types.h is needed for mode_t

Dear Gwenhael Goavec-Merou,

Quoted 5 lines · Gwenhael Goavec-Merou

From: Gwenhael Goavec-Merou <[email protected]>

fix :
http://autobuild.buildroot.net/results/8e3/8e3afd5f22312f0660f631fdb82bbb64eda913fb/build-end.log

Quoted 7 lines

error: 'mode_t' has not been declared

Signed-off-by Gwenhael Goavec-Merou <[email protected]>
---
 .../gnuradio/0002-types_t-is-need-for-mode_t.patch    | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 package/gnuradio/0002-types_t-is-need-for-mode_t.patch

Applied, thanks. As we discussed, please submit the patch upstream. Thanks!

Thomas

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

PATCH 1/1 gnuradio: sys/types.h is needed for mode_t

Dear Thomas Petazzoni

Quoted 17 lines · On Sat, 29 Aug 2015 23:21:14 +0200 Thomas Petazzoni…

Dear Gwenhael Goavec-Merou,

Quoted 5 lines · Gwenhael Goavec-Merou

From: Gwenhael Goavec-Merou <[email protected]>

fix :
http://autobuild.buildroot.net/results/8e3/8e3afd5f22312f0660f631fdb82bbb64eda913fb/build-end.log

Quoted 8 lines

error: 'mode_t' has not been declared

Signed-off-by: Gwenhael Goavec-Merou
<[email protected]> ---

 .../gnuradio/0002-types_t-is-need-for-mode_t.patch    | 19
+++++++++++++++++++ 1 file changed, 19 insertions(+)
 create mode 100644

package/gnuradio/0002-types_t-is-need-for-mode_t.patch

Quoted 3 lines

Applied, thanks. As we discussed, please submit the patch upstream. Thanks!

Done :
https://github.com/gnuradio/gnuradio/commit/3e2dc168467304212328abd061bf10f0c511f17f
Gwenhael

#3 lore raw

PATCH 1/1 gnuradio: sys/types.h is needed for mode_t

Hello,

Quoted 5 lines · gwenhael.goavec
Quoted 3 lines

Applied, thanks. As we discussed, please submit the patch upstream. Thanks!

Done :
https://github.com/gnuradio/gnuradio/commit/3e2dc168467304212328abd061bf10f0c511f17f

Excellent, thanks for following up on this!

Thomas

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