Re: [Qemu-trivial] [Qemu-devel] [PATCH] configure: Check if struct fsxattr is available from linux header

[Markus Armbruster]

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.