Re: [PATCH 1/2] ti-rpmsg-char-examples: Fix cross compile

Suman Anna

Hi Denys,

On 10/29/20 3:09 PM, Khem Raj wrote:
On Thu, Oct 29, 2020 at 11:09 AM Denys Dmytriyenko <denys@...> wrote:

On Mon, Oct 26, 2020 at 10:43:32PM -0400, Denys Dmytriyenko via wrote:
On Mon, Oct 26, 2020 at 04:24:11PM -0700, Khem Raj wrote:
I think it should be applied but then I don't know how else this
component is used. maybe there are some toolchains which don't have
sysrooots used to compile this
This is meant to be cross-compiled for the target, in which case accessing
host headers or libraries is a bad idea regardless of the toolchain...
Khem, Suman,

Any conclusion here? Should this change be taken upstream by Suman? Or should
this reside here as a patch with "Upstream-Status: Inappropriate [OE-specific]"?
its needed in OE for sure. Other places I don't know so literally your call.
If it is needed by OE, I am ok with this going in for now. I will leave it to
your expertise for the ack for this patch.

I need to look into this a bit more to see if I need to revise the build stuff
in the actual source repo, or I need to tweak the OE recipe to keep both happy.
I also cross-compile on my host machine when directly using automake, and as
such I don't use --prefix as /usr.

I need to have a successful OE environment for me to evaluate a proper fix, but
I am currently running into various issues when building tisdk-default-image
(tisdk-tiny-image and tisdk-base-image were ok).



On Mon, Oct 26, 2020 at 2:20 PM Denys Dmytriyenko <denys@...> wrote:

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 <raj.khem@...>
...-Remove-use-of-includedir-and-libdir.patch | 37 +++++++++++++++++++
recipes-ti/ipc/ | 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 <raj.khem@...>
+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 <raj.khem@...>
+ examples/ | 5 +----
+ 1 file changed, 1 insertion(+), 4 deletions(-)
+diff --git a/examples/ b/examples/
+index 5a9bedb..d547e41 100644
+--- a/examples/
++++ b/examples/
+@@ -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}
diff --git a/recipes-ti/ipc/ b/recipes-ti/ipc/
index 142e5bf0..cbe4d7fd 100644
--- a/recipes-ti/ipc/
+++ b/recipes-ti/ipc/
@@ -2,6 +2,8 @@ SUMMARY = "TI RPMsg Char Sample Applications and Tests"


+SRC_URI += "file://0001-examples-Remove-use-of-includedir-and-libdir.patch"
DEPENDS = "ti-rpmsg-char"

inherit autotools pkgconfig

Join to automatically receive all group messages.