* [Qemu-devel] [PATCH] configure: remove ${config_host_ld} variable
@ 2013-06-06 12:53 Ed Maste
2013-06-07 22:33 ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev
0 siblings, 1 reply; 2+ messages in thread
From: Ed Maste @ 2013-06-06 12:53 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Ed Maste
It was only used in one place (and already expanded in one other).
Signed-off-by: Ed Maste <emaste@freebsd.org>
---
configure | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/configure b/configure
index 1654413..e197b91 100755
--- a/configure
+++ b/configure
@@ -3556,7 +3556,6 @@ echo "-> Your SDL version is too old - please upgrade to have SDL support"
fi
config_host_mak="config-host.mak"
-config_host_ld="config-host.ld"
echo "# Automatically generated by configure - do not modify" >config-all-disas.mak
@@ -4071,7 +4070,7 @@ if test "$gcov" = "yes" ; then
fi
# generate list of library paths for linker script
-$ld --verbose -v 2> /dev/null | grep SEARCH_DIR > ${config_host_ld}
+$ld --verbose -v 2> /dev/null | grep SEARCH_DIR > config-host.ld
# use included Linux headers
if test "$linux" = "yes" ; then
--
1.7.11.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH] configure: remove ${config_host_ld} variable
2013-06-06 12:53 [Qemu-devel] [PATCH] configure: remove ${config_host_ld} variable Ed Maste
@ 2013-06-07 22:33 ` Michael Tokarev
0 siblings, 0 replies; 2+ messages in thread
From: Michael Tokarev @ 2013-06-07 22:33 UTC (permalink / raw)
To: Ed Maste; +Cc: qemu-trivial, qemu-devel
06.06.2013 16:53, Ed Maste wrote:
> It was only used in one place (and already expanded in one other).
Thanks, applied to the trivial patches queue.
I think we should just as well get rid of ${config_host_mak},
as it makes no sense too... ;) And while at it, maybe use
common constructs like
{ echo a; echo b; echo c; ... } > config-host.mak
But that's for another time :)
/mjt
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-07 22:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-06 12:53 [Qemu-devel] [PATCH] configure: remove ${config_host_ld} variable Ed Maste
2013-06-07 22:33 ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).