From: "Adeodato Simó" <dato@net.com.org.es>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] contrib/workdir: create logs/refs and rr-cache in the origin repository
Date: Sun, 18 Jan 2009 12:38:30 +0100 [thread overview]
Message-ID: <20090118113830.GA1394@chistera.yi.org> (raw)
In-Reply-To: <7vbpu54cxe.fsf@gitster.siamese.dyndns.org>
* Junio C Hamano [Sat, 17 Jan 2009 17:31:57 -0800]:
> Adeodato Simó <dato@net.com.org.es> writes:
> > If logs/refs or rr-cache are dangling symlinks in the workdir, and reflogs
> > and/or rerere are enabled, commit will die with "fatal: Could not create
> > directory". (In the case of rr-cache, it will die after having created the
> > commit.)
> > This commit just creates logs/refs and rr-cache in the origin repository if
> > they don't exist already.
> Hmm, is that better than not creating the symlink of the borrowed
> repository does not have them?
I would say so. I'll agree this covers a less common case, because one
normally starts a regular repo, work on it, and at some point realize
you'd like another checkout, and create a workdir. By that point, logs/refs
surely should exist in the original repo.
However, I've as of late directly created bare repositories knowing that
I wanted to work just with workdirs against it. In this case, the logs
for each checkout'ed branch will be stored in the workdirs and not the
repo, so deleting the workdir will make you lose those logs. Which is
bad, since workdirs should always be safe to delete.
As I said, I realized this is a bit of a cornercase, but I think it
would be nice solving in the proposed way. (If you want, I can put a
shorter version of the above rationale in the commit message.)
Thanks,
--
Adeodato Simó dato at net.com.org.es
Debian Developer adeodato at debian.org
La música es de los que la quieren escuchar y de nadie más.
-- Andrés Calamaro
next prev parent reply other threads:[~2009-01-18 11:39 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-17 16:15 [PATCH] contrib/workdir: create logs/refs and rr-cache in the origin repository Adeodato Simó
2009-01-18 1:31 ` Junio C Hamano
2009-01-18 11:38 ` Adeodato Simó [this message]
2009-01-18 19:59 ` Junio C Hamano
2009-01-19 12:20 ` Adeodato Simó
2009-01-19 19:48 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090118113830.GA1394@chistera.yi.org \
--to=dato@net.com.org.es \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).