* [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t
@ 2015-08-29 15:10 Gwenhael Goavec-Merou
2015-08-29 21:21 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Gwenhael Goavec-Merou @ 2015-08-29 15:10 UTC (permalink / raw)
To: buildroot
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
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>
--
2.4.6
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t
2015-08-29 15:10 [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t Gwenhael Goavec-Merou
@ 2015-08-29 21:21 ` Thomas Petazzoni
2015-09-02 7:02 ` gwenhael.goavec
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2015-08-29 21:21 UTC (permalink / raw)
To: buildroot
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
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t
2015-08-29 21:21 ` Thomas Petazzoni
@ 2015-09-02 7:02 ` gwenhael.goavec
2015-09-02 7:13 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: gwenhael.goavec @ 2015-09-02 7:02 UTC (permalink / raw)
To: buildroot
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t
2015-09-02 7:02 ` gwenhael.goavec
@ 2015-09-02 7:13 ` Thomas Petazzoni
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2015-09-02 7:13 UTC (permalink / raw)
To: buildroot
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
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-09-02 7:13 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-29 15:10 [Buildroot] [PATCH 1/1] gnuradio: sys/types.h is needed for mode_t Gwenhael Goavec-Merou
2015-08-29 21:21 ` Thomas Petazzoni
2015-09-02 7:02 ` gwenhael.goavec
2015-09-02 7:13 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox