Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-05 Thread C. Michael Pilato
On 02/05/2014 05:45 AM, Julian Foad wrote: > C. Michael Pilato wrote: > >> Mostly looking >> for general feedback on the idea of the "sounds useful if it works" or >> "you *really* don't want to go there" variety, but deeper review is of >> course welcome, too. > > It seems a reasonable enhanceme

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-05 Thread Julian Foad
Daniel Shahaf wrote: > C. Michael Pilato wrote: >> Introduce '--ignore-dates' option for 'svnadmin load', > > [...] However, 'load --ignore-dates' has another use-case: preventing > the "-r {DATE} doesn't work as intended" syndrome when loading a > project's history into an existing repository.

RE: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-05 Thread Bert Huijben
Fuhrmann [mailto:stefan.fuhrm...@wandisco.com] Sent: woensdag 5 februari 2014 06:08 To: Branko Čibej Cc: Subversion Development Subject: Re: [PATCH][RFC] svnadmin load --ignore-dates On Wed, Feb 5, 2014 at 6:03 AM, Stefan Fuhrmann mailto:stefan.fuhrm...@wandisco.com> > wrote: On Wed, Feb 5, 201

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-05 Thread Julian Foad
C. Michael Pilato wrote: > Mostly looking > for general feedback on the idea of the "sounds useful if it works" or > "you *really* don't want to go there" variety, but deeper review is of > course welcome, too. It seems a reasonable enhancement to me. Simple and to the point. (A couple of style

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-04 Thread Stefan Fuhrmann
On Wed, Feb 5, 2014 at 6:03 AM, Stefan Fuhrmann < stefan.fuhrm...@wandisco.com> wrote: > On Wed, Feb 5, 2014 at 4:02 AM, Branko Čibej wrote: > >> On 05.02.2014 03:10, Daniel Shahaf wrote: >> >> However, 'load --ignore-dates' has another use-case: preventing the "-r >> {DATE} doesn't work as inte

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-04 Thread Stefan Fuhrmann
On Wed, Feb 5, 2014 at 4:02 AM, Branko Čibej wrote: > On 05.02.2014 03:10, Daniel Shahaf wrote: > > However, 'load --ignore-dates' has another use-case: preventing the "-r > {DATE} doesn't work as intended" syndrome when loading a project's history > into an existing repository. > > > I would ar

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-04 Thread Branko Čibej
On 05.02.2014 03:10, Daniel Shahaf wrote: > However, 'load --ignore-dates' has another use-case: preventing the > "-r {DATE} doesn't work as intended" syndrome when loading a project's > history into an existing repository. I would argue that this is an implementation bug, pure and simple. The fac

Re: [PATCH][RFC] svnadmin load --ignore-dates

2014-02-04 Thread Daniel Shahaf
C. Michael Pilato wrote on Tue, Feb 04, 2014 at 17:13:12 -0500: > Didn't want to commit this directly just yet, given my somewhat > disconnected attention level in the project as of late. Mostly looking > for general feedback on the idea of the "sounds useful if it works" or > "you *really* don't

[PATCH][RFC] svnadmin load --ignore-dates

2014-02-04 Thread C. Michael Pilato
Didn't want to commit this directly just yet, given my somewhat disconnected attention level in the project as of late. Mostly looking for general feedback on the idea of the "sounds useful if it works" or "you *really* don't want to go there" variety, but deeper review is of course welcome, too.