Hmm, interesting. It doesn't actually appear to be using LDFLAGS from the environment at all just now. I'll take another look. Thanks.<br><div class="gmail_quote"><div dir="ltr">On Wed, May 11, 2016 at 9:33 AM Mark Hatle <<a href="mailto:mark.hatle@windriver.com">mark.hatle@windriver.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 5/11/16 11:25 AM, Christopher Larson wrote:<br>
> From: Christopher Larson <<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>><br>
><br>
> Signed-off-by: Christopher Larson <<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>><br>
> ---<br>
> .../pseudo/pseudo/obey-ldflags.patch | 48 ++++++++++++++++++++++<br>
> meta/recipes-devtools/pseudo/<a href="http://pseudo_1.7.5.bb" rel="noreferrer" target="_blank">pseudo_1.7.5.bb</a> | 16 ++++----<br>
> 2 files changed, 56 insertions(+), 8 deletions(-)<br>
> create mode 100644 meta/recipes-devtools/pseudo/pseudo/obey-ldflags.patch<br>
><br>
> diff --git a/meta/recipes-devtools/pseudo/pseudo/obey-ldflags.patch b/meta/recipes-devtools/pseudo/pseudo/obey-ldflags.patch<br>
> new file mode 100644<br>
> index 0000000..7e4c629<br>
> --- /dev/null<br>
> +++ b/meta/recipes-devtools/pseudo/pseudo/obey-ldflags.patch<br>
> @@ -0,0 +1,48 @@<br>
> +From 0ace81a687355a3c55caa161b51972a82f5c413f Mon Sep 17 00:00:00 2001<br>
> +From: Christopher Larson <<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>><br>
> +Date: Mon, 9 May 2016 17:00:57 -0700<br>
> +Subject: [PATCH] Obey external LDFLAGS the way we obey CFLAGS<br>
> +<br>
> +Signed-off-by: Christopher Larson <<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>><br>
> +---<br>
> + Makefile.in | 8 ++++----<br>
> + 1 file changed, 4 insertions(+), 4 deletions(-)<br>
> +<br>
> +diff --git a/Makefile.in b/Makefile.in<br>
> +index 6511814..22ef625 100644<br>
> +--- a/Makefile.in<br>
> ++++ b/Makefile.in<br>
> +@@ -109,26 +109,26 @@ pseudo: $(PSEUDO)<br>
> + $(PSEUDO): $(BIN) pseudo.o $(SHOBJS) $(DBOBJS) pseudo_client.o pseudo_server.o pseudo_ipc.o<br>
> + $(CC) $(CFLAGS) $(CFLAGS_PSEUDO) -o $(PSEUDO) \<br>
> + pseudo.o pseudo_server.o pseudo_client.o pseudo_ipc.o \<br>
> +- $(DBOBJS) $(SHOBJS) $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
> ++ $(DBOBJS) $(SHOBJS) $(LDFLAGS) $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
<br>
I may be wrong, but I thought the 'client_ldflags' was supposed to inherit the<br>
LDFLAGS, allowing them to be sanitized.<br>
<br>
The issue was that pseudo was in a strange situation where it may have to build<br>
both 32-bit and 64-bit clients in order to function properly.<br>
<br>
(Perhaps my memory is out of date and it's no longer a concern?)<br>
<br>
--Mark<br>
<br>
> +<br>
> + pseudolog: $(PSEUDOLOG)<br>
> +<br>
> + $(PSEUDOLOG): $(BIN) pseudolog.o $(SHOBJS) $(DBOBJS) pseudo_client.o pseudo_ipc.o<br>
> + $(CC) $(CFLAGS) $(CFLAGS_PSEUDO) -o $(PSEUDOLOG) pseudolog.o pseudo_client.o pseudo_ipc.o \<br>
> +- $(DBOBJS) $(SHOBJS) $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
> ++ $(DBOBJS) $(SHOBJS) $(LDFLAGS) $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
> +<br>
> + pseudodb: $(PSEUDODB)<br>
> +<br>
> + $(PSEUDODB): $(BIN) pseudodb.o $(SHOBJS) $(DBOBJS) pseudo_ipc.o<br>
> + $(CC) $(CFLAGS) $(CFLAGS_PSEUDO) -o $(PSEUDODB) pseudodb.o \<br>
> +- $(DBOBJS) $(SHOBJS) pseudo_ipc.o $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
> ++ $(DBOBJS) $(SHOBJS) pseudo_ipc.o $(LDFLAGS) $(DB_LDFLAGS) $(CLIENT_LDFLAGS)<br>
> +<br>
> + libpseudo: $(LIBPSEUDO)<br>
> +<br>
> + $(LIBPSEUDO): $(LIB) $(WRAPOBJS) pseudo_client.o pseudo_ipc.o $(SHOBJS)<br>
> + $(CC) $(CFLAGS) $(CFLAGS_PSEUDO) -shared -o $(LIBPSEUDO) \<br>
> + pseudo_client.o pseudo_ipc.o \<br>
> +- $(WRAPOBJS) $(SHOBJS) $(CLIENT_LDFLAGS)<br>
> ++ $(WRAPOBJS) $(SHOBJS) $(LDFLAGS) $(CLIENT_LDFLAGS)<br>
> +<br>
> + # *everything* now relies on stuff that's generated in the<br>
> + # wrapper process.<br>
> +--<br>
> +2.8.0<br>
> +<br>
> diff --git a/meta/recipes-devtools/pseudo/<a href="http://pseudo_1.7.5.bb" rel="noreferrer" target="_blank">pseudo_1.7.5.bb</a> b/meta/recipes-devtools/pseudo/<a href="http://pseudo_1.7.5.bb" rel="noreferrer" target="_blank">pseudo_1.7.5.bb</a><br>
> index d90e644..dfb9b39 100644<br>
> --- a/meta/recipes-devtools/pseudo/<a href="http://pseudo_1.7.5.bb" rel="noreferrer" target="_blank">pseudo_1.7.5.bb</a><br>
> +++ b/meta/recipes-devtools/pseudo/<a href="http://pseudo_1.7.5.bb" rel="noreferrer" target="_blank">pseudo_1.7.5.bb</a><br>
> @@ -1,13 +1,13 @@<br>
> require pseudo.inc<br>
><br>
> -SRC_URI = " \<br>
> - <a href="http://downloads.yoctoproject.org/releases/pseudo/$%7BBPN%7D-$%7BPV%7D.tar.bz2" rel="noreferrer" target="_blank">http://downloads.yoctoproject.org/releases/pseudo/${BPN}-${PV}.tar.bz2</a> \<br>
> - file://0001-configure-Prune-PIE-flags.patch \<br>
> - file://fallback-passwd \<br>
> - file://fallback-group \<br>
> - file://moreretries.patch \<br>
> - file://handle-remove-xattr.patch \<br>
> -"<br>
> +SRC_URI = "<a href="http://downloads.yoctoproject.org/releases/pseudo/$%7BBPN%7D-$%7BPV%7D.tar.bz2" rel="noreferrer" target="_blank">http://downloads.yoctoproject.org/releases/pseudo/${BPN}-${PV}.tar.bz2</a> \<br>
> + file://0001-configure-Prune-PIE-flags.patch \<br>
> + file://fallback-passwd \<br>
> + file://fallback-group \<br>
> + file://moreretries.patch \<br>
> + file://handle-remove-xattr.patch \<br>
> + file://obey-ldflags.patch \<br>
> + "<br>
><br>
> SRC_URI[md5sum] = "c10209938f03128d0c193f041ff3596d"<br>
> SRC_URI[sha256sum] = "fd89cadec984d3b8202aca465898b1bb4350e0d63ba9aa9ac899f6f50270e688"<br>
><br>
<br>
</blockquote></div>