diff mbox

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

Message ID 1440861006-26198-1-git-send-email-gwenj@trabucayre.com
State Accepted
Headers show

Commit Message

Gwenhael Goavec-Merou Aug. 29, 2015, 3:10 p.m. UTC 
From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>

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

error: 'mode_t' has not been declared

Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
---
 .../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

Comments

Thomas Petazzoni Aug. 29, 2015, 9:21 p.m. UTC | #1 
Dear Gwenhael Goavec-Merou,

On Sat, 29 Aug 2015 17:10:06 +0200, Gwenhael Goavec-Merou wrote:
> From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> 
> fix :
> http://autobuild.buildroot.net/results/8e3/8e3afd5f22312f0660f631fdb82bbb64eda913fb/build-end.log
> 
> error: 'mode_t' has not been declared
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> ---
>  .../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
Gwenhael Goavec-Merou Sept. 2, 2015, 7:02 a.m. UTC | #2 
Dear Thomas Petazzoni
On Sat, 29 Aug 2015 23:21:14 +0200
Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Dear Gwenhael Goavec-Merou,
> 
> On Sat, 29 Aug 2015 17:10:06 +0200, Gwenhael Goavec-Merou wrote:
> > From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@trabucayre.com>
> > 
> > fix :
> > http://autobuild.buildroot.net/results/8e3/8e3afd5f22312f0660f631fdb82bbb64eda913fb/build-end.log
> > 
> > error: 'mode_t' has not been declared
> > 
> > Signed-off-by: Gwenhael Goavec-Merou
> > <gwenhael.goavec-merou@trabucayre.com> ---
> >  .../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!
> 
Done :
https://github.com/gnuradio/gnuradio/commit/3e2dc168467304212328abd061bf10f0c511f17f
Gwenhael
Thomas Petazzoni Sept. 2, 2015, 7:13 a.m. UTC | #3 
Hello,

On Wed, 2 Sep 2015 09:02:41 +0200, gwenhael.goavec wrote:

> > 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
diff mbox

Patch

diff --git a/package/gnuradio/0002-types_t-is-need-for-mode_t.patch b/package/gnuradio/0002-types_t-is-need-for-mode_t.patch
new file mode 100644
index 0000000..7730d2d
--- /dev/null
+++ b/package/gnuradio/0002-types_t-is-need-for-mode_t.patch
@@ -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 <gwenhael.goavec-merou@trabucayre.com>
+---
+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>