Re: [PATCH 4/5] git-svn: fix bottleneck in stash_placeholder_list()

From: Eric Wong <normalperson@yhbt.net>
Date: May 1, 2013, 21:30 UTC
Message-ID: <20130501213031.GA13056@dcvr.yhbt.net>
In-Reply-To: <455264907.20130501235104@gmail.com>

Ilya Basin <basinilya@gmail.com> wrote:

Show 5 quoted lines

> JCH> comment line "# added by git-svn only to keep the directory" and
> JCH> consider a directory that has nothing but .gitignore that consists
> JCH> of only that exact comment line an "added placeholder" directory to
> JCH> work it around.
> Sounds good, but it's not I who decided to use the config file.

Ugh, I didn't review Ray's original commit closely enough to notice this :x

Perhaps we should migrate users to use YAML storage for this, instead (we already use YAML for Git::SVN::Memoize::YAML).

Fwiw, I've never been a fan of placeholders only accepted it since it's off-by-default but it worked well enough for Ray.

My personal philosophy has always been: git svn users should leave no trace or indication they're using a non-standard SVN client.