qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Michael Roth <mdroth@linux.vnet.ibm.com>
To: Steve Ellcey  <sellcey@imgtec.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Build problem with qemu Makefile change
Date: Mon, 23 Nov 2015 12:35:57 -0600	[thread overview]
Message-ID: <20151123183557.6284.25007@loki> (raw)
In-Reply-To: <efaa6a39-5ae7-4f0a-85c1-88957045be3e@BAMAIL02.ba.imgtec.org>

Quoting Steve Ellcey  (2015-11-23 12:06:57)
> My qemu build has been failing since this checkin by Michael Roth:
> 
>         http://lists.nongnu.org/archive/html/qemu-devel/2015-11/msg03991.html
> 
> which changed the top-level Makefile.  Is anyone else seeing this?
> 
> I am building a qemu for MIPS on an x86 linux box with:
> 
> configure --prefix=/scratch/sellcey/repos/q/install-mips-mti-linux-gnu --disable-tools --disable-system --disable-werror --target-list=mips-linux-user,mipsel-linux-user,mipsn32-linux-user,mipsn32el-linux-user,mips64-linux-user,mips64el-linux-user
> 
> make --jobs=3 all
> make --jobs=3 install
> 
> The error message is:
> 
> install -d -m 0755 "/scratch/sellcey/repos/q/install-mips-mti-linux-gnu/bin"
> libtool --quiet --mode=install install -c -m 0755  "/scratch/sellcey/repos/q/install-mips-mti-linux-gnu/bin"
> libtool: install: you must specify a destination
> libtool: install: Try `libtool --help --mode=install' for more information.
> make: *** [install] Error 1
> Error: Make command failed, stopping build.
> 
> 
> If I remove the Makefile patch then my build works and the libtool install
> line looks like:
> 
> install -d -m 0755 "/scratch/sellcey/repos/q/install-mips-mti-linux-gnu/bin"
> libtool --quiet --mode=install install -c -m 0755 qemu-ga  "/scratch/sellcey/repos/q2/install-mips-mti-linux-gnu/bin"
> 
> I think the failure may be related to my use of '--disable-tools' or
> '--disable-system' in the configure and how this interacts with the
> Makefile change.

Looks like --disable-tools results in qemu-ga being
the only entry in $TOOLS.

With the filter added in my patch, this results in install-prog
getting called with an empty list, which generates the error.

So now we need an extra check to avoid calling install-prog with an
empty list. I'll post a patch today. Thanks for the catch.

> 
> Steve Ellcey
> sellcey@imgtec.com
> 

  reply	other threads:[~2015-11-23 18:36 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-23 18:06 [Qemu-devel] Build problem with qemu Makefile change Steve Ellcey 
2015-11-23 18:35 ` Michael Roth [this message]
2015-11-23 22:09   ` Michael Roth
2015-11-23 22:28     ` Steve Ellcey

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151123183557.6284.25007@loki \
    --to=mdroth@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sellcey@imgtec.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).