Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/wipe: fix musl build
@ 2016-01-31 20:44 Bernd Kuhls
  2016-02-02 13:10 ` Arnout Vandecappelle
  2016-02-07 14:00 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2016-01-31 20:44 UTC (permalink / raw)
  To: buildroot

The build error was not yet found by the autobuilders:

In file included from main.c:46:0:
rand.h:31:9: error: unknown type name ?u_int32_t?
 typedef u_int32_t u_rand_t;
         ^
<builtin>: recipe for target 'main.o' failed

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/wipe/0001-musl.patch | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)
 create mode 100644 package/wipe/0001-musl.patch

diff --git a/package/wipe/0001-musl.patch b/package/wipe/0001-musl.patch
new file mode 100644
index 0000000..35b3af8
--- /dev/null
+++ b/package/wipe/0001-musl.patch
@@ -0,0 +1,28 @@
+Fix musl build
+
+In file included from main.c:46:0:
+rand.h:31:9: error: unknown type name ?u_int32_t?
+ typedef u_int32_t u_rand_t;
+         ^
+<builtin>: recipe for target 'main.o' failed
+
+Patch sent upstream: https://sourceforge.net/p/wipe/patches/4/
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+
+diff -uNr wipe-2.3.1.org/rand.h wipe-2.3.1/rand.h
+--- wipe-2.3.1.org/rand.h	2003-12-03 04:01:02.000000000 +0100
++++ wipe-2.3.1/rand.h	2016-01-31 21:39:54.000000000 +0100
+@@ -22,10 +22,9 @@
+ 
+ #ifdef HAVE_STDINT_H
+ # include "stdint.h"
+-#else
+-# ifndef LINUX
++#endif
++#if !defined(u_int32_t)
+ #  define u_int32_t uint32_t
+-# endif
+ #endif
+ 
+ typedef u_int32_t u_rand_t;
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/1] package/wipe: fix musl build
  2016-01-31 20:44 [Buildroot] [PATCH 1/1] package/wipe: fix musl build Bernd Kuhls
@ 2016-02-02 13:10 ` Arnout Vandecappelle
  2016-02-07 14:00 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2016-02-02 13:10 UTC (permalink / raw)
  To: buildroot

On 31-01-16 21:44, Bernd Kuhls wrote:
> The build error was not yet found by the autobuilders:
> 
> In file included from main.c:46:0:
> rand.h:31:9: error: unknown type name ?u_int32_t?
>  typedef u_int32_t u_rand_t;
>          ^
> <builtin>: recipe for target 'main.o' failed
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/wipe/0001-musl.patch | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>  create mode 100644 package/wipe/0001-musl.patch
> 
> diff --git a/package/wipe/0001-musl.patch b/package/wipe/0001-musl.patch
> new file mode 100644
> index 0000000..35b3af8
> --- /dev/null
> +++ b/package/wipe/0001-musl.patch
> @@ -0,0 +1,28 @@
> +Fix musl build
> +
> +In file included from main.c:46:0:
> +rand.h:31:9: error: unknown type name ?u_int32_t?
> + typedef u_int32_t u_rand_t;
> +         ^
> +<builtin>: recipe for target 'main.o' failed
> +
> +Patch sent upstream: https://sourceforge.net/p/wipe/patches/4/
> +
> +Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> +
> +diff -uNr wipe-2.3.1.org/rand.h wipe-2.3.1/rand.h
> +--- wipe-2.3.1.org/rand.h	2003-12-03 04:01:02.000000000 +0100
> ++++ wipe-2.3.1/rand.h	2016-01-31 21:39:54.000000000 +0100
> +@@ -22,10 +22,9 @@
> + 
> + #ifdef HAVE_STDINT_H
> + # include "stdint.h"
> +-#else
> +-# ifndef LINUX
> ++#endif
> ++#if !defined(u_int32_t)
> + #  define u_int32_t uint32_t
> +-# endif
> + #endif

 Wouldn't it be better to use uint32_t everywhere?

 However, since the project hasn't had any activity since 2009, I don't think we
can expect much from upstream, so this quick-and-dirty patch should be OK.

Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>


 Regards,
 Arnout

> + 
> + typedef u_int32_t u_rand_t;
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/1] package/wipe: fix musl build
  2016-01-31 20:44 [Buildroot] [PATCH 1/1] package/wipe: fix musl build Bernd Kuhls
  2016-02-02 13:10 ` Arnout Vandecappelle
@ 2016-02-07 14:00 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2016-02-07 14:00 UTC (permalink / raw)
  To: buildroot

Dear Bernd Kuhls,

On Sun, 31 Jan 2016 21:44:14 +0100, Bernd Kuhls wrote:
> The build error was not yet found by the autobuilders:
> 
> In file included from main.c:46:0:
> rand.h:31:9: error: unknown type name ?u_int32_t?
>  typedef u_int32_t u_rand_t;
>          ^
> <builtin>: recipe for target 'main.o' failed
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/wipe/0001-musl.patch | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)
>  create mode 100644 package/wipe/0001-musl.patch

Applied, thanks.

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-02-07 14:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-31 20:44 [Buildroot] [PATCH 1/1] package/wipe: fix musl build Bernd Kuhls
2016-02-02 13:10 ` Arnout Vandecappelle
2016-02-07 14:00 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox