linux-kernel - Re: [PATCH v3 21/27] MAINTAINERS: get rid of non-existing Documentation/fpga

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180614135254.75cfc17d@coco.lan>
Date:   Thu, 14 Jun 2018 13:52:54 -0300
From:   Mauro Carvalho Chehab <mchehab+samsung@...nel.org>
To:     Alan Tull <atull@...nel.org>
Cc:     Linux Doc Mailing List <linux-doc@...r.kernel.org>,
        Mauro Carvalho Chehab <mchehab@...radead.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        Jonathan Corbet <corbet@....net>,
        Moritz Fischer <mdf@...nel.org>, linux-fpga@...r.kernel.org
Subject: Re: [PATCH v3 21/27] MAINTAINERS: get rid of non-existing
 Documentation/fpga

Em Thu, 14 Jun 2018 11:39:20 -0500
Alan Tull <atull@...nel.org> escreveu:

> On Thu, Jun 14, 2018 at 11:09 AM, Mauro Carvalho Chehab
> <mchehab+samsung@...nel.org> wrote:
> > This directory seems to have moved to a new place
> > (Documentation/driver-api/fpga/), with is already at MAINTAINERS.
> >
> > So, get rid of the old location.  
> 
> Wow, you caught that fast!

Just a coincidence :-) I had to rebase a pending patch series close to
the end of the merge window.

When I ran the check tool:
	./scripts/documentation-file-ref-check

It detected this.

> There's other documentation coming to this
> directory soon.  IMO we might as well leave it.  We don't need to keep
> adding and deleting this line from MAINTAINERS.

Yeah, sure. I'll drop this hunk for v4.

> 
> Also, why didn't you cc linux-fpga mailing list on this?  I guess you
> ran get_maintainer.pl after adding this patch? :)

Yeah, that was the case. I also had to sanitize the c/c for this
specific patch as get_maintainers.pl isn't good enough to check for
MAINTAINERS access... it usually just get some random noise...

> 
> Alan
> 
> >
> > Cc: Alan Tull <atull@...nel.org>
> > Cc: Moritz Fischer <mdf@...nel.org>
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@...nel.org>
> > ---
> >  MAINTAINERS | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 783ce44b3e7a..67641f5bb373 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -5632,7 +5632,6 @@ L:        linux-fpga@...r.kernel.org
> >  S:     Maintained
> >  T:     git git://git.kernel.org/pub/scm/linux/kernel/git/atull/linux-fpga.git
> >  Q:     http://patchwork.kernel.org/project/linux-fpga/list/
> > -F:     Documentation/fpga/
> >  F:     Documentation/driver-api/fpga/
> >  F:     Documentation/devicetree/bindings/fpga/
> >  F:     drivers/fpga/
> > --
> > 2.17.1
> >  



Thanks,
Mauro