Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-24 Thread Florian Achleitner
On Tuesday 24 July 2012 14:50:49 Jonathan Nieder wrote: > > It is unclear how this is different from giving the ceiling by > > specifying it as the "END" in -rSTART:END command line. Is this > > feature really needed? > > I think the idea is that you put this script (or a symlink to it) on > your

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-24 Thread Jonathan Nieder
Hi, Junio C Hamano wrote: > Florian Achleitner writes: >> To ease testing without depending on a reachable svn server, this >> compact python script mimics parts of svnrdumps behaviour. >> It requires the remote url to start with sim://. [...] >> To allow using the same dump file for simulating

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-24 Thread Erik Faye-Lund
On Mon, Jul 23, 2012 at 2:44 PM, Florian Achleitner wrote: > + sys.exit(ret) > \ No newline at end of file Nit: add a \n after "sys.exit(ret)", perhaps? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo inf

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Junio C Hamano
Florian Achleitner writes: > To ease testing without depending on a reachable svn server, this > compact python script mimics parts of svnrdumps behaviour. > It requires the remote url to start with sim://. > Eventual slashes at the end of the url are stripped. s/ventual/xcess/ perhaps? > The u

[PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Florian Achleitner
To ease testing without depending on a reachable svn server, this compact python script mimics parts of svnrdumps behaviour. It requires the remote url to start with sim://. Eventual slashes at the end of the url are stripped. The url specifies the path of the svn dump file (as created by svnrdump)

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Jeff King
On Mon, Jul 23, 2012 at 09:46:49PM +0200, Matthieu Moy wrote: > > Damn. That's usually no problem with kmail either, if the config is right. > > I've already used git-send-email several times. > > But for replying to threads and adding several Cc: addresses it's a little > > cumbersome. > > How d

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Matthieu Moy
Florian Achleitner writes: > On Monday 23 July 2012 18:24:40 Matthieu Moy wrote: >> You also have whitespace damages (i.e. line wrapping introduced by your >> mailer). Using git-send-email avoids this kind of problem (there are >> also some advices for some mailers in Documentation/SubmittingPatc

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Florian Achleitner
On Monday 23 July 2012 18:24:40 Matthieu Moy wrote: > You also have whitespace damages (i.e. line wrapping introduced by your > mailer). Using git-send-email avoids this kind of problem (there are > also some advices for some mailers in Documentation/SubmittingPatches). Damn. That's usually no pro

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Matthieu Moy
Florian Achleitner writes: > I had to fix the missing sign-off anyways.. > > contrib/svn-fe/svnrdump_sim.py | 53 > You also have whitespace damages (i.e. line wrapping introduced by your mailer). Using git-send-email avoids this kind of problem (there

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Junio C Hamano
Florian Achleitner writes: > It requires the remote url to start with sim://. > Start and end revisions are evaluated. It is a bit unclear where "start" and "end" comes from, and if "evaluated" is the most important aspect of the handling of these two values. Do you mean the tool takes start an

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Florian Achleitner
To ease testing without depending on a reachable svn server, this compact python script mimics parts of svnrdumps behaviour. It requires the remote url to start with sim://. Start and end revisions are evaluated. If the requested revision doesn't exist, as it is the case with incremental imports, i

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Florian Achleitner
On Monday 23 July 2012 07:59:21 Jonathan Nieder wrote: > Florian Achleitner wrote: > > To ease testing without depending on a reachable svn server, this > > compact python script mimics parts of svnrdumps behaviour. > > Thanks. Mind if I forge your sign-off? Ups. No problem, anyways I've added i

Re: [PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Jonathan Nieder
Florian Achleitner wrote: > To ease testing without depending on a reachable svn server, this > compact python script mimics parts of svnrdumps behaviour. Thanks. Mind if I forge your sign-off? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord..

[PATCH] Add a svnrdump-simulator replaying a dump file for testing.

2012-07-23 Thread Florian Achleitner
To ease testing without depending on a reachable svn server, this compact python script mimics parts of svnrdumps behaviour. It requires the remote url to start with sim://. Start and end revisions are evaluated. If the requested revision doesn't exist, as it is the case with incremental imports, i