Re: [PATCH 18/18] gitignore: ignore more files

Daniel Vetter
2015-08-14T16:08:01Z

On Fri, Aug 14, 2015 at 08:20:22AM -0700, Jesse Barnes wrote:
> git clean updates the .gitignore file? Not having to run git clean is the whole point of this patch...

I looked at this patch first, but later noticed that you have a few
renames where you don't update the .gitignore.
>
> On 08/14/2015 01:09 AM, Daniel Vetter wrote:
> > On Thu, Aug 13, 2015 at 01:31:41PM -0700, Jesse Barnes wrote:
> >
> > git clean fixes this all, at least over here git status is clean.
> > -Daniel
> >
> >> ---
> >> .gitignore | 3 +++
> >> tests/.gitignore | 13 +++++++++++++
> >> tools/.gitignore | 8 ++++++++
> >> 3 files changed, 24 insertions(+)
> >>
> >> diff --git a/.gitignore b/.gitignore
> >> index a438c1c..533f6f1 100644
> >> --- a/.gitignore
> >> +++ b/.gitignore
> >> @@ -90,3 +90,6 @@ gtk-doc.m4
> >>
> >> piglit
> >> results
> >> +
> >> +*.orig
> >> +version.h
> >> diff --git a/tests/.gitignore b/tests/.gitignore
> >> index 31d35a5..e45cbb7 100644
> >> --- a/tests/.gitignore
> >> +++ b/tests/.gitignore
> >> @@ -99,6 +99,7 @@ gem_set_tiling_vs_blt
> >> gem_set_tiling_vs_gtt
> >> gem_set_tiling_vs_pwrite
> >> gem_storedw_batches_loop
> >> +gem_storedw_loop
> >> gem_storedw_loop_blt
> >> gem_storedw_loop_bsd
> >> gem_storedw_loop_render
> >> @@ -169,3 +170,15 @@ prime_udl
> >> template
> >> test-list.txt
> >> testdisplay
> >> +ddi_compute_wrpll
> >> +gem_vmap_blits
> >> +gem_wait_render_timeout
> >> +igt_fork_helper
> >> +igt_list_only
> >> +igt_no_exit
> >> +igt_no_exit_list_only
> >> +igt_no_subtest
> >> +igt_simulation

But all the igt_* have moved to lib/tests/ so you definitely have a pile
of old build system gunk that needs to be cleaned out first with git
clean.

> >> +pm_pc8
> >> +pm_psr
> >> +
> >> diff --git a/tools/.gitignore b/tools/.gitignore
> >> index 49bd24f..7bf91e3 100644
> >> --- a/tools/.gitignore
> >> +++ b/tools/.gitignore
> >> @@ -37,3 +37,11 @@ intel_vga_write
> >> intel_watermark
> >> skl_compute_wrpll
> >> skl_ddb_allocation
> >> +forcewaked
> >> +intel_dpio_read
> >> +intel_dpio_write
> >> +intel_gpu_dump
> >> +intel_nc_read
> >> +intel_nc_write
> >> +intel_punit_read
> >> +intel_punit_write

Same with those, Jani killed them all iirc.
-Daniel

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

--
Daniel Vetter
Software Engineer, Intel Corporation