On 20/02/13 18:19, Jeffrey Honig wrote:
Paul Eggleton <[email protected]> wrote:

On Wednesday 20 February 2013 10:38:29 Mark Hatle wrote:
On 2/13/13 4:44 AM, Jack Mitchell wrote:
On 12/02/13 22:22, Jeffrey C Honig wrote:
[ CQID: WIND00403568 ]

Hiawatha was overriding existing C flags instead of prepending warning
flags. This caused build failures on multilibs because the C compiler
could not find the proper sysroot.

Signed-off-by: Jeffrey C Honig <[email protected]>
---

    recipes-httpd/hiawatha/files/cmake.patch |   16 ++++++++++++++++
    recipes-httpd/hiawatha/hiawatha_8.6.bb   |    3 +++
    2 files changed, 19 insertions(+)
    create mode 100644 recipes-httpd/hiawatha/files/cmake.patch

diff --git a/recipes-httpd/hiawatha/files/cmake.patch
b/recipes-httpd/hiawatha/files/cmake.patch new file mode 100644
index 0000000..f597781
--- /dev/null
+++ b/recipes-httpd/hiawatha/files/cmake.patch
@@ -0,0 +1,16 @@
+It is necessary to include the original CMAKE_C_FLAGS to insure that
+the sysroot is passed to the compiler, instead of just setting warning
+options.  Otherwise we'll get problem croos compiling in some
+circumstances.
+
+--- a/CMakeLists.txt   2012-10-31 14:16:45.000000000 -0400
++++ b/CMakeLists.txt   2013-02-12 14:07:04.297058345 -0500
+@@ -2,7 +2,7 @@
+ project(Hiawatha C)
+
+ # Compiler
+-set(CMAKE_C_FLAGS "-Wall -Wextra")
++set(CMAKE_C_FLAGS "-Wall -Wextra ${CMAKE_C_FLAGS}")
+ set(CMAKE_BUILD_TYPE "RelWithDebInfo")
+
+ # Options
diff --git a/recipes-httpd/hiawatha/hiawatha_8.6.bb
b/recipes-httpd/hiawatha/hiawatha_8.6.bb index 837354b..d46a525 100644
--- a/recipes-httpd/hiawatha/hiawatha_8.6.bb
+++ b/recipes-httpd/hiawatha/hiawatha_8.6.bb
@@ -4,9 +4,12 @@ LICENSE = "GPLv2"

    LIC_FILES_CHKSUM =
    "file://LICENSE;md5=751419260aa954499f7abaabaa882bbe"
    DEPENDS = "libxml2 libxslt"

+PR = "r1"
+

    SECTION = "net"
SRC_URI = "http://hiawatha-webserver.org/files/${BPN}-${PV}.tar.gz \

+           file://cmake.patch \

               file://hiawatha-init"
SRC_URI[md5sum] = "372ea9e8329ed36e4fb781fdc1a6734c"
What's the upstream status on this? If you email Hugo (the developer)
directly he usually responds within a few hours and will pull the
changes into the next release.
I'm following up for Jeff.  He has contacted Hugo and the above patch has
been included upstream.
Great. Should I still apply this to meta-webserver then or do we wait for a
new upstream release?
We have a local patch applied, we are good.  It is fine to wait.

Thanks!

Jeff


8.8 has been released if someone wants to pick this up. I would myself but I'm currently blocked by a binutils failure.

--

  Jack Mitchell ([email protected])
  Embedded Systems Engineer
  http://www.embed.me.uk

--


_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to