Jakub Narebski <jnareb@xxxxxxxxx> writes:
> Junio C Hamano wrote:
>> Jakub Narebski <jnareb@xxxxxxxxx> writes:
>>
>>> Initial (root) commit has no parents, and $co{'parent'} is
>>> undefined. Use '--root' for initial commit.
>>>
>>> This fixes "Use of uninitialized value in open at gitweb/gitweb.perl
>>> line 4925." warning.
> [...]
>>> --- a/gitweb/gitweb.perl
>>> +++ b/gitweb/gitweb.perl
>>> @@ -4923,7 +4923,8 @@ XML
>>>
>>> # get list of changed files
>>> open my $fd, "-|", git_cmd(), "diff-tree", '-r', @diff_opts,
>>> - $co{'parent'}, $co{'id'}, "--", (defined $file_name ? $file_name : ())
>>> + $co{'parent'} || "--root",
>>> + $co{'id'}, "--", (defined $file_name ? $file_name : ())
>>> or next;
>>> my @difftree = map { chomp; $_ } <$fd>;
>>> close $fd
>>
>> I do not think you would need to make --root conditional...
>
> I need at last make $co{'parent'} conditional, i.e. at least
> "$co{'parent'} || ()" or equivalent (e.g. like for $file_name).
> I cannot omit $co{'parent'} because parents might be rewritten;
> feeds accept now path limiting. So why not this way?
Ah, clever (buuuuuuut misleading) ;-)
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
