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]

Date:   Thu, 14 Jun 2018 13:09:38 -0500
From:   Alan Tull <atull@...nel.org>
To:     Mauro Carvalho Chehab <mchehab+samsung@...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

On Thu, Jun 14, 2018 at 11:52 AM, Mauro Carvalho Chehab
<mchehab+samsung@...nel.org> wrote:
> 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.

Nice tool.  Thanks for the explanation, it's always good to learn
about another one.

>
>> 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.
>

Thanks!

>>
>> 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