Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-20 Thread Florian Achleitner
On Monday 20 August 2012 09:45:30 Jonathan Nieder wrote: > Florian Achleitner wrote: > > Currently, the mark number is equal to the svn revision number the commit > > corresponds to. I didn't want to break that, but not mandatory. We could > > also split the mark namespace by reserving one or more

Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-20 Thread Jonathan Nieder
Florian Achleitner wrote: > Currently, the mark number is equal to the svn revision number the commit > corresponds to. I didn't want to break that, but not mandatory. We could also > split the mark namespace by reserving one or more of the most significant bits > as a type specifier. > I'll devel

Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-20 Thread Florian Achleitner
On Sunday 19 August 2012 23:57:23 Junio C Hamano wrote: > Florian Achleitner writes: > >> This change makes me uncomfortable. > >> We are doubling up on hashing with fast-import. > >> This introduces git-specific logic into vcs-svn. > > IIUC, vcs-svn/fast-export is meant to produce a stream in th

Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-19 Thread Junio C Hamano
Florian Achleitner writes: >> This change makes me uncomfortable. >> We are doubling up on hashing with fast-import. >> This introduces git-specific logic into vcs-svn. IIUC, vcs-svn/fast-export is meant to produce a stream in the fast-import format, and that format is meant to be VCS agnostic,

Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-19 Thread Florian Achleitner
On Sunday 19 August 2012 04:37:35 David Michael Barr wrote: > On Sat, Aug 18, 2012 at 6:40 AM, Florian Achleitner > > wrote: > > Hi! > > > > This patch series should prepare vcs-svn/svndump.* for branch > > detection. When starting with this feature I found that the existing > > functions are no

Re: [RFC 1/5] GSOC: prepare svndump for branch detection

2012-08-18 Thread David Michael Barr
On Sat, Aug 18, 2012 at 6:40 AM, Florian Achleitner wrote: > Hi! > > This patch series should prepare vcs-svn/svndump.* for branch > detection. When starting with this feature I found that the existing > functions are not yet appropriate for that. > These rewrites the node handling part of svndump