public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] gitignore: Add files starting with .
@ 2017-02-10 20:37 Michel Thierry
  2017-02-15 12:24 ` Joonas Lahtinen
  0 siblings, 1 reply; 7+ messages in thread
From: Michel Thierry @ 2017-02-10 20:37 UTC (permalink / raw)
  To: intel-gfx; +Cc: Marius Vlad

I cant be the only one that have added .tags by mistake.

Cc: Marius Vlad <marius.c.vlad@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
---
 .gitignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitignore b/.gitignore
index 6204965a..e1457bf6 100644
--- a/.gitignore
+++ b/.gitignore
@@ -81,6 +81,7 @@ core
 *.swo
 *.swp
 *.dirstamp
+.*
 cscope.*
 TAGS
 build-aux/
-- 
2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH i-g-t] gitignore: Add files starting with .
  2017-02-10 20:37 [PATCH i-g-t] gitignore: Add files starting with Michel Thierry
@ 2017-02-15 12:24 ` Joonas Lahtinen
  2017-02-15 18:23   ` Michel Thierry
  2017-02-15 18:24   ` [PATCH i-g-t v2] " Michel Thierry
  0 siblings, 2 replies; 7+ messages in thread
From: Joonas Lahtinen @ 2017-02-15 12:24 UTC (permalink / raw)
  To: Michel Thierry, intel-gfx; +Cc: Marius Vlad

On pe, 2017-02-10 at 12:37 -0800, Michel Thierry wrote:
> I cant be the only one that have added .tags by mistake.
> 
> Cc: Marius Vlad <marius.c.vlad@intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Signed-off-by: Michel Thierry <michel.thierry@intel.com>

<SNIP>

> +++ b/.gitignore
> @@ -81,6 +81,7 @@ core
>  *.swo
>  *.swp
>  *.dirstamp
> +.*

You need to at least have "!.gitignore" to avoid ignoring further
changes :)

Regards, Joonas

>  cscope.*
>  TAGS
>  build-aux/
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH i-g-t] gitignore: Add files starting with .
  2017-02-15 12:24 ` Joonas Lahtinen
@ 2017-02-15 18:23   ` Michel Thierry
  2017-02-15 18:24   ` [PATCH i-g-t v2] " Michel Thierry
  1 sibling, 0 replies; 7+ messages in thread
From: Michel Thierry @ 2017-02-15 18:23 UTC (permalink / raw)
  To: Joonas Lahtinen, intel-gfx



On 15/02/17 04:24, Joonas Lahtinen wrote:
> On pe, 2017-02-10 at 12:37 -0800, Michel Thierry wrote:
>> I cant be the only one that have added .tags by mistake.
>>
>> Cc: Marius Vlad <marius.c.vlad@intel.com>
>> Cc: Petri Latvala <petri.latvala@intel.com>
>> Signed-off-by: Michel Thierry <michel.thierry@intel.com>
>
> <SNIP>
>
>> +++ b/.gitignore
>> @@ -81,6 +81,7 @@ core
>>  *.swo
>>  *.swp
>>  *.dirstamp
>> +.*
>
> You need to at least have "!.gitignore" to avoid ignoring further
> changes :)
>

My plan was to prevent further changes (not).

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH i-g-t v2] gitignore: Add files starting with .
  2017-02-15 12:24 ` Joonas Lahtinen
  2017-02-15 18:23   ` Michel Thierry
@ 2017-02-15 18:24   ` Michel Thierry
  2017-02-16  9:26     ` Petri Latvala
  1 sibling, 1 reply; 7+ messages in thread
From: Michel Thierry @ 2017-02-15 18:24 UTC (permalink / raw)
  To: intel-gfx

I cant be the only one that have added .tags by mistake.

v2: Do not ignore .gitignore

Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
---
 .gitignore | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/.gitignore b/.gitignore
index 6204965a..4d5867a2 100644
--- a/.gitignore
+++ b/.gitignore
@@ -78,9 +78,11 @@ core
 #		Edit the following section as needed
 # For example, !report.pc overrides *.pc. See 'man gitignore'
 #
+!.gitignore
 *.swo
 *.swp
 *.dirstamp
+.*
 cscope.*
 TAGS
 build-aux/
-- 
2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH i-g-t v2] gitignore: Add files starting with .
  2017-02-15 18:24   ` [PATCH i-g-t v2] " Michel Thierry
