Skip to content
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

Git Diff failed #278

Closed
pbastowski opened this issue Mar 24, 2014 · 12 comments
Closed

Git Diff failed #278

pbastowski opened this issue Mar 24, 2014 · 12 comments
Labels

Comments

@pbastowski
Copy link

I clicked the diff button in the bottom pane for a file with changes and then the error was displayed.


Brackets 0.37.0-12014 (mac), Git 1.8.5.2 (Apple Git-48) & Brackets-Git 0.12.0

Git Diff failed

fatal: ambiguous argument 'docs/_custom_responsive/_default/.todo': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
@zaggino
Copy link
Member

zaggino commented Mar 24, 2014

Does it do that for all files or only this particular one?

@pbastowski
Copy link
Author

Hi Martin

Sometimes I don't even get the files listed or the diff buttons. Like right
now, when I click the git toolbar button I get an empty list.

On 24 March 2014 12:24, Martin Zagora notifications@github.com wrote:

Does it do that for all files or only this particular one?

Reply to this email directly or view it on GitHubhttps://github.com//issues/278#issuecomment-38433460
.

@zaggino
Copy link
Member

zaggino commented Mar 24, 2014

@pbastowski does this happen for a very large files? How many files do you usually have in the list?

@pbastowski
Copy link
Author

Martin,

I am working with a web app, where at any time I may have 15 to 20 files
open at the same time in Brackets. Up to 5 or 6 may have changes in them
that GIT detects. Once the error shows up the plugin stops showing the diff
buttons, but it still colors the gutter where the changes are detected.

If I reload Brackets then I get to see the diff buttons on all files once
again, but, clicking one produces the error and then it doesn't work until
I reload Brackets, and so on...

I have removed the plugin currently, because I experienced strange
behaviour with Brackets. Unfortunately I haven't written down the details
of the problems, but I need to try something to get brackets to be stable
and GIT plugin was one of the plugins I updated recently, which is when
problems with stability started.

One problem was data loss, as in, twice, my source reverted to an earlier
version mysteriously. I don't know if it has anything to do with the GIT
plugin, or any of the other plugins I updated.

Paul

On 24 March 2014 20:41, Martin Zagora notifications@github.com wrote:

@pbastowski https://github.com/pbastowski does this happen for a very
large files? How many files do you usually have in the list?

Reply to this email directly or view it on GitHubhttps://github.com//issues/278#issuecomment-38491030
.

@zaggino
Copy link
Member

zaggino commented Mar 24, 2014

Well that's unfortunate.

I'd recommend leaving our extension for now because I know we have issues when there are very large results from Git but it usually works when working on normal sized projects and files. I for example use this plugin daily even when working on main Brackets repository and everything is fine. I'm working on a fix for quite a few issues currently completely rewriting our source-code and I'd be happy if you could try out the new (rewritten) version once it's available (which may be a few weeks).

@pbastowski
Copy link
Author

I'd be very happy to try it, Martin. Let me know when it's ready :)

And thank you very much for responding so quickly in the first place.

Paul

On 24 Mar 2014, at 22:09, Martin Zagora notifications@github.com wrote:

Well that's unfortunate.

I'd recommend leaving our extension for now because I know we have issues when there are very large results from Git but it usually works when working on normal sized projects and files. I for example use this plugin daily even when working on main Brackets repository and everything is fine. I'm working on a fix for quite a few issues currently completely rewriting our source-code and I'd be happy if you could try out the new (rewritten) version once it's available (which may be a few weeks).


Reply to this email directly or view it on GitHub.

@zaggino
Copy link
Member

zaggino commented Apr 25, 2014

@pbastowski We have located a problem which was probably causing your stability issues - read more in #436. Latest version 0.13.10 is free of it and shouldn't cause any more stability problems. We have done a lot of work since 0.12.0 when this issue was raised, see https://github.com/zaggino/brackets-git/blob/master/CHANGELOG.md for details.

@pbastowski
Copy link
Author

Hi Martin.

I have just installed the latest version and will work with it next week,
when I'm back at work.

Thanks for letting me know.

Paul

On 25 April 2014 15:13, Martin Zagora notifications@github.com wrote:

@pbastowski https://github.com/pbastowski We have located a problem
which was probably causing your stability issues - read more in #436#436.
Latest version 0.13.10 is free of it and shouldn't cause any more
stability problems. We have done a lot of work since 0.12.0 when this
issue was raised, see
https://github.com/zaggino/brackets-git/blob/master/CHANGELOG.md for
details.


Reply to this email directly or view it on GitHubhttps://github.com//issues/278#issuecomment-41390654
.

@pbastowski
Copy link
Author

Hi Martin

I think I am having trouble configuring the plugin on my Brackets 38 on mac
os x. When I modify a file and then try to click on it or on the diff
button I get the attached message.

Any idea what's breaking it?

My config is (automatically set up):

Path to executable: git
Custom terminal command: /Users/paulbastowski/Library/Application
Support/Brackets/extensions/user/zaggino.brackets-git/shell/terminal.osa
Command line arguments: $1

Thanks

Paul

On 25 April 2014 15:13, Martin Zagora notifications@github.com wrote:

@pbastowski https://github.com/pbastowski We have located a problem
which was probably causing your stability issues - read more in #436#436.
Latest version 0.13.10 is free of it and shouldn't cause any more
stability problems. We have done a lot of work since 0.12.0 when this
issue was raised, see
https://github.com/zaggino/brackets-git/blob/master/CHANGELOG.md for
details.


Reply to this email directly or view it on GitHubhttps://github.com//issues/278#issuecomment-41390654
.

@zaggino
Copy link
Member

zaggino commented Apr 27, 2014

I'm afraid GitHub doesn't allow attachments to pass so you'll need to mail me to zaggino at gmail or paste the error here.

@pbastowski
Copy link
Author

​Clicking diff results in the following:

Error: fatal: ambiguous argument
'docs/_custom_responsive/_default/.todo': unknown revision or path not
in the working tree.
Use '--' to separate paths from revisions, like this:

'git [...] -- [...]'

Clicking on a file in the list of changed flies results in the following
Error message:

"Error opening file

An error has occurred when trying to open the file ......... The file could
not be found."

It's a weird error this one, because how did it detect changes to the file
in the first place? And then not be able to find it.

Thanks for your help, Martin.

Paul

On 28 April 2014 00:29, Martin Zagora notifications@github.com wrote:

I'm afraid GitHub doesn't allow attachments to pass so you'll need to mail
me to zaggino at gmail or paste the error here.


Reply to this email directly or view it on GitHubhttps://github.com//issues/278#issuecomment-41511509
.

zaggino added a commit that referenced this issue Apr 28, 2014
@zaggino
Copy link
Member

zaggino commented Apr 28, 2014

Try version 0.13.12 the error will be different.

But probably not solved - you should use the Report bug button that we have in the extension to submit bugs, it shows important information like versions used (brackets, git and this extension) and also other errors that may have happened before this.

@zaggino zaggino closed this as completed May 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants