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