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

Fix issue #17 #18

Closed
wants to merge 2 commits into from
Closed

Fix issue #17 #18

wants to merge 2 commits into from

Conversation

domq
Copy link
Contributor

@domq domq commented Feb 7, 2015

An archeological deep dive to get git-gutter+-stage-hunks to work over tramp.

* Code pasted from the last time it existed in the tramp
  codebase (emacsmirror/tramp@ece1338),
  except s/call-process/process-file/ just like in git-gutter+-call-git

* Guard against double definition, for the heck of it
Change git-gutter+-call-git-on-current-buffer to call either
#'call-process-region or #'tramp-sh-handle-call-process-region,
#depending on context, just like git-gutter+-call-git already does
@nonsequitur
Copy link
Owner

Thanks! I've added some minor cleanups.
(Here's a diff, Github's diff is broken).
Are you okay with that? I'll push it then.

@domq
Copy link
Contributor Author

domq commented Feb 16, 2015

Sure, looks good to me. Thanks for the review!

Dominique Quatravaux
+41 79 609 4072

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants