Re: [Qemu-devel] [PATCH] get_maintainer.pl: Default to --no-git-fallback

[Michael S. Tsirkin]

On Wed, Oct 22, 2014 at 10:10:58AM +0200, Thomas Huth wrote:
> On Wed, 22 Oct 2014 10:01:43 +0300
> "Michael S. Tsirkin" <address@hidden> wrote:
> 
> > On Wed, Oct 22, 2014 at 08:39:59AM +0200, Markus Armbruster wrote:
> > > "Michael S. Tsirkin" <address@hidden> writes:
> > > 
> > > > On Tue, Oct 21, 2014 at 03:29:14PM +0200, Markus Armbruster wrote:
> > > >> "Michael S. Tsirkin" <address@hidden> writes:
> > > >> 
> > > >> > On Tue, Oct 21, 2014 at 02:22:41PM +0200, Markus Armbruster wrote:
> > > [...]
> > > >> >> My patch to get_maintainers.pl triggered a whole thread, while the
> > > >> >> message I sent on MAINTAINERS coverage got just one reply so far, 
> > > >> >> and
> > > >> >> even that one's really just about get_maintainers.pl.  
> > > >> >> Disappointing.
> > > >> >> Looks like we're still looking for an easy technical fix.  I doubt 
> > > >> >> there
> > > >> >> is one.
> > > >> >
> > > >> > At least for myself, that's because I'm Cc'd directly on the patch
> > > >> > but not on the MAINTAINERS coverage mail.
> > > >> > And that's ... because get_maintainers picks my mail from git?
> > > >> >
> > > >> > See how it's useful now?
> > > >> 
> > > >> Except that's not what happened.
> > > >> 
> > > >>     $ scripts/get_maintainer.pl --git-fallback -f 
> > > >> scripts/get_maintainer.pl 
> > > >> 
> > > >> No output.  I picked you from git-log manually.
> > > >
> > > > Weird.
> > > > It works for me:
> > > >  ./scripts/get_maintainer.pl -f scripts/get_maintainer.pl 
> > > > "Michael S. Tsirkin" <address@hidden> (commit_signer:1/1=100%)
> > > >
> > > > Maybe --git-fallback is broken?
> > > 
> > > I tried on master (i.e. without my patch, clean tree, with and without
> > > --git-fallback.  Just tried it again, same result.
> > 
> > Well ... I don't know why.
> > It's clearly a bug.
> > Different perl version? Different git version?
> > 
> > Can you try tracing it?
> > Does it exec git?
> 
> I've got the same behaviour as Markus (i.e. no output), and I think
> this is due to get_maintainer.pl using $email_git_since = "1-year-ago"
> ... since there wasn't a commit to that file in the last year, git-log
> simply does not output any entry at all.
> Do you have a non-upstream commit in your git tree that changes that
> file? That would explain why you get some output here.
> 
>  Thomas

Aha. Right. Thanks!