[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] configure: Check if struct fsxat
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH] configure: Check if struct fsxattr is available from linux header |
Date: |
Fri, 29 Apr 2016 09:52:58 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Jan Vesely <address@hidden> writes:
> On Tue, 2016-04-12 at 17:09 -0400, Jan Vesely wrote:
>> On Sat, 2016-03-19 at 21:18 -0400, Jan Vesely wrote:
>> >
>> > Fixes build with new linux headers.
>> >
>> > Signed-off-by: Jan Vesely <address@hidden>
>> > ---
>> > linux-headers-4.5 and xfsprogs-3.2.4 in my case
>> >
>> > configure | 18 ++++++++++++++++++
>> > 1 file changed, 18 insertions(+)
>> >
>> > diff --git a/configure b/configure
>> > index b88d0db..bb64d6c 100755
>> > --- a/configure
>> > +++ b/configure
>> > @@ -4474,6 +4474,21 @@ if test "$fortify_source" != "no"; then
>> > fi
>> > fi
>> >
>> > +########################################
>> > +# check if struct fsxattr is available
>> > +
>> > +have_fsxattr=no
>> > +cat > $TMPC << EOF
>> > +#include <linux/fs.h>
>> > +struct fsxattr foo;
>> > +int main(void) {
>> > + return 0;
>> > +}
>> > +EOF
>> > +if compile_prog "" "" ; then
>> > + have_fsxattr=yes
>> > +fi
>> > +
>> > ##########################################
>> > # End of CC checks
>> > # After here, no more $cc or $ld runs
>> > @@ -5137,6 +5152,9 @@ fi
>> > if test "$have_ifaddrs_h" = "yes" ; then
>> > echo "HAVE_IFADDRS_H=y" >> $config_host_mak
>> > fi
>> > +if test "$have_fsxattr" = "yes" ; then
>> > + echo "HAVE_FSXATTR=y" >> $config_host_mak
>> > +fi
>> > if test "$vte" = "yes" ; then
>> > echo "CONFIG_VTE=y" >> $config_host_mak
>> > echo "VTE_CFLAGS=$vte_cflags" >> $config_host_mak
>> gentle ping.
>> the docs say trivial patches are collected every week. yet the last
>> one
>> was in Feb.
>>
>
> gentle ping v2. Is there a maintainer for configure? get_maintainer.pl
> was not helpful.
This might be trivial if you explain what's broken without it in more
detail. The explanation belongs into the commit message.
Now let's look for victims^Wreviewers:
$ scripts/get_maintainer.pl -f configure
get_maintainer.pl: No maintainers found, printing recent contributors.
get_maintainer.pl: Do not blindly cc: them on patches! Use common sense.
"Daniel P. Berrange" <address@hidden> (commit_signer:21/95=22%)
Paolo Bonzini <address@hidden> (commit_signer:21/95=22%)
Stefan Hajnoczi <address@hidden> (commit_signer:15/95=16%)
Gerd Hoffmann <address@hidden> (commit_signer:11/95=12%)
Peter Maydell <address@hidden> (commit_signer:11/95=12%)
address@hidden (open list:All patches CC here)
$ scripts/get_maintainer.pl -f --git-since "5-years-ago" configure
get_maintainer.pl: No maintainers found, printing recent contributors.
get_maintainer.pl: Do not blindly cc: them on patches! Use common sense.
Peter Maydell <address@hidden> (commit_signer:135/686=20%)
Anthony Liguori <address@hidden> (commit_signer:119/686=17%)
Stefan Hajnoczi <address@hidden> (commit_signer:113/686=16%)
Paolo Bonzini <address@hidden> (commit_signer:110/686=16%)
Stefan Weil <address@hidden> (commit_signer:78/686=11%)
address@hidden (open list:All patches CC here)
$ scripts/get_maintainer.pl --git-blame -f configure
Peter Maydell <address@hidden> (authored
lines:611/5975=10%,commits:91/703=13%)
Juan Quintela <address@hidden> (authored
lines:486/5975=8%,commits:78/703=11%)
"Daniel P. Berrange" <address@hidden> (authored lines:355/5975=6%)
Anthony Liguori <address@hidden> (commits:178/703=25%)
Paolo Bonzini <address@hidden> (commits:90/703=13%)
Stefan Hajnoczi <address@hidden> (commits:88/703=13%)
address@hidden (open list:All patches CC here)
Suggest to copy Paolo Bonzini, Peter Maydell, Stefan Hajnoczi on your
respin.
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Jan Vesely, 2016/04/12
- [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Jan Vesely, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Stefan Weil, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Peter Maydell, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Stefan Weil, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Peter Maydell, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Stefan Weil, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Peter Maydell, 2016/04/29
- Re: [Qemu-trivial] [PATCH] configure: Check if struct fsxattr is available from linux header, Jan Vesely, 2016/04/29
- [Qemu-trivial] [PATCH v2] configure: Check if struct fsxattr is available from linux header, Jan Vesely, 2016/04/29