From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49796) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkXUc-00060b-0z for qemu-devel@nongnu.org; Sun, 24 Nov 2013 06:04:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VkXUU-0006wE-Bt for qemu-devel@nongnu.org; Sun, 24 Nov 2013 06:04:45 -0500 Received: from [2a03:4000:1::4e2f:c7ac:d] (port=47957 helo=v220110690675601.yourvserver.net) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VkXUU-0006vZ-50 for qemu-devel@nongnu.org; Sun, 24 Nov 2013 06:04:38 -0500 Message-ID: <5291DD42.1050603@weilnetz.de> Date: Sun, 24 Nov 2013 12:04:34 +0100 From: Stefan Weil MIME-Version: 1.0 References: <20131124093704.GA26763@redhat.com> In-Reply-To: <20131124093704.GA26763@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] configure: re-add current options in config-host.mak List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" , qemu-devel@nongnu.org Cc: Peter Maydell , Fam Zheng , Anthony Liguori , Stefan Hajnoczi , Paolo Bonzini , Richard Henderson Am 24.11.2013 10:37, schrieb Michael S. Tsirkin: > commit bdf523e6923f1e667055e947e71fc600549cefae > "configure: Add config.status to recreate the current configuration" > moved current configure flags to config.status > > This makes sense, but this makes it harder to bisect problems that > trigger with a specific set of flags only, since old configure is not > smart enough to find them there. > > Simply add the options as a comment in the format > old configure expected - they are then ignored by the new configure. > > Cc: Stefan Weil > Cc: Fam Zheng > Signed-off-by: Michael S. Tsirkin > --- > configure | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/configure b/configure > index 508f6a5..f6d5ccd 100755 > --- a/configure > +++ b/configure > @@ -3797,6 +3797,7 @@ config_host_mak="config-host.mak" > echo "# Automatically generated by configure - do not modify" >config-all-disas.mak > > echo "# Automatically generated by configure - do not modify" > $config_host_mak > +sed -n -e 's/exec/# Configured with:/p' config.status >> $config_host_mak > echo >> $config_host_mak > > echo all: >> $config_host_mak Do we really need this? It only makes git bisect iterations from versions after 2013-11-19 to versions before that date easier. If a typical git bisect needs 10 steps, then in the worst cast 5 of them won't automatically run configure. This is normally not a big problem because there is already a configuration, and many changes of file configure don't change that configuration. It's always possible to run configure explicitly at each step of the bisection process, so in case of doubt there is an easy fall-back solution. Before I introduced automatic reconfiguration, calling configure + make was normal for git bisect. I'd prefer to avoid code like this patch which is only marginally useful in a very specific development use case. People who can run git bisect will be able to help themselves if they really get a problem without the patch. Regards, Stefan