Pre-commit hook not handling multiple files correctly

[tvkanters]

On Ubuntu 18.04, after using addKtlintCheckGitPreCommitHook and trying to commit multiple files, the pre-commit hook fails with a message Task 'path/to/second/File.kt' not found in root project 'ProjectName'.

After some investigation, it turns out this is due to this line in the pre-commit hook:

./gradlew --quiet ktlintCheck -PinternalKtlintGitFilter=$CHANGED_FILES

Because there are no quotes around $CHANGED_FILES, the files after the first one were seen as a separate Gradle task. It can be fixed by adding the quotes:

./gradlew --quiet ktlintCheck -PinternalKtlintGitFilter="$CHANGED_FILES"

For now we've got a bash script to patch it:

sed -i 's/\(-PinternalKtlintGitFilter=\)\(\$.*\)$/\1"\2"/' .git/hooks/pre-commit