unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#6137: [Yuriy Vostrikov] emacs, vc-git
       [not found] <jwvr5lnjvmk.fsf-monnier+emacs@gnu.org>
@ 2010-05-07 21:39 ` Glenn Morris
  2012-11-30  7:43 ` Chong Yidong
  1 sibling, 0 replies; 2+ messages in thread
From: Glenn Morris @ 2010-05-07 21:39 UTC (permalink / raw)
  To: 6137; +Cc: delamonpansie


[ Resending from help-debbugs, where it ended up. Please specify a
package if you want to use the submit@debbugs address to create a new
report.]


Stefan Monnier wrote:
  
  [ I do not use Git enough to be sure if his patch is the right way to
    fix it.  It does look OK, tho maybe the right (but harder) place to
    fix it is in Tramp.  Can someone take a look and install it?  --Stef  ]
  
  Hello.
  You are last commiter of vc-git.el so i'm writing to you.
  
  It's probably a good idea to forcefully disable pager then calling git
  command from vc-git.el.
  If buffer is tramp controlled then git will be run on remote host by
  tramp magic. In this particular case git stdout will be connected to
  pty instead of pipe and default git behavior to call $PAGER in this
  case.
  Which is effectively makes vc-git broken on tramp controlled buffers.
  Patch is attached.
  
  
  === modified file 'lisp/vc-git.el'
  --- lisp/vc-git.el	2010-04-21 02:05:24 +0000
  +++ lisp/vc-git.el	2010-05-05 09:28:56 +0000
  @@ -966,7 +966,7 @@
   (defun vc-git-command (buffer okstatus file-or-list &rest flags)
     "A wrapper around `vc-do-command' for use in vc-git.el.
   The difference to vc-do-command is that this function always invokes `git'."
  -  (apply 'vc-do-command (or buffer "*vc*") okstatus "git" file-or-list flags))
  +    (apply 'vc-do-command (or buffer "*vc*") okstatus "git" file-or-list (cons "--no-pager" flags)))
   
   (defun vc-git--empty-db-p ()
     "Check if the git db is empty (no commit done yet)."






^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#6137: [Yuriy Vostrikov] emacs, vc-git
       [not found] <jwvr5lnjvmk.fsf-monnier+emacs@gnu.org>
  2010-05-07 21:39 ` bug#6137: [Yuriy Vostrikov] emacs, vc-git Glenn Morris
@ 2012-11-30  7:43 ` Chong Yidong
  1 sibling, 0 replies; 2+ messages in thread
From: Chong Yidong @ 2012-11-30  7:43 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Yuriy Vostrikov, 6137-done

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> [ I do not use Git enough to be sure if his patch is the right way to
>   fix it.  It does look OK, tho maybe the right (but harder) place to
>   fix it is in Tramp.  Can someone take a look and install it?  --Stef  ]
>
> Hello.
> You are last commiter of vc-git.el so i'm writing to you.
>
> It's probably a good idea to forcefully disable pager then calling git
> command from vc-git.el.  If buffer is tramp controlled then git will
> be run on remote host by tramp magic. In this particular case git
> stdout will be connected to pty instead of pipe and default git
> behavior to call $PAGER in this case.  Which is effectively makes
> vc-git broken on tramp controlled buffers.  Patch is attached.

Since no one's done anything about this, and the fix looks OK, I've
committed it to trunk.





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-11-30  7:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <jwvr5lnjvmk.fsf-monnier+emacs@gnu.org>
2010-05-07 21:39 ` bug#6137: [Yuriy Vostrikov] emacs, vc-git Glenn Morris
2012-11-30  7:43 ` Chong Yidong

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).