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

Jani Nikula
2015-08-17T07:46:10Z

On Fri, 14 Aug 2015, Jesse Barnes <jbarnes@virtuousgeek.org> wrote:
> 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.

Ooops, I had sent the patch to the list, but never pushed. Done now.

BR,
Jani.

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

--
Jani Nikula, Intel Open Source Technology Center