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

[Jani Nikula <jani.nikula@linux.intel.com>]

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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx