* Re: [Qemu-devel] [PATCH v2 1/1] configure: unset interfering variables [not found] ` <1401273557-6578-2-git-send-email-cornelia.huck@de.ibm.com> @ 2014-06-06 7:41 ` Cornelia Huck 2014-06-06 8:11 ` Paolo Bonzini 0 siblings, 1 reply; 3+ messages in thread From: Cornelia Huck @ 2014-06-06 7:41 UTC (permalink / raw) To: qemu-devel; +Cc: Peter Maydell, Markus Armbruster On Wed, 28 May 2014 12:39:17 +0200 Cornelia Huck <cornelia.huck@de.ibm.com> wrote: > The check for big or little endianness relies on grep reporting > match/non-match on the generated binary. If the user specified > --binary-files=without-match in their GREP_OPTIONS, this will fail. > > Let's follow what autoconf does and unset GREP_OPTIONS and CLICOLOR_FORCE > at the beginning of the script. > > Reported-by: Eugene (jno) Dvurechenski <jno@linux.vnet.ibm.com> > Suggested-by: Markus Armbruster <armbru@redhat.com> > Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> > --- > configure | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/configure b/configure > index 0e516f9..525da56 100755 > --- a/configure > +++ b/configure > @@ -3,6 +3,11 @@ > # qemu configure script (c) 2003 Fabrice Bellard > # > > +# Unset some variables known to interfere with behavior of common tools, > +# just as autoconf does. > +CLICOLOR_FORCE= GREP_OPTIONS= > +unset CLICOLOR_FORCE GREP_OPTIONS > + > # Temporary directory used for files created while > # configure runs. Since it is in the build directory > # we can safely blow away any previous version of it Is there an obvious tree to merge configure changes through, or should I just throw it into my next s390 pull request? ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/1] configure: unset interfering variables 2014-06-06 7:41 ` [Qemu-devel] [PATCH v2 1/1] configure: unset interfering variables Cornelia Huck @ 2014-06-06 8:11 ` Paolo Bonzini 2014-06-06 8:23 ` Cornelia Huck 0 siblings, 1 reply; 3+ messages in thread From: Paolo Bonzini @ 2014-06-06 8:11 UTC (permalink / raw) To: Cornelia Huck, qemu-devel; +Cc: Peter Maydell, Markus Armbruster Il 06/06/2014 09:41, Cornelia Huck ha scritto: > On Wed, 28 May 2014 12:39:17 +0200 > Cornelia Huck <cornelia.huck@de.ibm.com> wrote: > >> The check for big or little endianness relies on grep reporting >> match/non-match on the generated binary. If the user specified >> --binary-files=without-match in their GREP_OPTIONS, this will fail. >> >> Let's follow what autoconf does and unset GREP_OPTIONS and CLICOLOR_FORCE >> at the beginning of the script. >> >> Reported-by: Eugene (jno) Dvurechenski <jno@linux.vnet.ibm.com> >> Suggested-by: Markus Armbruster <armbru@redhat.com> >> Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> >> --- >> configure | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/configure b/configure >> index 0e516f9..525da56 100755 >> --- a/configure >> +++ b/configure >> @@ -3,6 +3,11 @@ >> # qemu configure script (c) 2003 Fabrice Bellard >> # >> >> +# Unset some variables known to interfere with behavior of common tools, >> +# just as autoconf does. >> +CLICOLOR_FORCE= GREP_OPTIONS= >> +unset CLICOLOR_FORCE GREP_OPTIONS >> + >> # Temporary directory used for files created while >> # configure runs. Since it is in the build directory >> # we can safely blow away any previous version of it > > Is there an obvious tree to merge configure changes through, or should > I just throw it into my next s390 pull request? Hi, I picked up this patch and will push it to a "configure" branch on github. Paolo ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/1] configure: unset interfering variables 2014-06-06 8:11 ` Paolo Bonzini @ 2014-06-06 8:23 ` Cornelia Huck 0 siblings, 0 replies; 3+ messages in thread From: Cornelia Huck @ 2014-06-06 8:23 UTC (permalink / raw) To: Paolo Bonzini; +Cc: Peter Maydell, qemu-devel, Markus Armbruster On Fri, 06 Jun 2014 10:11:27 +0200 Paolo Bonzini <pbonzini@redhat.com> wrote: > Il 06/06/2014 09:41, Cornelia Huck ha scritto: > > On Wed, 28 May 2014 12:39:17 +0200 > > Cornelia Huck <cornelia.huck@de.ibm.com> wrote: > > > >> The check for big or little endianness relies on grep reporting > >> match/non-match on the generated binary. If the user specified > >> --binary-files=without-match in their GREP_OPTIONS, this will fail. > >> > >> Let's follow what autoconf does and unset GREP_OPTIONS and CLICOLOR_FORCE > >> at the beginning of the script. > >> > >> Reported-by: Eugene (jno) Dvurechenski <jno@linux.vnet.ibm.com> > >> Suggested-by: Markus Armbruster <armbru@redhat.com> > >> Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> > >> --- > >> configure | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/configure b/configure > >> index 0e516f9..525da56 100755 > >> --- a/configure > >> +++ b/configure > >> @@ -3,6 +3,11 @@ > >> # qemu configure script (c) 2003 Fabrice Bellard > >> # > >> > >> +# Unset some variables known to interfere with behavior of common tools, > >> +# just as autoconf does. > >> +CLICOLOR_FORCE= GREP_OPTIONS= > >> +unset CLICOLOR_FORCE GREP_OPTIONS > >> + > >> # Temporary directory used for files created while > >> # configure runs. Since it is in the build directory > >> # we can safely blow away any previous version of it > > > > Is there an obvious tree to merge configure changes through, or should > > I just throw it into my next s390 pull request? > > Hi, > > I picked up this patch and will push it to a "configure" branch on github. > > Paolo > Cool, thx! ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-06-06 8:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1401273557-6578-1-git-send-email-cornelia.huck@de.ibm.com>
[not found] ` <1401273557-6578-2-git-send-email-cornelia.huck@de.ibm.com>
2014-06-06 7:41 ` [Qemu-devel] [PATCH v2 1/1] configure: unset interfering variables Cornelia Huck
2014-06-06 8:11 ` Paolo Bonzini
2014-06-06 8:23 ` Cornelia Huck
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).