Re: [PATCH 2/2] t/Makefile: always test all lint targets when running tests

2014-07-08 Thread Junio C Hamano
On Tue, Jul 8, 2014 at 12:24 PM, Jens Lehmann wrote: > > Am 07.07.2014 20:13, schrieb Junio C Hamano: > > > > So I am not very enthusiastic to see this change myself. > > Ok, I understand we do not want to lightly risk false positives. I > just noticed that I accidentally forgot to sign off this s

Re: [PATCH 2/2] t/Makefile: always test all lint targets when running tests

2014-07-08 Thread Jeff King
On Mon, Jul 07, 2014 at 11:13:11AM -0700, Junio C Hamano wrote: > Jens Lehmann writes: > > > Only the two targets "test-lint-duplicates" and "test-lint-executable" are > > currently executed when running the test target. This was done on purpose > > when the TEST_LINT variable was added in 81127

Re: [PATCH 2/2] t/Makefile: always test all lint targets when running tests

2014-07-08 Thread Jens Lehmann
Am 07.07.2014 20:13, schrieb Junio C Hamano: > Jens Lehmann writes: > >> Only the two targets "test-lint-duplicates" and "test-lint-executable" are >> currently executed when running the test target. This was done on purpose >> when the TEST_LINT variable was added in 81127d74. But as this does n

Re: [PATCH 2/2] t/Makefile: always test all lint targets when running tests

2014-07-07 Thread Junio C Hamano
Jens Lehmann writes: > Only the two targets "test-lint-duplicates" and "test-lint-executable" are > currently executed when running the test target. This was done on purpose > when the TEST_LINT variable was added in 81127d74. But as this does not > include the "test-lint-shell-syntax" target add

[PATCH 2/2] t/Makefile: always test all lint targets when running tests

2014-07-03 Thread Jens Lehmann
Only the two targets "test-lint-duplicates" and "test-lint-executable" are currently executed when running the test target. This was done on purpose when the TEST_LINT variable was added in 81127d74. But as this does not include the "test-lint-shell-syntax" target added the same day in commit c7ce7