-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle trunk branches tags in the middle of svn url #7
base: master
Are you sure you want to change the base?
Handle trunk branches tags in the middle of svn url #7
Conversation
…rabbing the part before trunk
… (ie not having to navigate to each svnexternals dir before launch
…ry where there are .git externals below the current dir - project rootdir - .git_externals - .git_externals/external1 - .git_externals/external2 ...
Hello André, I didn't get any feedback on my this pull request. If you're not happy with it, please tell me what you need for it to be accepted. Also I didn't want to navigate to each git_external in order to execute git-svn-externals-update. So I changed so that it looks any externals in or below the current dir.
|
Alternatively you could keep the |
Sorry, I don't get your comment. It's about the commit befc0f0? |
It was about your pull request post. I didn't read the commit diffs. My comment doesn't say what I meant. I propose to use I prefer this to be configurable by the user. |
Hi,
When executing your script on the following svn url : https://svn.bestofmedia.com/frontend-repository/frontendCommons/sWebserviceClientGenerator/trunk/lib/bomLibrairies
the script looses track because the sed expression deletes the "trunk" part instead of getting the part before "trunk". Here is a echo of the commande that is executed :
But also in the svn clone we should rather do something like
git svn clone https://svn.bestofmedia.com/frontend-repository/frontendCommons/sWebserviceClientGenerator -T trunk/lib/bomLibraries -b branches/lib/bomLibraries -t tags/lib/bomLibraries lib/bomLibrairies
No? Anyway that is the fix I'm submitting. Please tell me what you think
Johan