On Fri, 14 Aug 2015, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx