-
Notifications
You must be signed in to change notification settings - Fork 443
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
Command failed (output redirection?) #132
Comments
use --verbose to see what is wrong |
|
I am seeing the same. This indicates it might be related to changes after git 1.8.1.3. http://stackoverflow.com/questions/17852249/svn2git-cannot-setup-tracking-information |
Possibly related to this in branch.c? static int check_tracking_branch(struct remote *remote, void *cb_data) static int validate_remote_tracking_branch(char *ref) From a diff of v1.8.1.3 and v1.8.3.4. Looks like it verifies whether the branch exists on the remote but we don't have a remote yet after the svn fetch. This was introduced in git 1.8.3.2 |
Confirmed that it all works with git 1.8.3.1. That doesn't mean the code I pointed to is the problem, but it's a strong suggestion. |
Having the same issue with git 1.8.4. |
#136 sounds related. |
Confirming that downgrading git to 1.8.3.1 worked for me as well. |
Same problem here... Git 1.8.5... |
I have this issue with git version 1.8.3.2. |
Changing this line https://github.com/nirvdrum/svn2git/blob/master/lib/svn2git/migration.rb#L319 At least it works now for me without any issues. you can find the |
Thanks. I need to do a better job at version detection. We're supporting a wide swath of git versions, many with slightly differing behavior. |
a release soon would be really cool :) Thanks for the fix! |
I'm planning on one this weekend. |
Looking forward to a release. |
If the fix released yet? And if so how do I update? I've tried running |
No. I'm still in the midst of verifying on older git releases. |
@ava-dylang you can fix this also manually |
@DanielRuf Thanks; I had got it working, I had too. Thanks to your for listing the files to change! Saved me so much undue headache. <3 |
@ava-dylang No problem, this is also the fix, which will be applied to the next release. |
The fix proposed by DanielRuf worked fine for me. I'm on git version 1.8.3.4 (Apple Git-47) |
I bumped into this too and fixed it manually. Thanks for your work on this tool! |
Broken on git 1.9.1 as well (stock git from Ubuntu repos). |
Just echoing, this fix worked for me, please release. 👍 |
@DanielRuf thanks! |
@erickertz you are welcome =) |
I had this problem on Ubuntu with Git 1.9.3. and can confirm that the fix from @DanielRuf works there too. 👍 |
I had this problem on OSX 10.9.2 with git 1.9.3 as well. Can confirm fix from @DanielRuf works here too. Fix should be released, github's official "importing svn into github" page recommends svn2git - experiencing errors with the most updated versions of everything has got to be turning a lot of people away who didn't do half an hour of searching to find this ticket. :) |
I'm working through several tickets right now. Making sure it works on all 3 LTS Ubuntu releases, Mac, and Windows, with system rubies and JRuby is a bit time-consuming. Almost every time I merge a PR, it ends up breaking someone else, so I'm being more conservative about that now until I can verify fixes work across the board. Trust me, I do want to get this fixed. |
Maybe you can ask GitHub to help with the testing =O |
I think this fix should work for all users. |
I just got the 2.3.0 release out, which required some hairy process management to support passing STDIN through the underlying git-svn process. That should free me up to look at this shortly. |
Okay, so as far as I can tell this issue got hijacked. @technoskald's original problem seems to have nothing to do with all the follow-on comments. I'm trying to address both but that does make resolving this one a bit trickier. |
For more of the history, it appears this behavior changed in in git 1.8.3.2. I happen to disagree with the change, but I understand the rationale. Tracking a remote is still valid when pulling changes. In any event, the proposed change here is not semantically equivalent and we have users that are using svn2git to basically keep git forks in sync. So, I'm reticent to just blindly make the change. I'll try to come up with something that works in both cases. FYI, it appears this may break again when git 2.0 comes out. |
I just pushed 2.3.1 out which which should fix the problem everyone but @technoskald was reporting. I still need to get to the bottom of that one, but it's looking like it may be either git-svn related or repository-layout related. If you're using git < 1.8.3.2, the semantics are retained (tracking information is established), but you'll see a deprecation message. If you're on git >= 1.8.3.2, branches should be created fine now. If you find otherwise, please open a new issue with details. Thanks. |
For my own sanity, I filed #165 for the issue originally reported here. For anyone affected by that, please follow the new issue. I'll be closing this one, treating it as related to tracking remote SVN branches. |
With git 1.9.1 and svn2git 2.3.2, the bug mentioned in post 3 does not seem to be fixed. Maybe a regression? I just ran A possible reason why the bug was not fixed might be that git's error message is in my native language (French). Since the fix seems to rely on git's error messages, maybe svn2git should take care of setting git's language before executing commands? |
Isn't there Git 2.8 already? Git 1.9.1 is from 18-Mar-2014. See here for a working workaround (should fix this): #132 (comment) References, that also Git 1.9.x is affected: #132 (comment) This was changed in Git 1.8.x This is still a bug in svn2git which has to use the new syntax for tracking branches and which was changed in Git 1.8.x. Unfotunately there is no new release of svn2git, so you could try using the master branch or fix the file manually with the aforementioned workaround (well, it's mor elike the right fix of this bug / incompability with new Git versions). So instead of using an outdated and insecure version of Git, just fix the mentioned file in svn2git and it should be fine. I hope this fixes your issue. |
@DanielRuf Thanks for you answer. Git 1.9.1 is the current version of Git on Ubuntu LTS (trusty). But the new LTS will be released in the following days, as every 18 months, so yes, it's really old. See http://packages.ubuntu.com/fr/trusty/git As for the issue:
So, to sum up, everything fine with both the fix released in 2.3.1 and the workaround of changing the environment variable. I was just warning the developer (@nirvdrum?) that his fix does not seem to work in non-english environments. |
Possibly makes sense to open a new issue for this problem related to the system language. The proposed fix using But even if my system language is in German I use Git in English and so I never faced or noticed this problem, I'm fine with English ;-) The possible usage of It seems this issue is still in the master branch, at least I got no result when I tried grepping for I think @nirvdrum is quite busy with JRuby and development for svn2git stalled. There are still many open PRs and issues. Documenting this as a separate issue with the workaround using |
I've resolved this issue within a new release in svn2git.net. In case anyone wants it, just download the latest release from https://github.com/mazong1123/svn2gitnet/releases |
Got the following error:
The text was updated successfully, but these errors were encountered: