Commit c5f2cffb authored by Robert Speicher's avatar Robert Speicher

Merge branch 'no-argf' into 'master'

ARGF does more than we want

ARGF is a virtual IO stream that treats the elements of ARGV as paths
to files to be opened. In practice, git-receive-pack will probably
never pass arguments to pre-receive and post-receive because it is not
supposed to (according to 'man 5 githooks'). In case it does, however
our hooks should not have the surprise behavior of treating those
arguments as files to be opened.

See merge request !66
parents f2325a45 d5860f45
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
# This file was placed here by GitLab. It makes sure that your pushed commits # This file was placed here by GitLab. It makes sure that your pushed commits
# will be processed properly. # will be processed properly.
refs = ARGF.read refs = $stdin.read
key_id = ENV['GL_ID'] key_id = ENV['GL_ID']
repo_path = Dir.pwd repo_path = Dir.pwd
......
...@@ -3,7 +3,7 @@ ...@@ -3,7 +3,7 @@
# This file was placed here by GitLab. It makes sure that your pushed commits # This file was placed here by GitLab. It makes sure that your pushed commits
# will be processed properly. # will be processed properly.
refs = ARGF.read refs = $stdin.read
key_id = ENV['GL_ID'] key_id = ENV['GL_ID']
repo_path = Dir.pwd repo_path = Dir.pwd
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment