# [PATCH] git-relink.perl: warn() instead of die() on directory open failure 3 messages from 2008-01-11 to 2008-01-12. Participants: Brandon Casey, Junio C Hamano. Thread: https://gitlist.dev/t/11585 ## Brandon Casey, 2008-01-11 18:11 Subject: [PATCH] git-relink.perl: warn() instead of die() on directory open failure Message-ID: <4787B15D.3070105@nrlssc.navy.mil> URL: https://gitlist.dev/e/4787B15D.3070105%40nrlssc.navy.mil ``` Signed-off-by: Brandon Casey --- Sometimes the repository to link to is not under your control. If it contains files or unreadable directories, git-relink will die without this patch. -brandon git-relink.perl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/git-relink.perl b/git-relink.perl index f6b4f6a..391b12d 100755 --- a/git-relink.perl +++ b/git-relink.perl @@ -69,7 +69,7 @@ sub do_scan_directory($$$) { my $dfulldir = sprintf("%sobjects/%s/",$dstdir,$subdir); opendir(S,$sfulldir) - or die "Failed to opendir $sfulldir: $!"; + or (warn "Failed to opendir $sfulldir: $!" and return); foreach my $file (grep(!/\.{1,2}$/, readdir(S))) { my $sfilename = $sfulldir . $file; -- 1.5.4.rc2.84.gf85fd-dirty ``` ## Junio C Hamano, 2008-01-12 02:14 Subject: Re: [PATCH] git-relink.perl: warn() instead of die() on directory open failure Message-ID: <7vabnbzt6t.fsf@gitster.siamese.dyndns.org> URL: https://gitlist.dev/e/7vabnbzt6t.fsf%40gitster.siamese.dyndns.org In-Reply-To: <4787B15D.3070105@nrlssc.navy.mil> ``` Brandon Casey writes: > Signed-off-by: Brandon Casey > --- > > > Sometimes the repository to link to is not under your control. > If it contains files or unreadable directories, git-relink will > die without this patch. I am not so sure if dying is a bad behaviour, if it is because you are trying to link against an object store that you may not be able to read. I actually think we should actively refuse to, in order to prevent future problems. After seeing the command die, you will talk to the owner of that "master" object store and ask him to fix permissions (or he may choose to say "please do not share with me"). However, if the "master" object store has pruned its loose objects since you read @hashdirs out of it, that opendir() can fail, and we should silently ignore the whole $subdir. ``` ## Brandon Casey, 2008-01-12 02:26 Subject: Re: [PATCH] git-relink.perl: warn() instead of die() on directory open failure Message-ID: <4788256B.2080007@nrlssc.navy.mil> URL: https://gitlist.dev/e/4788256B.2080007%40nrlssc.navy.mil In-Reply-To: <7vabnbzt6t.fsf@gitster.siamese.dyndns.org> ``` Junio C Hamano wrote: > Brandon Casey writes: > >> Signed-off-by: Brandon Casey >> --- >> >> >> Sometimes the repository to link to is not under your control. >> If it contains files or unreadable directories, git-relink will >> die without this patch. > > I am not so sure if dying is a bad behaviour, if it is because > you are trying to link against an object store that you may not > be able to read. I actually think we should actively refuse to, > in order to prevent future problems. After seeing the command > die, you will talk to the owner of that "master" object store > and ask him to fix permissions (or he may choose to say "please > do not share with me"). The case for me was that the objects directory contained temporary pack files. This is a perfectly valid state for a git repository, but relink() currently aborts the whole effort when it encounters a non-directory. Stale tmp packs can remain if the user aborted a git command before it was finished. (hmm, maybe git-gc --prune could remove these too?) I only mentioned the unreadable directory case as another possibility. You're probably right about that one. -brandon ```