>> }
>>
>> @@ -458,9 +459,12 @@ sub find_empty_directories {
>> my $skip_added = 0;
>> foreach my $t (qw/dir_prop file_prop/) {
>> foreach my $path (keys %{ $self->{$t} }) {
>> - if (exists $self->{$t}->{dirname($path)}) {
>> - $skip_added = 1;
>> - last;
>> + if (length $self->git_path($path)) {
>> + $path = dirname($path);
>> + if ($dir eq $self->git_path($path) &&
>> exists $self->{$t}->{$path}) {
>> + $skip_added = 1;
>> + last;
>> + }
JCH> I am reading that this is a solution for your second issue (use
JCH> git_path() to convert $path). An empty $path would be a top-level
JCH> and skipping it corresponds to the "next if $dir eq '.'" at the
JCH> beginning of the loop, I guess.
JCH> When "$dir ne $self->git_path(dirname($path))", what should happen?
'ls-tree' will be executed.
I guess, the original idea was to save processes, although I don't
know why the dir is in @deleted_gpath, if it has children.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
