linux-kernel - Re: [PATCH] .gitignore: Add compilation database files

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: <CAK7LNAQqUtUPSu=UmtZ-dqsjUgUO-f6U1tQa9VPmJ944poh6zA@mail.gmail.com>
Date:   Wed, 24 Jul 2019 01:36:36 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Toru Komatsu <k0ma@...m0k.jp>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        clang-built-linux <clang-built-linux@...glegroups.com>
Subject: Re: [PATCH] .gitignore: Add compilation database files

On Wed, Jul 24, 2019 at 1:28 AM Toru Komatsu <k0ma@...m0k.jp> wrote:
>
> On 07/24, Masahiro Yamada wrote:
> > Just a nit.
> >
> > The patch title is:
> > .gitignore: Add compilation database "files"
> >
> > Maybe, should it be singular?
> >
> >
> > On Sun, Jul 21, 2019 at 5:55 PM Toru Komatsu <k0ma@...m0k.jp> wrote:
> > >
> > > This file is used by clangd to use language server protocol.
> > > It can be generated at each compile using scripts/gen_compile_commands.py.
> > > Therefore it is different depending on the environment and should be
> > > ignored.
> > >
> > > Signed-off-by: Toru Komatsu <k0ma@...m0k.jp>
> > > ---
> > >  .gitignore | 3 +++
> > >  1 file changed, 3 insertions(+)
> > >
> > > diff --git a/.gitignore b/.gitignore
> > > index 8f5422cba6e2..025d887f64f1 100644
> > > --- a/.gitignore
> > > +++ b/.gitignore
> > > @@ -142,3 +142,6 @@ x509.genkey
> > >
> > >  # Kdevelop4
> > >  *.kdev4
> > > +
> > > +# Clang's compilation database files
> > > +/compile_commands.json
> > > --
> > > 2.17.1
> > >
> >
> >
> > --
> > Best Regards
> > Masahiro Yamada
>
> --
>
> Thanks for your review.
>
>  Sorry, this point which you pointed out is my mistake.
>  It is should be "file".
>
>  I'm begginer because this patch is my first time,
>  What should I do next?


This patch is trivial enough.

I will change "files" -> "file"
(patch subject and code),
then I will apply it.

Thanks.




-- 
Best Regards
Masahiro Yamada