From patchwork Fri Jul 21 11:16:00 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Khem Raj X-Patchwork-Id: 108468 Delivered-To: patch@linaro.org Received: by 10.182.45.195 with SMTP id p3csp507032obm; Fri, 21 Jul 2017 04:16:15 -0700 (PDT) X-Received: by 10.98.63.204 with SMTP id z73mr7010668pfj.278.1500635775553; Fri, 21 Jul 2017 04:16:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1500635775; cv=none; d=google.com; s=arc-20160816; b=WgMr9S9fekC3IAZmP0KCqBcr4wbJQ/xIyBAc+GIiDW3dntguexb4Adg/tDi50DqUhT PSDSDA/3BGBs/fB7oeogT8OEQxA15N9/dd2f6Gpgmcpz/3Ljj/a3mdmTZ2UWFC7IV1pu v8WM7xagZV/+/O+LL4xBgLgtmX9fXi+Pr/PiNKqHlYwpYPvGqnJGE4+5xrgOnIR1DBJY CVXhnC8X5Af4t5Ky7Pzexpu2DsBAq8xUTFX9QYla2ApULEWc0RKc5l6Gq7ETd9W7XuJW hg2bs1G0iOgUx8pi20MmlJYTHzcifySdTqsiyeIWuSBFr+IYj4t+zxViQi4mSBmAnyCJ iReg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=errors-to:sender:content-transfer-encoding:mime-version :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:message-id:date:to:from:dkim-signature :delivered-to:arc-authentication-results; bh=DL+BDqLIWQ6gQYdhbsvJBuHiq2NFCQ4oiELUECvFxXg=; b=DaOtoMsLrruUj1YXJhtpxLa2MmMZNx++swnJeoTs7GggZ99j3FOOIIlMp2PDt3dacA ZvbWh7kU6IRDfcfG7tQyHcHndXnZRd/bMCB/TINWeYx6Fz/cBCwPlDIM3rlXq2lL7TjU B5bnAPKqkHAXKe4QWwSfmSY3p6kvOyYSpKnU9i9E3DmYYg5gNDvWv1Y1fXIqI8wrlg3N rEYEmFJKI95Ltkv4BC239srdhC4/f28tGlWPg7R1bEc2uOKY7URqInFFm33OvEj9E3HE 1V8pB2zaEgl6QTP0HpJEYkBJ3Grd5ULnmiTf/79B6mEhDCxRQI8fD4fiOZb/0UtZKpuv TrTA== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.b=MjNVckWM; spf=pass (google.com: best guess record for domain of openembedded-devel-bounces@lists.openembedded.org designates 140.211.169.62 as permitted sender) smtp.mailfrom=openembedded-devel-bounces@lists.openembedded.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmail.com Return-Path: Received: from mail.openembedded.org (mail.openembedded.org. [140.211.169.62]) by mx.google.com with ESMTP id y44si2931396plh.726.2017.07.21.04.16.15; Fri, 21 Jul 2017 04:16:15 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of openembedded-devel-bounces@lists.openembedded.org designates 140.211.169.62 as permitted sender) client-ip=140.211.169.62; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.b=MjNVckWM; spf=pass (google.com: best guess record for domain of openembedded-devel-bounces@lists.openembedded.org designates 140.211.169.62 as permitted sender) smtp.mailfrom=openembedded-devel-bounces@lists.openembedded.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=gmail.com Received: from review.yoctoproject.org (localhost [127.0.0.1]) by mail.openembedded.org (Postfix) with ESMTP id 71697783FF; Fri, 21 Jul 2017 11:16:12 +0000 (UTC) X-Original-To: openembedded-devel@lists.openembedded.org Delivered-To: openembedded-devel@lists.openembedded.org Received: from mail-pg0-f66.google.com (mail-pg0-f66.google.com [74.125.83.66]) by mail.openembedded.org (Postfix) with ESMTP id 8D2E6783E1 for ; Fri, 21 Jul 2017 11:16:11 +0000 (UTC) Received: by mail-pg0-f66.google.com with SMTP id d193so5172774pgc.2 for ; Fri, 21 Jul 2017 04:16:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id; bh=CJHxdEDvnTigaLOKYdmh7HOZ84VQtMx0WnAULO2D9kM=; b=MjNVckWMmuJvrwF6GMYlgCuRYfmSjW29VdNIiJbb+HuXj7LF3xV8Vla+djkuFfiLPo cnt8oskVnapX+lICqTFb+7DVqig6JP/H/fRoASdMtXYIJjrVeo+oNvNxpJdBO6KFa1FP 6vome+qVnvWuTMP7h5tBJAzwQE31nhHl6dLe2J2ys7xyHrZ/J0xSSU6BjL+UfkOFlDut AHp0wkUBxjWacA3cssaXOdI1L0cqixkbuP1wWm3c8DLJ0TXjBlpfWH8v97o4GUUAFacV WeyAYcO6fdjasFNcEPGA5DgbOcSDCXKb/JTlTXQtSaz9oh6ri9+jPFd9/eV85qAXDk0s BfMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=CJHxdEDvnTigaLOKYdmh7HOZ84VQtMx0WnAULO2D9kM=; b=nwO+/+clqPRKQBujgJ4bBFBHI5NUqXPy2tRwfmPbi9O2fMmRCWCWEWsr5XyCXChspG SPQD4eLSG13NsMhvA79jmxIpSeRiIrq6yBXJkDttcnKv2UXEdu1qcWLD8qjlhU1lT+yz 0w3a1GrBZO8yqmiriM0/77Yeot1rAIbyBAvYDO6fpIrUQA54zUrPXxaZF/oYD+C4WVlA nh7YG09d7/HdqpptokxneHOMKtw318mM1uNX9m8Oa/0Aw9OZ8geBOefkIQ3YIqOC2VaN x3zPltt6A79gajMWhLtnrnxFBm0xrRQtonU/cHOsE72vj6NhVLof/8vrozxqKhudEYXD p97Q== X-Gm-Message-State: AIVw1114GdTjU9eO3a8EvPaUcBdi8YWFnMWvM5XV+x6dx6tQnBJPxNsJ FlwI5agMgMFQdH4B X-Received: by 10.99.126.86 with SMTP id o22mr7084837pgn.385.1500635772466; Fri, 21 Jul 2017 04:16:12 -0700 (PDT) Received: from localhost.localdomain ([2601:646:8882:b8c:6245:cbff:fe64:591]) by smtp.gmail.com with ESMTPSA id b16sm9129223pfm.84.2017.07.21.04.16.11 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 21 Jul 2017 04:16:11 -0700 (PDT) From: Khem Raj To: openembedded-devel@lists.openembedded.org Date: Fri, 21 Jul 2017 04:16:00 -0700 Message-Id: <20170721111602.2547-1-raj.khem@gmail.com> X-Mailer: git-send-email 2.13.3 Subject: [oe] [meta-networking][PATCH 1/3] openflow: Fix build with musl X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: openembedded-devel-bounces@lists.openembedded.org Errors-To: openembedded-devel-bounces@lists.openembedded.org Regenerate configure before running oe_runconf Signed-off-by: Khem Raj --- .../recipes-protocols/openflow/openflow.inc | 3 +- ...use-strlcpy-from-libc-before-defining-own.patch | 64 ++++++++++++++++++++++ ...lib-netdev-Adjust-header-include-sequence.patch | 59 ++++++++++++++++++++ .../recipes-protocols/openflow/openflow_git.bb | 5 ++ 4 files changed, 129 insertions(+), 2 deletions(-) create mode 100644 meta-networking/recipes-protocols/openflow/openflow/0001-Check-and-use-strlcpy-from-libc-before-defining-own.patch create mode 100644 meta-networking/recipes-protocols/openflow/openflow/0002-lib-netdev-Adjust-header-include-sequence.patch -- 2.13.3 -- _______________________________________________ Openembedded-devel mailing list Openembedded-devel@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-devel diff --git a/meta-networking/recipes-protocols/openflow/openflow.inc b/meta-networking/recipes-protocols/openflow/openflow.inc index 0ae2d1f1d..cccbfa19a 100644 --- a/meta-networking/recipes-protocols/openflow/openflow.inc +++ b/meta-networking/recipes-protocols/openflow/openflow.inc @@ -27,9 +27,8 @@ S = "${WORKDIR}/git" inherit autotools-brokensep pkgconfig -do_configure() { +do_configure_prepend() { ./boot.sh - oe_runconf } do_install_append() { diff --git a/meta-networking/recipes-protocols/openflow/openflow/0001-Check-and-use-strlcpy-from-libc-before-defining-own.patch b/meta-networking/recipes-protocols/openflow/openflow/0001-Check-and-use-strlcpy-from-libc-before-defining-own.patch new file mode 100644 index 000000000..952274bb9 --- /dev/null +++ b/meta-networking/recipes-protocols/openflow/openflow/0001-Check-and-use-strlcpy-from-libc-before-defining-own.patch @@ -0,0 +1,64 @@ +From 7b62e5884353b247f542844d1e4687d0e9211999 Mon Sep 17 00:00:00 2001 +From: Khem Raj +Date: Thu, 20 Jul 2017 04:27:32 -0700 +Subject: [PATCH 1/2] Check and use strlcpy from libc before defining own + +This is required especially on musl where +function prototype conflicts and causes build +failures. + +Signed-off-by: Khem Raj +--- + configure.ac | 2 +- + lib/util.c | 2 ++ + lib/util.h | 1 + + 3 files changed, 4 insertions(+), 1 deletion(-) + +diff --git a/configure.ac b/configure.ac +index 13064f6..596c43f 100644 +--- a/configure.ac ++++ b/configure.ac +@@ -57,7 +57,7 @@ OFP_CHECK_HWTABLES + OFP_CHECK_HWLIBS + AC_SYS_LARGEFILE + +-AC_CHECK_FUNCS([strsignal]) ++AC_CHECK_FUNCS([strlcpy strsignal]) + + AC_ARG_VAR(KARCH, [Kernel Architecture String]) + AC_SUBST(KARCH) +diff --git a/lib/util.c b/lib/util.c +index 21cc28d..1f341b1 100644 +--- a/lib/util.c ++++ b/lib/util.c +@@ -138,6 +138,7 @@ xasprintf(const char *format, ...) + return s; + } + ++#ifndef HAVE_STRLCPY + void + strlcpy(char *dst, const char *src, size_t size) + { +@@ -148,6 +149,7 @@ strlcpy(char *dst, const char *src, size_t size) + dst[n_copy] = '\0'; + } + } ++#endif + + void + ofp_fatal(int err_no, const char *format, ...) +diff --git a/lib/util.h b/lib/util.h +index fde681f..9e45ea9 100644 +--- a/lib/util.h ++++ b/lib/util.h +@@ -41,6 +41,7 @@ + #include + #include + #include "compiler.h" ++#include "config.h" + + #ifndef va_copy + #ifdef __va_copy +-- +2.13.3 + diff --git a/meta-networking/recipes-protocols/openflow/openflow/0002-lib-netdev-Adjust-header-include-sequence.patch b/meta-networking/recipes-protocols/openflow/openflow/0002-lib-netdev-Adjust-header-include-sequence.patch new file mode 100644 index 000000000..75180fe32 --- /dev/null +++ b/meta-networking/recipes-protocols/openflow/openflow/0002-lib-netdev-Adjust-header-include-sequence.patch @@ -0,0 +1,59 @@ +From 5bba224edea38607e8732081f86679ffd8b218ab Mon Sep 17 00:00:00 2001 +From: Khem Raj +Date: Thu, 20 Jul 2017 04:29:04 -0700 +Subject: [PATCH 2/2] lib/netdev: Adjust header include sequence + +Specify libc headers before kernel UAPIs +this helps compiling with musl where otherwise +it uses the definition from kernel and complains +about double definition in libc headers + +Signed-off-by: Khem Raj +--- + lib/netdev.c | 11 +++++------ + 1 file changed, 5 insertions(+), 6 deletions(-) + +diff --git a/lib/netdev.c b/lib/netdev.c +index 3b6fbc5..c7de25e 100644 +--- a/lib/netdev.c ++++ b/lib/netdev.c +@@ -39,7 +39,6 @@ + #include + #include + #include +-#include + + /* Fix for some compile issues we were experiencing when setting up openwrt + * with the 2.4 kernel. linux/ethtool.h seems to use kernel-style inttypes, +@@ -57,10 +56,6 @@ + #define s64 __s64 + #endif + +-#include +-#include +-#include +-#include + #include + #include + #include +@@ -68,12 +63,16 @@ + #include + #include + #include +-#include + #include + #include + #include + #include + #include ++#include ++#include ++#include ++#include ++#include + + #include "fatal-signal.h" + #include "list.h" +-- +2.13.3 + diff --git a/meta-networking/recipes-protocols/openflow/openflow_git.bb b/meta-networking/recipes-protocols/openflow/openflow_git.bb index 6403bfb22..eceb45e94 100644 --- a/meta-networking/recipes-protocols/openflow/openflow_git.bb +++ b/meta-networking/recipes-protocols/openflow/openflow_git.bb @@ -2,3 +2,8 @@ include ${BPN}.inc SRCREV = "c84f33f09d5dbcfc9b489f64cb30475bf36f653a" PV = "1.0+git${SRCPV}" + +SRC_URI += "\ + file://0001-Check-and-use-strlcpy-from-libc-before-defining-own.patch \ + file://0002-lib-netdev-Adjust-header-include-sequence.patch \ + "