Skip to contentSkip to conversation

package/imlib2: fix musl compilation issue

PATCH New Brendan Heading 2 participants 2 messages Aug 3, 2015 – Aug 6, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/imlib2: fix musl compilation issue

Fixes http://autobuild.buildroot.net/results/a6e/a6e61bfb376946b479085f243601d6e1634adc6d/

musl is strict about including time.h when using time_t.

---
 .../0007-fix-compilation-issues-with-musl.patch    | 31 ++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 package/imlib2/0007-fix-compilation-issues-with-musl.patch
package/imlib2/0007-fix-compilation-issues-with-musl.patch new file +31 -0
new file mode 100644
index 0000000..50e88a2
@@ -0,0 +1,31 @@
+From bdfa1169c549122a8dc848b84469458101adeb20 Mon Sep 17 00:00:00 2001
+From: Brendan Heading <[email protected]>
+Date: Mon, 3 Aug 2015 12:44:55 +0100
+Subject: [PATCH 1/1] fix compilation issues with musl
+
+Using time_t strictly requires time.h to be included.
+
+NOTE I've stuck to the convention on this project where most of the
+files appear to pick up most of their headers by including "common.h".
+
+Upstream-status: pending
+Signed-off-by: Brendan Heading <[email protected]>
+---
+ src/lib/common.h | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/src/lib/common.h b/src/lib/common.h
+index 798965f..9053826 100644
+--- a/src/lib/common.h
++++ b/src/lib/common.h
+@@ -9,6 +9,7 @@
+ #include <config.h>
+ #include <string.h>
+ #include <math.h>
++#include <time.h>
+ #ifdef WITH_DMALLOC
+ #include <dmalloc.h>
+ #endif
+-- 
+2.4.3
+
Signature · 1 line
2.4.3
#1 lore raw

PATCH 1/1 package/imlib2: fix musl compilation issue

Dear Brendan Heading,

Quoted 9 lines · Brendan Heading

Fixes http://autobuild.buildroot.net/results/a6e/a6e61bfb376946b479085f243601d6e1634adc6d/

musl is strict about including time.h when using time_t.

---
 .../0007-fix-compilation-issues-with-musl.patch    | 31 ++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 package/imlib2/0007-fix-compilation-issues-with-musl.patch

Applied, thanks.

Thomas

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