From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fllv0015.ext.ti.com (fllv0015.ext.ti.com [198.47.19.141]) by mx.groups.io with SMTP id smtpd.web12.4427.1603747202931300725 for ; Mon, 26 Oct 2020 14:20:03 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@ti.com header.s=ti-com-17q1 header.b=GPTod7OA; spf=pass (domain: ti.com, ip: 198.47.19.141, mailfrom: denys@ti.com) Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 09QLK1s0105901; Mon, 26 Oct 2020 16:20:01 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1603747201; bh=AZB0BDkApOnZDgCdWXMZm0uCxCq/WLj2aPKfVV3X/LY=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=GPTod7OAObddaSiWym2Jd5TQxSZxeA0tLQkoUS9/hVk7GXuNCb+S8pxJWd3kcX+26 gX7eTWcHRqvyRnYV/jre76td4h2ZifbhVaKp9iLPX8plHqD0ZwyRYQkWIhCMlpPpnh 5a7mLBHIj83ehJf/y27X1LBD0awnJlsot3e6sKg8= Received: from DFLE105.ent.ti.com (dfle105.ent.ti.com [10.64.6.26]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 09QLK1GW116389 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 26 Oct 2020 16:20:01 -0500 Received: from DFLE101.ent.ti.com (10.64.6.22) by DFLE105.ent.ti.com (10.64.6.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Mon, 26 Oct 2020 16:20:01 -0500 Received: from fllv0039.itg.ti.com (10.64.41.19) by DFLE101.ent.ti.com (10.64.6.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Mon, 26 Oct 2020 16:20:01 -0500 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 09QLK0Te044378; Mon, 26 Oct 2020 16:20:01 -0500 Date: Mon, 26 Oct 2020 17:20:00 -0400 From: "Denys Dmytriyenko" To: Khem Raj CC: Subject: Re: [meta-ti] [PATCH 1/2] ti-rpmsg-char-examples: Fix cross compile Message-ID: <20201026212000.GO19741@beryl> References: <20201025065934.3308920-1-raj.khem@gmail.com> MIME-Version: 1.0 In-Reply-To: <20201025065934.3308920-1-raj.khem@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Sat, Oct 24, 2020 at 11:59:33PM -0700, Khem Raj wrote: > This fixes problems during cross compile where it starts to peek into > build host's include and library paths and causes errors with clang and > QA warnings with gcc > > Signed-off-by: Khem Raj > --- > ...-Remove-use-of-includedir-and-libdir.patch | 37 +++++++++++++++++++ > recipes-ti/ipc/ti-rpmsg-char-examples_git.bb | 2 + > 2 files changed, 39 insertions(+) > create mode 100644 recipes-ti/ipc/ti-rpmsg-char-examples/0001-examples-Remove-use-of-includedir-and-libdir.patch > > diff --git a/recipes-ti/ipc/ti-rpmsg-char-examples/0001-examples-Remove-use-of-includedir-and-libdir.patch b/recipes-ti/ipc/ti-rpmsg-char-examples/0001-examples-Remove-use-of-includedir-and-libdir.patch > new file mode 100644 > index 00000000..34807152 > --- /dev/null > +++ b/recipes-ti/ipc/ti-rpmsg-char-examples/0001-examples-Remove-use-of-includedir-and-libdir.patch > @@ -0,0 +1,37 @@ > +From 1a26cb577ba54dd9214eeb94fed0ce3a0f4603b6 Mon Sep 17 00:00:00 2001 > +From: Khem Raj > +Date: Sat, 24 Oct 2020 23:46:39 -0700 > +Subject: [PATCH] examples: Remove use of includedir and libdir > + > +These point to standard include and library paths which already are used > +from sysroot relative paths secondly, they use absolute paths which > +means we are asking to add -I/usr/include which might be ok in a native > +build but not in cross build. This helps mitigate QA errors found with > +OE build system since it finds it adding host include and library search > +paths to linker/compiler commandline What's the Upstream-Status on this? Should it be "Inappropriate [OE-specific]"? > +Signed-off-by: Khem Raj > +--- > + examples/Makefile.am | 5 +---- > + 1 file changed, 1 insertion(+), 4 deletions(-) > + > +diff --git a/examples/Makefile.am b/examples/Makefile.am > +index 5a9bedb..d547e41 100644 > +--- a/examples/Makefile.am > ++++ b/examples/Makefile.am > +@@ -1,6 +1,4 @@ > +-INCLUDE = -I${includedir} > +- > +-AM_CFLAGS = -Wall -g -O2 $(INCLUDE) > ++AM_CFLAGS = -Wall -g -O2 > + > + bin_PROGRAMS = rpmsg_char_simple > + > +@@ -8,4 +6,3 @@ bin_PROGRAMS = rpmsg_char_simple > + rpmsg_char_simple_SOURCES = rpmsg_char_simple.c > + rpmsg_char_simple_CPPFLAGS = $(AM_CFLAGS) > + rpmsg_char_simple_LDADD = -lpthread -lti_rpmsg_char > +-rpmsg_char_simple_LDFLAGS = -L${libdir} > +-- > +2.29.1 > + > diff --git a/recipes-ti/ipc/ti-rpmsg-char-examples_git.bb b/recipes-ti/ipc/ti-rpmsg-char-examples_git.bb > index 142e5bf0..cbe4d7fd 100644 > --- a/recipes-ti/ipc/ti-rpmsg-char-examples_git.bb > +++ b/recipes-ti/ipc/ti-rpmsg-char-examples_git.bb > @@ -2,6 +2,8 @@ SUMMARY = "TI RPMsg Char Sample Applications and Tests" > > require ti-rpmsg-char.inc > > +SRC_URI += "file://0001-examples-Remove-use-of-includedir-and-libdir.patch" > + > DEPENDS = "ti-rpmsg-char" > > inherit autotools pkgconfig > -- > 2.29.1 > > > >