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

Jesse Barnes
2015-08-14T18:24:10Z

On 08/14/2015 09:07 AM, Daniel Vetter wrote:
> 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.

ah ok, I can drop those. I'll clean and such and just add the minimal set here.

Jesse