On Sat, Nov 29, 2014 at 7:17 PM, Martin Gagnon <eme...@gmail.com> wrote:
> I've just test with a svnadmin dump file created from a CVS repository > using cvs2svn. In order to make the import work, I had to remove the > part that call fossil_fatal() when not finding the UUID because cvs2svn > doesn't generate any UUID record. > > According to what I saw in import.c, the UUID is not > used in the fossil svn import code. > > Since the UUID record is not mandatory in the svnadmin dump format, I > guess we can remove UUID section ? > > What do you think? According to the svn-dump spec the UUID is mandatory, and so I checked for it. Since it is indeed not actually used (at least currently) in the fossil import, I will make it optional. I am still working on the previous comments I got, and hope to have a fixed version ready soon, but am I little busy at the moment with a new job, so it may take a while. -- ˙uʍop-ǝpısdn sı ɹoʇıuoɯ ɹnoʎ 'sıɥʇ pɐǝɹ uɐɔ noʎ ɟı
_______________________________________________ fossil-dev mailing list fossil-dev@lists.fossil-scm.org http://sqlite.org:8080/cgi-bin/mailman/listinfo/fossil-dev
