* [mcs][PATCH] tgt: ensure build LDFLAGS apply to all executables
@ 2017-01-09 15:04 Mark Asselstine
2017-01-09 17:25 ` Bruce Ashfield
0 siblings, 1 reply; 2+ messages in thread
From: Mark Asselstine @ 2017-01-09 15:04 UTC (permalink / raw)
To: bruce.ashfield; +Cc: meta-virtualization
Some of the tgt executables were not being built with the gnu-hash
causing a QA error. There is no apparent reason to have differences in
how the tgt executables are built so ensure they all use the LDFLAGS.
Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
---
.../tgt/files/0001-usr-Makefile-WARNING-fix.patch | 2 +-
...Makefile-apply-LDFLAGS-to-all-executables.patch | 35 ++++++++++++++++++++++
meta-openstack/recipes-support/tgt/tgt_git.bb | 8 +++--
3 files changed, 41 insertions(+), 4 deletions(-)
create mode 100644 meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
diff --git a/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch b/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
index 15f5482..f749101 100644
--- a/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
+++ b/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
@@ -22,7 +22,7 @@ index 1fae7e7..2db109c 100644
TGTD_DEP = $(TGTD_OBJS:.o=.d)
-LDFLAGS = -Wl,-E,-rpath=$(libdir)
-+LDFLAGS = -Wl,-E
++LDFLAGS += -Wl,-E
.PHONY:all
all: $(PROGRAMS) $(MODULES)
diff --git a/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch b/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
new file mode 100644
index 0000000..d480ef7
--- /dev/null
+++ b/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
@@ -0,0 +1,35 @@
+From a815ac8ee16b344d9e24b445957f32bf2aafa532 Mon Sep 17 00:00:00 2001
+From: Mark Asselstine <mark.asselstine@windriver.com>
+Date: Thu, 5 Jan 2017 11:07:51 -0500
+Subject: [PATCH] usr/Makefile: apply LDFLAGS to all executables
+
+Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
+---
+ usr/Makefile | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/usr/Makefile b/usr/Makefile
+index cc8df11..c55fd68 100644
+--- a/usr/Makefile
++++ b/usr/Makefile
+@@ -78,7 +78,7 @@ TGTADM_OBJS = tgtadm.o concat_buf.o
+ TGTADM_DEP = $(TGTADM_OBJS:.o=.d)
+
+ tgtadm: $(TGTADM_OBJS)
+- $(CC) $^ -o $@
++ $(CC) $^ -o $@ $(LDFLAGS)
+
+ -include $(TGTADM_DEP)
+
+@@ -86,7 +86,7 @@ TGTIMG_OBJS = tgtimg.o libssc.o libcrc32c.o
+ TGTIMG_DEP = $(TGTIMG_OBJS:.o=.d)
+
+ tgtimg: $(TGTIMG_OBJS)
+- $(CC) $^ -o $@
++ $(CC) $^ -o $@ $(LDFLAGS)
+
+ -include $(TGTIMG_DEP)
+
+--
+2.7.4
+
diff --git a/meta-openstack/recipes-support/tgt/tgt_git.bb b/meta-openstack/recipes-support/tgt/tgt_git.bb
index 5c594b1..9ca181a 100644
--- a/meta-openstack/recipes-support/tgt/tgt_git.bb
+++ b/meta-openstack/recipes-support/tgt/tgt_git.bb
@@ -10,6 +10,7 @@ PV = "1.0.67+git${SRCPV}"
SRC_URI = "git://github.com/fujita/tgt.git \
file://0001-Correct-the-path-of-header-files-check-in-Yocto-buil.patch \
file://0001-usr-Makefile-WARNING-fix.patch \
+ file://usr-Makefile-apply-LDFLAGS-to-all-executables.patch \
"
SRC_URI += "file://tgtd.init"
@@ -21,9 +22,10 @@ inherit update-rc.d
CFLAGS += ' -I. -DUSE_SIGNALFD -DUSE_TIMERFD -D_GNU_SOURCE -DTGT_VERSION=\\"1.0.63\\" -DBSDIR=\\"${libdir}/backing-store\\"'
-do_compile() {
- oe_runmake SYSROOT="${STAGING_DIR_TARGET}" -e programs conf scripts
-}
+#do_compile() {
+# oe_runmake SYSROOT="${STAGING_DIR_TARGET}" -e programs conf scripts
+#}
+EXTRA_OEMAKE = "-e programs conf scripts"
do_install() {
oe_runmake -e DESTDIR="${D}" install-programs install-conf install-scripts
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [mcs][PATCH] tgt: ensure build LDFLAGS apply to all executables
2017-01-09 15:04 [mcs][PATCH] tgt: ensure build LDFLAGS apply to all executables Mark Asselstine
@ 2017-01-09 17:25 ` Bruce Ashfield
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Ashfield @ 2017-01-09 17:25 UTC (permalink / raw)
To: Mark Asselstine; +Cc: meta-virtualization
On 2017-01-09 10:04 AM, Mark Asselstine wrote:
> Some of the tgt executables were not being built with the gnu-hash
> causing a QA error. There is no apparent reason to have differences in
> how the tgt executables are built so ensure they all use the LDFLAGS.
merged.
Bruce
>
> Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
> ---
> .../tgt/files/0001-usr-Makefile-WARNING-fix.patch | 2 +-
> ...Makefile-apply-LDFLAGS-to-all-executables.patch | 35 ++++++++++++++++++++++
> meta-openstack/recipes-support/tgt/tgt_git.bb | 8 +++--
> 3 files changed, 41 insertions(+), 4 deletions(-)
> create mode 100644 meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
>
> diff --git a/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch b/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
> index 15f5482..f749101 100644
> --- a/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
> +++ b/meta-openstack/recipes-support/tgt/files/0001-usr-Makefile-WARNING-fix.patch
> @@ -22,7 +22,7 @@ index 1fae7e7..2db109c 100644
> TGTD_DEP = $(TGTD_OBJS:.o=.d)
>
> -LDFLAGS = -Wl,-E,-rpath=$(libdir)
> -+LDFLAGS = -Wl,-E
> ++LDFLAGS += -Wl,-E
>
> .PHONY:all
> all: $(PROGRAMS) $(MODULES)
> diff --git a/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch b/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
> new file mode 100644
> index 0000000..d480ef7
> --- /dev/null
> +++ b/meta-openstack/recipes-support/tgt/files/usr-Makefile-apply-LDFLAGS-to-all-executables.patch
> @@ -0,0 +1,35 @@
> +From a815ac8ee16b344d9e24b445957f32bf2aafa532 Mon Sep 17 00:00:00 2001
> +From: Mark Asselstine <mark.asselstine@windriver.com>
> +Date: Thu, 5 Jan 2017 11:07:51 -0500
> +Subject: [PATCH] usr/Makefile: apply LDFLAGS to all executables
> +
> +Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
> +---
> + usr/Makefile | 4 ++--
> + 1 file changed, 2 insertions(+), 2 deletions(-)
> +
> +diff --git a/usr/Makefile b/usr/Makefile
> +index cc8df11..c55fd68 100644
> +--- a/usr/Makefile
> ++++ b/usr/Makefile
> +@@ -78,7 +78,7 @@ TGTADM_OBJS = tgtadm.o concat_buf.o
> + TGTADM_DEP = $(TGTADM_OBJS:.o=.d)
> +
> + tgtadm: $(TGTADM_OBJS)
> +- $(CC) $^ -o $@
> ++ $(CC) $^ -o $@ $(LDFLAGS)
> +
> + -include $(TGTADM_DEP)
> +
> +@@ -86,7 +86,7 @@ TGTIMG_OBJS = tgtimg.o libssc.o libcrc32c.o
> + TGTIMG_DEP = $(TGTIMG_OBJS:.o=.d)
> +
> + tgtimg: $(TGTIMG_OBJS)
> +- $(CC) $^ -o $@
> ++ $(CC) $^ -o $@ $(LDFLAGS)
> +
> + -include $(TGTIMG_DEP)
> +
> +--
> +2.7.4
> +
> diff --git a/meta-openstack/recipes-support/tgt/tgt_git.bb b/meta-openstack/recipes-support/tgt/tgt_git.bb
> index 5c594b1..9ca181a 100644
> --- a/meta-openstack/recipes-support/tgt/tgt_git.bb
> +++ b/meta-openstack/recipes-support/tgt/tgt_git.bb
> @@ -10,6 +10,7 @@ PV = "1.0.67+git${SRCPV}"
> SRC_URI = "git://github.com/fujita/tgt.git \
> file://0001-Correct-the-path-of-header-files-check-in-Yocto-buil.patch \
> file://0001-usr-Makefile-WARNING-fix.patch \
> + file://usr-Makefile-apply-LDFLAGS-to-all-executables.patch \
> "
> SRC_URI += "file://tgtd.init"
>
> @@ -21,9 +22,10 @@ inherit update-rc.d
>
> CFLAGS += ' -I. -DUSE_SIGNALFD -DUSE_TIMERFD -D_GNU_SOURCE -DTGT_VERSION=\\"1.0.63\\" -DBSDIR=\\"${libdir}/backing-store\\"'
>
> -do_compile() {
> - oe_runmake SYSROOT="${STAGING_DIR_TARGET}" -e programs conf scripts
> -}
> +#do_compile() {
> +# oe_runmake SYSROOT="${STAGING_DIR_TARGET}" -e programs conf scripts
> +#}
> +EXTRA_OEMAKE = "-e programs conf scripts"
>
> do_install() {
> oe_runmake -e DESTDIR="${D}" install-programs install-conf install-scripts
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-01-09 17:25 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-09 15:04 [mcs][PATCH] tgt: ensure build LDFLAGS apply to all executables Mark Asselstine
2017-01-09 17:25 ` Bruce Ashfield
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.