From: Michael Tokarev <mjt@tls.msk.ru>
To: Stewart Smith <stewart@linux.vnet.ibm.com>
Cc: qemu-trivial@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-trivial] [PATCH v2 0/6] configure: make output and errors more friendly
Date: Sat, 01 Feb 2014 13:55:14 +0400 [thread overview]
Message-ID: <52ECC482.3080700@msgid.tls.msk.ru> (raw)
In-Reply-To: <1390527551-22069-1-git-send-email-stewart@linux.vnet.ibm.com>
24.01.2014 05:39, Stewart Smith wrote:
> This expands on my previous patch which was just the "configure: helpfully
> output package names for some missing dependencies" patch.
>
> Based on feedback I've added a hint to the could not find DTC error message.
>
> There's also other changes that were suggested and the ordering of output
> from configure and configure --help which I found rather useful when
> attempting to find what package was needed for each configure option.
>
> Stewart Smith (6):
>1 configure: add hint of libfdt to DTC dependency not found message
>2 configure: sort output of configure results alphabetically
>3 configure: alphabetize output of --help and separate into groups
>4 configure: add help for --disable-xfsctl and --enable-xfsctl
>5 configure: add hints to a remedy for feature_not_found errors
>6 configure: helpfully output package names for some missing
> dependencies.
I think I'd happily take 1, 2, 5 and 6, but 3 and 4 really needs to be
redone, mentioning every option just once, not twice as it is done
now. After that's done, it might be not necessary anymore for 3 at all.
Or I can wait for a resend of whole series. Or alternatively I can
try to rework it myself, but for now I'm quite busy so can't promise
anything.
Please note that major tweaks in ./configure will most likely conflict
with modules support series which has been posted for revew again.
What should I do?
Thanks,
/mjt
WARNING: multiple messages have this Message-ID (diff)
From: Michael Tokarev <mjt@tls.msk.ru>
To: Stewart Smith <stewart@linux.vnet.ibm.com>
Cc: qemu-trivial@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Qemu-trivial] [PATCH v2 0/6] configure: make output and errors more friendly
Date: Sat, 01 Feb 2014 13:55:14 +0400 [thread overview]
Message-ID: <52ECC482.3080700@msgid.tls.msk.ru> (raw)
In-Reply-To: <1390527551-22069-1-git-send-email-stewart@linux.vnet.ibm.com>
24.01.2014 05:39, Stewart Smith wrote:
> This expands on my previous patch which was just the "configure: helpfully
> output package names for some missing dependencies" patch.
>
> Based on feedback I've added a hint to the could not find DTC error message.
>
> There's also other changes that were suggested and the ordering of output
> from configure and configure --help which I found rather useful when
> attempting to find what package was needed for each configure option.
>
> Stewart Smith (6):
>1 configure: add hint of libfdt to DTC dependency not found message
>2 configure: sort output of configure results alphabetically
>3 configure: alphabetize output of --help and separate into groups
>4 configure: add help for --disable-xfsctl and --enable-xfsctl
>5 configure: add hints to a remedy for feature_not_found errors
>6 configure: helpfully output package names for some missing
> dependencies.
I think I'd happily take 1, 2, 5 and 6, but 3 and 4 really needs to be
redone, mentioning every option just once, not twice as it is done
now. After that's done, it might be not necessary anymore for 3 at all.
Or I can wait for a resend of whole series. Or alternatively I can
try to rework it myself, but for now I'm quite busy so can't promise
anything.
Please note that major tweaks in ./configure will most likely conflict
with modules support series which has been posted for revew again.
What should I do?
Thanks,
/mjt
next prev parent reply other threads:[~2014-02-01 9:55 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-23 4:54 [Qemu-trivial] [PATCH] configure: helpfully output package names for some missing dependencies Stewart Smith
2014-01-23 4:54 ` [Qemu-devel] " Stewart Smith
2014-01-23 6:21 ` [Qemu-trivial] " Stefan Weil
2014-01-23 6:21 ` Stefan Weil
2014-01-23 10:28 ` [Qemu-trivial] " Peter Maydell
2014-01-23 10:28 ` Peter Maydell
2014-01-23 11:08 ` [Qemu-trivial] " Stewart Smith
2014-01-23 11:08 ` Stewart Smith
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 0/6] configure: make output and errors more friendly Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 1/6] configure: add hint of libfdt to DTC dependency not found message Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 6:13 ` [Qemu-trivial] " Stefan Weil
2014-01-24 6:13 ` Stefan Weil
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 2/6] configure: sort output of configure results alphabetically Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 3/6] configure: alphabetize output of --help and separate into groups Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 6:30 ` [Qemu-trivial] " Stefan Weil
2014-01-24 6:30 ` Stefan Weil
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 4/6] configure: add help for --disable-xfsctl and --enable-xfsctl Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 5/6] configure: add hints to a remedy for feature_not_found errors Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-01-24 1:39 ` [Qemu-trivial] [PATCH v2 6/6] configure: helpfully output package names for some missing dependencies Stewart Smith
2014-01-24 1:39 ` [Qemu-devel] " Stewart Smith
2014-02-01 9:55 ` Michael Tokarev [this message]
2014-02-01 9:55 ` [Qemu-devel] [Qemu-trivial] [PATCH v2 0/6] configure: make output and errors more friendly Michael Tokarev
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=52ECC482.3080700@msgid.tls.msk.ru \
--to=mjt@tls.msk.ru \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@nongnu.org \
--cc=stewart@linux.vnet.ibm.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 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.