All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Jackson <ian.jackson@citrix.com>
To: Eric Blake <eblake@redhat.com>
Cc: Anthony PERARD <anthony.perard@citrix.com>,
	Juergen Gross <jgross@suse.com>,
	Janosch Frank <frankja@linux.vnet.ibm.com>,
	Thomas Huth <thuth@redhat.com>,
	Stefano  Stabellini <sstabellini@kernel.org>,
	Peter Maydell <peter.maydell@linaro.org>,
	"Kent R. Spillner" <kspillner@acm.org>,
	qemu-devel@nongnu.org, Ross Lagerwall <ross.lagerwall@citrix.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	xen-devel@lists.xenproject.org
Subject: Re: [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash
Date: Tue, 24 Apr 2018 16:05:01 +0100	[thread overview]
Message-ID: <23263.18333.710326.753665@mariner.uk.xensource.com> (raw)
In-Reply-To: <4402483b-17fd-60e0-4f00-05887a11320d@redhat.com>

Eric Blake writes ("Re: [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash"):
> That's still fork-heavy.  You could do:
> 
> test -n "$BASH_VERSION" && eval '
> echo >>config.log "
> funcs: ${FUNCNAME[*]}
> lines: ${BASH_LINENO[*]}
> files: ${BASH_SOURCE[*]}"'
> 
> which avoids the fork, but remains silent on dash.

Thanks.  I will adopt this.  Although I will use if ... then as that
seems to be the usual style.  (&& would break with set -e which
configure does not use...)

Ian.

WARNING: multiple messages have this Message-ID (diff)
From: Ian Jackson <ian.jackson@citrix.com>
To: Eric Blake <eblake@redhat.com>
Cc: Juergen Gross <jgross@suse.com>,
	Peter Maydell <peter.maydell@linaro.org>,
	Thomas Huth <thuth@redhat.com>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Janosch Frank <frankja@linux.vnet.ibm.com>,
	"Kent R. Spillner" <kspillner@acm.org>,
	qemu-devel@nongnu.org, Ross Lagerwall <ross.lagerwall@citrix.com>,
	xen-devel@lists.xenproject.org,
	Anthony PERARD <anthony.perard@citrix.com>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash
Date: Tue, 24 Apr 2018 16:05:01 +0100	[thread overview]
Message-ID: <23263.18333.710326.753665@mariner.uk.xensource.com> (raw)
In-Reply-To: <4402483b-17fd-60e0-4f00-05887a11320d@redhat.com>

Eric Blake writes ("Re: [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash"):
> That's still fork-heavy.  You could do:
> 
> test -n "$BASH_VERSION" && eval '
> echo >>config.log "
> funcs: ${FUNCNAME[*]}
> lines: ${BASH_LINENO[*]}
> files: ${BASH_SOURCE[*]}"'
> 
> which avoids the fork, but remains silent on dash.

Thanks.  I will adopt this.  Although I will use if ... then as that
seems to be the usual style.  (&& would break with set -e which
configure does not use...)

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

  reply	other threads:[~2018-04-24 15:05 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-19 16:45 [Qemu-devel] [PATCH v7 00/16] xen: xen-domid-restrict improvements Ian Jackson
2018-04-19 16:45 ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 01/16] checkpatch: Add xendevicemodel_handle to the list of types Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 02/16] AccelClass: Introduce accel_setup_post Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 03/16] xen: link against xentoolcore Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-23 15:05   ` [Qemu-devel] " Anthony PERARD
2018-04-23 15:05     ` Anthony PERARD
2018-04-24 14:28     ` [Qemu-devel] " Ian Jackson
2018-04-24 14:28       ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 04/16] xen: restrict: use xentoolcore_restrict_all Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 05/16] xen: defer call to xen_restrict until just before os_setup_post Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-23 14:28   ` [Qemu-devel] " Anthony PERARD
2018-04-23 14:28     ` Anthony PERARD
2018-04-23 15:58   ` [Qemu-devel] " Anthony PERARD
2018-04-23 15:58     ` Anthony PERARD
2018-04-24 15:08     ` Ian Jackson
2018-04-24 15:08     ` [Qemu-devel] " Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 06/16] xen: destroy_hvm_domain: Move reason into a variable Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 07/16] xen: move xc_interface compatibility fallback further up the file Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 08/16] xen: destroy_hvm_domain: Try xendevicemodel_shutdown Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 09/16] os-posix: cleanup: Replace fprintfs with error_report in change_process_uid Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 10/16] os-posix: Provide new -runas <uid>:<gid> facility Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-20 12:56   ` [Qemu-devel] " Markus Armbruster
2018-04-20 12:56     ` Markus Armbruster
2018-04-19 16:45 ` [Qemu-devel] [PATCH 11/16] xen: Use newly added dmops for mapping VGA memory Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 12/16] xen: Remove now-obsolete xen_xc_domain_add_to_physmap Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-23 15:38   ` [Qemu-devel] " Anthony PERARD
2018-04-23 15:38   ` Anthony PERARD
2018-04-19 16:45 ` [Qemu-devel] [PATCH 13/16] xen: Expect xenstore write to fail when restricted Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 14/16] os-posix: cleanup: Replace fprintf with error_report in remaining call sites Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 20:25   ` [Qemu-devel] " Philippe Mathieu-Daudé
2018-04-19 20:25   ` Philippe Mathieu-Daudé
2018-04-20 10:19     ` Ian Jackson
2018-04-20 10:19       ` Ian Jackson
2018-04-19 16:45 ` [Qemu-devel] [PATCH 15/16] os-posix: cleanup: Replace perror with error_report Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-19 20:31   ` [Qemu-devel] " Philippe Mathieu-Daudé
2018-04-19 20:31     ` Philippe Mathieu-Daudé
2018-04-24 14:53     ` Ian Jackson
2018-04-24 14:53       ` Ian Jackson
2018-04-24 15:18       ` Daniel P. Berrangé
2018-04-24 15:18         ` Daniel P. Berrangé
2018-04-24 15:40         ` Eric Blake
2018-04-24 15:40           ` Eric Blake
2018-04-24 15:43           ` Eric Blake
2018-04-24 15:43             ` Eric Blake
2018-04-24 15:54             ` Daniel P. Berrangé
2018-04-24 15:54               ` Daniel P. Berrangé
2018-04-24 16:20     ` Markus Armbruster
2018-04-24 16:20       ` Markus Armbruster
2018-04-19 16:45 ` [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash Ian Jackson
2018-04-19 16:45   ` Ian Jackson
2018-04-23 16:21   ` [Qemu-devel] " Anthony PERARD
2018-04-23 16:21     ` Anthony PERARD
2018-04-23 16:38     ` [Qemu-devel] " Daniel P. Berrangé
2018-04-23 17:12       ` Ian Jackson
2018-04-23 17:12       ` Ian Jackson
2018-04-23 16:38     ` Daniel P. Berrangé
2018-04-23 16:38     ` Ian Jackson
2018-04-23 16:38       ` Ian Jackson
2018-04-23 20:28       ` [Qemu-devel] " Eric Blake
2018-04-23 20:28         ` Eric Blake
2018-04-24 15:05         ` Ian Jackson [this message]
2018-04-24 15:05           ` Ian Jackson
2018-04-19 17:03 ` [Qemu-devel] [PATCH v7 00/16] xen: xen-domid-restrict improvements no-reply
2018-04-19 17:03   ` no-reply
2018-04-24 17:58 [Qemu-devel] [PATCH v8 " Ian Jackson
2018-04-24 17:58 ` [Qemu-devel] [PATCH 16/16] configure: do_compiler: Dump some extra info under bash Ian Jackson
2018-04-25  3:04   ` Thomas Huth

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=23263.18333.710326.753665@mariner.uk.xensource.com \
    --to=ian.jackson@citrix.com \
    --cc=anthony.perard@citrix.com \
    --cc=eblake@redhat.com \
    --cc=frankja@linux.vnet.ibm.com \
    --cc=jgross@suse.com \
    --cc=kspillner@acm.org \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=ross.lagerwall@citrix.com \
    --cc=sstabellini@kernel.org \
    --cc=thuth@redhat.com \
    --cc=xen-devel@lists.xenproject.org \
    /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 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.