@ 2017-02-16  9:26     ` Petri Latvala
  2017-02-16 19:34       ` [PATCH i-g-t v3] " Michel Thierry
  0 siblings, 1 reply; 7+ messages in thread
From: Petri Latvala @ 2017-02-16  9:26 UTC (permalink / raw)
  To: Michel Thierry, intel-gfx

On 02/15/2017 08:24 PM, Michel Thierry wrote:
> I cant be the only one that have added .tags by mistake.
>
> v2: Do not ignore .gitignore
>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Michel Thierry <michel.thierry@intel.com>
> ---
>   .gitignore | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/.gitignore b/.gitignore
> index 6204965a..4d5867a2 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -78,9 +78,11 @@ core
>   #		Edit the following section as needed
>   # For example, !report.pc overrides *.pc. See 'man gitignore'
>   #
> +!.gitignore
>   *.swo
>   *.swp
>   *.dirstamp
> +.*
>   cscope.*
>   TAGS
>   build-aux/

"!pattern" re-includes files excluded by _previous_ patterns. Reorder 
those changes so that .* is before !.gitignore.

For the record, that patch "works" for that particular source-root 
.gitignore file because it's already tracked by git and .gitignore only 
affects untracked files. The difference will only come up when adding a 
new .gitignore file in another directory.


-- 
Petri Latvala

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH i-g-t v3] gitignore: Add files starting with .
  2017-02-16  9:26     ` Petri Latvala
@ 2017-02-16 19:34       ` Michel Thierry
  2017-02-17  9:31         ` Petri Latvala
  0 siblings, 1 reply; 7+ messages in thread
From: Michel Thierry @ 2017-02-16 19:34 UTC (permalink / raw)
  To: intel-gfx

I cant be the only one that have added .tags by mistake.

v2: Do not ignore .gitignore
v3: Move !.gitignore at the end, otherwise it'll ignore new .gitignore
files in another directory (Petri)

Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
---
 .gitignore | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/.gitignore b/.gitignore
index 6204965a..de79020f 100644
--- a/.gitignore
+++ b/.gitignore
@@ -81,6 +81,7 @@ core
 *.swo
 *.swp
 *.dirstamp
+.*
 cscope.*
 TAGS
 build-aux/
@@ -93,3 +94,8 @@ intel-gpu-tools-*/
 
 piglit
 results
+
+#
+# git files that we don't want to ignore even it they are dot-files
+#
+!.gitignore
-- 
2.11.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH i-g-t v3] gitignore: Add files starting with .
  2017-02-16 19:34       ` [PATCH i-g-t v3] " Michel Thierry
@ 2017-02-17  9:31         ` Petri Latvala
  0 siblings, 0 replies; 7+ messages in thread
From: Petri Latvala @ 2017-02-17  9:31 UTC (permalink / raw)
  To: Michel Thierry, intel-gfx



On 02/16/2017 09:34 PM, Michel Thierry wrote:
> I cant be the only one that have added .tags by mistake.
>
> v2: Do not ignore .gitignore
> v3: Move !.gitignore at the end, otherwise it'll ignore new .gitignore
> files in another directory (Petri)
>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Michel Thierry <michel.thierry@intel.com>

Reviewed-by: Petri Latvala <petri.latvala@intel.com>

> ---
>   .gitignore | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/.gitignore b/.gitignore
> index 6204965a..de79020f 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -81,6 +81,7 @@ core
>   *.swo
>   *.swp
>   *.dirstamp
> +.*
>   cscope.*
>   TAGS
>   build-aux/
> @@ -93,3 +94,8 @@ intel-gpu-tools-*/
>   
>   piglit
>   results
> +
> +#
> +# git files that we don't want to ignore even it they are dot-files
> +#
> +!.gitignore

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-02-17  9:31 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-10 20:37 [PATCH i-g-t] gitignore: Add files starting with Michel Thierry
2017-02-15 12:24 ` Joonas Lahtinen
2017-02-15 18:23   ` Michel Thierry
2017-02-15 18:24   ` [PATCH i-g-t v2] " Michel Thierry
2017-02-16  9:26     ` Petri Latvala
2017-02-16 19:34       ` [PATCH i-g-t v3] " Michel Thierry
2017-02-17  9:31         ` Petri Latvala

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox