Re: git-svn and mergeinfo

From: Carlos Martín Nieto <cmn@elego.de>
Date: Sep 6, 2011, 14:28 UTC
Message-ID: <1315319309.9839.13.camel@bee.lab.cmartin.tk>
In-Reply-To: <20110906095256.205dd5d0@robyn.woti.com>

On Tue, 2011-09-06 at 09:52 -0400, Bryan Jacobs wrote:

Show 16 quoted lines

> On Tue, 06 Sep 2011 14:56:38 +0200
> Carlos Martín Nieto <cmn@elego.de> wrote:
> 
> > You can also save the mergeinfo to a file, add the line, and use
> > --mergeinfo=$(cat /tmp/some-file) to set it. It is indeed awkward, but
> > blindly replacing every space with a newline is not always the right
> > option. If a merged directory contains a space, this change will break
> > the mergeinfo, even if you're properly quoting your variable or using
> > the $(cat /some/file) method.
> > 
> > Cheers,
> >    cmn
> 
> Ah, a situation I neglected to consider! Perhaps we should revert this
> patch, since I worked up the initiative to write an
> auto-populating-mergeinfo patch for git-svn anyhow.

As it can cause regressions, I think reverting is the right option. And since git-svn is going to learn to do it by itself, the functionality isn't a big loss.

Cheers,
   cmn