-
-
Notifications
You must be signed in to change notification settings - Fork 933
random syntax error #257
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
Comments
If short flags are used, it is common practice for unix command to allow aggregating them like it's done here. However, I have never seen this with values. The spot where it happens is this one, which is old code written by Eric Brunson. I am sure he had his reasons to implement it that way, back in the days. To fix your issue, all you have to do is to make the call like this repo.git.commit(message="{0},{1},{2}".format(inc,vm,dateexp)) Note the long argument name. |
Works better, but I've still seen one
I suspect that sometimes it is not happy with commit messages same like previous ones. |
Something that should always work is if you commit using GitPython itself, like this: |
Well, works for me, so I think this may be closed. |
Will do. |
I found out that it's the same commit message few times in a row that results in this error. |
I tried all of the above and still can't commit anything.. even leaving out the messange and doing something like |
It's also possible to do something like this: |
My errors always lead to https://github.com/gitpython-developers/GitPython/blob/master/git/cmd.py#L440 in the stacktrace. @Norfeldt - can you confirm? I went to look into commit function code and was surprised stacktrace does not actually go into it. |
Maybe you don't have the Additional debugging can be performed by setting the |
First off all: thank you for the fast replies, sorry I haven't been able to reply with the same speed. I believe I had the same issues as PTBNL and then I implemented Billy Jin answer in my code - that worked. I rewrote the code to use gitpython again, and now it's working.. @ror6ax I believe that it was L440 that was written in the error. But I cannot confirm it, since it's working now.. (BTW I using windows 7.) |
Hi guys,
while working on a project, I met weird behaviour:
Usually it works fine. But once in a while gitpython somehow wants to run
-m
immidiately followed by message, without the space. Obviously, this fails.Any ideas on how to overcome this?
Thanks a lot,
Greg
The text was updated successfully, but these errors were encountered: