diff mbox series

[v7,2/4] gitignore: ignore a bit more

Message ID 20201012233740.190-3-luoyonggang@gmail.com (mailing list archive)
State New, archived
Headers show
Series Improve cirrus msys2 | expand

Commit Message

Yonggang Luo Oct. 12, 2020, 11:37 p.m. UTC
Enable the creating multiple build directory at the source root.
Ignore /meson/ and /roms/ for better search experience.

Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
 .gitignore | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Philippe Mathieu-Daudé Oct. 13, 2020, 8:38 a.m. UTC | #1
On 10/13/20 1:37 AM, Yonggang Luo wrote:
> Enable the creating multiple build directory at the source root.
> Ignore /meson/ and /roms/ for better search experience.
> 
> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> ---
>   .gitignore | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/.gitignore b/.gitignore
> index b32bca1315..f78ee9f297 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -1,5 +1,5 @@
>   /GNUmakefile
> -/build/
> +/build*/
>   *.pyc
>   .sdk
>   .stgit-*
> @@ -10,3 +10,5 @@ TAGS
>   *~
>   *.ast_raw
>   *.depend_raw
> +/meson/
> +/roms/**/*

Why?
Yonggang Luo Oct. 13, 2020, 8:53 a.m. UTC | #2
On Tue, Oct 13, 2020 at 4:38 PM Philippe Mathieu-Daudé <philmd@redhat.com>
wrote:
>
> On 10/13/20 1:37 AM, Yonggang Luo wrote:
> > Enable the creating multiple build directory at the source root.
> > Ignore /meson/ and /roms/ for better search experience.
> >
> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
> > ---
> >   .gitignore | 4 +++-
> >   1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/.gitignore b/.gitignore
> > index b32bca1315..f78ee9f297 100644
> > --- a/.gitignore
> > +++ b/.gitignore
> > @@ -1,5 +1,5 @@
> >   /GNUmakefile
> > -/build/
> > +/build*/
> >   *.pyc
> >   .sdk
> >   .stgit-*
> > @@ -10,3 +10,5 @@ TAGS
> >   *~
> >   *.ast_raw
> >   *.depend_raw
> > +/meson/
> > +/roms/**/*
>
> Why?
As I said, help for searching tool ignore it, meson and roms are thirdparty
packages.
>


--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
Thomas Huth Oct. 13, 2020, 9:01 a.m. UTC | #3
On 13/10/2020 10.53, 罗勇刚(Yonggang Luo) wrote:
> 
> 
> On Tue, Oct 13, 2020 at 4:38 PM Philippe Mathieu-Daudé <philmd@redhat.com
> <mailto:philmd@redhat.com>> wrote:
>>
>> On 10/13/20 1:37 AM, Yonggang Luo wrote:
>> > Enable the creating multiple build directory at the source root.
>> > Ignore /meson/ and /roms/ for better search experience.
>> >
>> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com
> <mailto:luoyonggang@gmail.com>>
>> > ---
>> >   .gitignore | 4 +++-
>> >   1 file changed, 3 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/.gitignore b/.gitignore
>> > index b32bca1315..f78ee9f297 100644
>> > --- a/.gitignore
>> > +++ b/.gitignore
>> > @@ -1,5 +1,5 @@
>> >   /GNUmakefile
>> > -/build/
>> > +/build*/

The naming of your private build directories is IMHO something that should
not be part of the public .gitignore file. I think you can use your
~/.gitignore file for that.

>> >   *.pyc
>> >   .sdk
>> >   .stgit-*
>> > @@ -10,3 +10,5 @@ TAGS
>> >   *~
>> >   *.ast_raw
>> >   *.depend_raw
>> > +/meson/
>> > +/roms/**/*
>>
>> Why?
> As I said, help for searching tool ignore it, meson and roms are thirdparty
> packages.

Hmm, but "git status" should still show whether there are modifications in
the subdirectories, so IMHO this should not be ignored?

 Thomas
Yonggang Luo Oct. 13, 2020, 9:23 a.m. UTC | #4
On Tue, Oct 13, 2020 at 5:02 PM Thomas Huth <thuth@redhat.com> wrote:
>
> On 13/10/2020 10.53, 罗勇刚(Yonggang Luo) wrote:
> >
> >
> > On Tue, Oct 13, 2020 at 4:38 PM Philippe Mathieu-Daudé <
philmd@redhat.com
> > <mailto:philmd@redhat.com>> wrote:
> >>
> >> On 10/13/20 1:37 AM, Yonggang Luo wrote:
> >> > Enable the creating multiple build directory at the source root.
> >> > Ignore /meson/ and /roms/ for better search experience.
> >> >
> >> > Signed-off-by: Yonggang Luo <luoyonggang@gmail.com
> > <mailto:luoyonggang@gmail.com>>
> >> > ---
> >> >   .gitignore | 4 +++-
> >> >   1 file changed, 3 insertions(+), 1 deletion(-)
> >> >
> >> > diff --git a/.gitignore b/.gitignore
> >> > index b32bca1315..f78ee9f297 100644
> >> > --- a/.gitignore
> >> > +++ b/.gitignore
> >> > @@ -1,5 +1,5 @@
> >> >   /GNUmakefile
> >> > -/build/
> >> > +/build*/
>
> The naming of your private build directories is IMHO something that should
> not be part of the public .gitignore file. I think you can use your
> ~/.gitignore file for that.
>
> >> >   *.pyc
> >> >   .sdk
> >> >   .stgit-*
> >> > @@ -10,3 +10,5 @@ TAGS
> >> >   *~
> >> >   *.ast_raw
> >> >   *.depend_raw
> >> > +/meson/
> >> > +/roms/**/*
> >>
> >> Why?
> > As I said, help for searching tool ignore it, meson and roms are
thirdparty
> > packages.
>
> Hmm, but "git status" should still show whether there are modifications in
> the subdirectories, so IMHO this should not be ignored?
Yeap, it's doens't affect git, just affect search tools. If this is
improper, skip it, I can use it locally
>
>  Thomas
>


--
         此致
礼
罗勇刚
Yours
    sincerely,
Yonggang Luo
diff mbox series

Patch

diff --git a/.gitignore b/.gitignore
index b32bca1315..f78ee9f297 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,5 @@ 
 /GNUmakefile
-/build/
+/build*/
 *.pyc
 .sdk
 .stgit-*
@@ -10,3 +10,5 @@  TAGS
 *~
 *.ast_raw
 *.depend_raw
+/meson/
+/roms/**/*