diff options
author | Junio C Hamano <gitster@pobox.com> | 2008-10-14 21:27:12 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-10-14 23:38:39 -0700 |
commit | eee0184da8457c1dbd4418f95d158917540da094 (patch) | |
tree | f2920493adc997f099abae2dd6384b7bce990e4b | |
parent | 67faaaba8338cf59c212ce5239eeb223a77acdf0 (diff) | |
download | git-eee0184da8457c1dbd4418f95d158917540da094.tar.gz git-eee0184da8457c1dbd4418f95d158917540da094.tar.xz |
Fix reading of cloud tags
The projectroot path could have SP in it, in which case iterating over
<$git_dir/ctags/*> does not correctly enumerate the cloud tags files at
all.
This can be observed by creating an empty t/trash directory and running
t9500 test. The $projectroot ends with "trash directory.t9500-gitweb-/"
and <$glob> would give "trash", which can be opened and reading from it
immediately yields undef, which in turn gives an undef value warning to
the standard error stream upon attempt to chomp it.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rwxr-xr-x | gitweb/gitweb.perl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl index 11168006c..41b68668e 100755 --- a/gitweb/gitweb.perl +++ b/gitweb/gitweb.perl @@ -1805,7 +1805,10 @@ sub git_get_project_ctags { my $ctags = {}; $git_dir = "$projectroot/$path"; - foreach (<$git_dir/ctags/*>) { + unless (opendir D, "$git_dir/ctags") { + return $ctags; + } + foreach (grep { -f $_ } map { "$git_dir/ctags/$_" } readdir(D)) { open CT, $_ or next; my $val = <CT>; chomp $val; @@ -1813,6 +1816,7 @@ sub git_get_project_ctags { my $ctag = $_; $ctag =~ s#.*/##; $ctags->{$ctag} = $val; } + closedir D; $ctags; } |