Our SVN checkout URLs always append /trunk since that is a common pattern, but it doesn't work with all projects. Also, it doesn't work on new empty repos.
We could check to see if 'trunk' is present and if not, omit the 'trunk' append. Additionally, it could be nice to let admins customize the checkout path to whatever they want.
Add SVN config setting for default checkout path, and default it to /trunk. And for new SNV repos, do an initial commit to setup /trunk, /branches, /tags.
There isn't a good way to make this configurable since our config options are editable after the tool is created, so I just made this create /trunk /tags and /branches in the repo init step so the checkout url makes sense.
I think a config option is still useful, it's not a problem for admins to set that after the repo is created, in case they remove trunk/tags/branches and use something else. E.g. https://sourceforge.net/adobe/cairngorm/code/
Please also add an option to customize what gets displayed for the checkout url in svn. (and hg/git if it makes sense)
I added the option. There's a sog ticket to change the ini file since we had trunk hardcoded in there. https://control.siteops.geek.net/sog/trac/ticket/19127
Seems to work ok, but I'm concerned about the changed asserts in the clone_from method. I think the svnsync calls should return 0 on success, which means the clone_from test is failing for some reason.
If svnsync returns # of commits on the source repo, we still have a problem, since the code is now asserting that we'll never clone a repo with more than 1 commit.
I figured out that svn didn't want to clone into a repo that already had data, so for clones I made it not set up the default dirs.
Merged to dev.