all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* 23.0.50; vc-diff needs CVS repository now
@ 2007-10-27  9:05 Nick Roberts
  2007-11-04  1:58 ` Glenn Morris
  0 siblings, 1 reply; 11+ messages in thread
From: Nick Roberts @ 2007-10-27  9:05 UTC (permalink / raw)
  To: emacs-pretest-bug


I have a copy of an unmodified file in CVS (varobj.c.~1.96~, say) and
vc-cvs-stay-local and vc-stay-local are t.  If I do vc-diff (V-x v =) in
the buffer with varobj.c (CVS:1.96 in mode-line), Emacs now tries to access the
CVS repository to compute the diff buffer, rather than using varobj.c.~1.96~.


In GNU Emacs 23.0.50.34 (i686-pc-linux-gnu, GTK+ Version 2.10.11)
 of 2007-10-25 on kahikatea.snap.net.nz
Windowing system distributor `The X.Org Foundation', version 11.0.70200000
configured using `configure  'CFLAGS=-g3''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_NZ.UTF-8
  locale-coding-system: utf-8
  default-enable-multibyte-characters: t

Major mode: VM Summary

Minor modes in effect:
  show-paren-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  unify-8859-on-encoding-mode: t
  utf-translate-cjk-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
<prior> <prior> <prior> <down-mouse-1> <mouse-1> C-s 
C-w C-w C-s C-s C-s <f5> <f5> <f5> <f5> <f5> <f5> <f5> 
<f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> 
<f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> 
<f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> <f5> 
<f5> <f5> <f5> <f5> <f5> <f5> <f6> <f6> <f6> <f6> <f6> 
<f6> <f6> <switch-frame> <down-mouse-1> <mouse-movement> 
<mouse-1> <help-echo> <switch-frame> <down-mouse-1> 
<drag-mouse-1> <down-mouse-1> <mouse-movement> <drag-mouse-1> 
<end> <down-mouse-2> <mouse-2> <down-mouse-1> <mouse-1> 
<next> <next> <next> <next> <next> <next> <next> <next> 
<prior> <help-echo> <down-mouse-1> <mouse-movement> 
<mouse-1> C-s C-w C-w C-w C-w C-w C-w C-w <help-echo> 
<switch-frame> <down-mouse-1> <mouse-movement> <mouse-1> 
<help-echo> <down-mouse-1> <mouse-movement> <drag-mouse-1> 
C-x C-b <help-echo> <help-echo> <down-mouse-2> <mouse-2> 
<down-mouse-1> <mouse-1> C-s C-s C-r <switch-frame> 
<down-mouse-1> <mouse-1> <help-echo> <switch-frame> 
<down-mouse-1> <mouse-1> <help-echo> <switch-frame> 
<down-mouse-1> <mouse-1> <help-echo> <help-echo> <switch-frame> 
<down-mouse-1> <mouse-1> <help-echo> <help-echo> <down-mouse-1> 
<mouse-movement> <mouse-1> C-x C-v C-g <down-mouse-1> 
<mouse-movement> <mouse-1> C-x C-b <help-echo> <help-echo> 
<help-echo> <down-mouse-2> <mouse-2> <help-echo> <down-mouse-1> 
<mouse-1> <help-echo> C-s C-w C-w C-w C-w C-w <down-mouse-1> 
<mouse-1> C-s C-s C-r <down-mouse-1> <mouse-movement> 
<mouse-movement> <drag-mouse-1> C-w <down-mouse-1> 
<mouse-movement> <mouse-movement> <drag-mouse-1> C-w 
<down-mouse-2> <mouse-2> <down-mouse-1> <mouse-1> <tab> 
<down-mouse-1> <mouse-movement> <mouse-movement> <drag-mouse-1> 
C-w <down-mouse-1> <drag-mouse-1> <down-mouse-1> <mouse-movement> 
<mouse-movement> <drag-mouse-1> C-w <return> C-x C-s 
<down-mouse-1> <mouse-1> <tab> <down-mouse-1> <mouse-1> 
<tab> C-x C-s <help-echo> <down-mouse-1> <mouse-movement> 
<mouse-movement> <mouse-1> <help-echo> <help-echo> 
<switch-frame> <down-mouse-1> <mouse-movement> <mouse-1> 
<down-mouse-1> <mouse-1> <f5> <f5> <f5> <switch-frame> 
<down-mouse-1> <mouse-1> <help-echo> <help-echo> <help-echo> 
<down-mouse-1> <mouse-movement> <mouse-1> <help-echo> 
<down-mouse-1> <mouse-1> <help-echo> <down-mouse-1> 
<mouse-1> C-x C-b <help-echo> <help-echo> <help-echo> 
<down-mouse-2> <mouse-2> C-x v = <help-echo> <help-echo> 
<help-echo> <help-echo> <down-mouse-2> <mouse-2> <help-echo> 
<down-mouse-1> <mouse-1> <down-mouse-1> <mouse-movement> 
<mouse-1> <down-mouse-1> <mouse-1> C-h v c <backspace> 
v c SPC c v s SPC s t a SPC SPC <return> <help-echo> 
<switch-frame> <down-mouse-1> <mouse-1> M-x r e p o 
p <backspace> r t SPC e m a SPC SPC <return>

Recent messages:
Mark saved where search started [2 times]
Quit
Mark saved where search started [2 times]
Mark set
Saving file /home/nickrob/src/gdb/varobj.c...
Wrote /home/nickrob/src/gdb/varobj.c
Saving file /home/nickrob/src/gdb/varobj.c...
Wrote /home/nickrob/src/gdb/varobj.c
Finding changes in /home/nickrob/src/gdb/testsuite/gdb.mi/mi-var-cmd.exp...
Type C-x 1 to remove help window.  
C-x v = runs the command vc-diff

-- 
Nick                                           http://www.inet.net.nz/~nickrob

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-10-27  9:05 23.0.50; vc-diff needs CVS repository now Nick Roberts
@ 2007-11-04  1:58 ` Glenn Morris
  2007-11-04 17:43   ` Eric S. Raymond
  0 siblings, 1 reply; 11+ messages in thread
From: Glenn Morris @ 2007-11-04  1:58 UTC (permalink / raw)
  To: esr; +Cc: Emacs developers


Any comment on this? The part of vc-diff-internal that does a local
diff where possible seems to have disappeared in the 2007-10-10 change
"rewrite for filesets".

Nick Roberts wrote:

> I have a copy of an unmodified file in CVS (varobj.c.~1.96~, say)
> and vc-cvs-stay-local and vc-stay-local are t. If I do vc-diff (V-x
> v =) in the buffer with varobj.c (CVS:1.96 in mode-line), Emacs now
> tries to access the CVS repository to compute the diff buffer,
> rather than using varobj.c.~1.96~.
>
>
> In GNU Emacs 23.0.50.34 (i686-pc-linux-gnu, GTK+ Version 2.10.11)
>  of 2007-10-25 on kahikatea.snap.net.nz

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-04  1:58 ` Glenn Morris
@ 2007-11-04 17:43   ` Eric S. Raymond
  2007-11-05  4:37     ` Nick Roberts
  2007-11-11 20:54     ` Dan Nicolaescu
  0 siblings, 2 replies; 11+ messages in thread
From: Eric S. Raymond @ 2007-11-04 17:43 UTC (permalink / raw)
  To: Glenn Morris; +Cc: esr, Emacs developers

Glenn Morris <rgm@gnu.org>:
> Any comment on this? The part of vc-diff-internal that does a local
> diff where possible seems to have disappeared in the 2007-10-10 change
> "rewrite for filesets".

That's right, because the front end was the wrong place to do it.  
Modern VCS diff tools automatically stay local when they can.
Subversion diff does the right thing, and it's not even an issue
in the 3G systems (they keep local repositories).  It's the
kind of variation that should be handled in the VCS-specific back ends.

Actually CVS is the only awkward case.  So what ought to happen is
for something like the old stay-local logic to be implemented in the CVS 
back end.

This is one of my two remaining to-do items for the mode.  But someone
who is more of a CVS expert than I might get it done faster.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-04 17:43   ` Eric S. Raymond
@ 2007-11-05  4:37     ` Nick Roberts
  2007-11-05 16:18       ` Eric S. Raymond
  2007-11-11 20:54     ` Dan Nicolaescu
  1 sibling, 1 reply; 11+ messages in thread
From: Nick Roberts @ 2007-11-05  4:37 UTC (permalink / raw)
  To: esr; +Cc: Glenn Morris, Emacs developers

 > This is one of my two remaining to-do items for the mode.

Perhaps you could share the other to-do item with us so that we know what
else doesn't work.

-- 
Nick                                           http://www.inet.net.nz/~nickrob

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-05  4:37     ` Nick Roberts
@ 2007-11-05 16:18       ` Eric S. Raymond
  2007-11-06  8:38         ` Richard Stallman
  0 siblings, 1 reply; 11+ messages in thread
From: Eric S. Raymond @ 2007-11-05 16:18 UTC (permalink / raw)
  To: Nick Roberts; +Cc: Glenn Morris, Emacs developers

Nick Roberts <nickrob@snap.net.nz>:
>  > This is one of my two remaining to-do items for the mode.
> 
> Perhaps you could share the other to-do item with us so that we know what
> else doesn't work.

The only other one is a minor item concerning what X-v = does when called
on a buffer visiting a file not under version control, but in a directory
which does have one or more responsible backends.  RMS and Stefan have 
a theory about how they want this to work, but I am still not certain
the theory is sound.  I need to take a hard look at this.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-05 16:18       ` Eric S. Raymond
@ 2007-11-06  8:38         ` Richard Stallman
  0 siblings, 0 replies; 11+ messages in thread
From: Richard Stallman @ 2007-11-06  8:38 UTC (permalink / raw)
  To: esr; +Cc: rgm, nickrob, emacs-devel

    The only other one is a minor item concerning what X-v = does when called
    on a buffer visiting a file not under version control, but in a directory
    which does have one or more responsible backends.

What does it do now in that case?

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-04 17:43   ` Eric S. Raymond
  2007-11-05  4:37     ` Nick Roberts
@ 2007-11-11 20:54     ` Dan Nicolaescu
  2007-11-11 21:06       ` Eric S. Raymond
                         ` (2 more replies)
  1 sibling, 3 replies; 11+ messages in thread
From: Dan Nicolaescu @ 2007-11-11 20:54 UTC (permalink / raw)
  To: esr; +Cc: Glenn Morris, Nick Roberts, Emacs developers

"Eric S. Raymond" <esr@thyrsus.com> writes:

  > Glenn Morris <rgm@gnu.org>:
  > > Any comment on this? The part of vc-diff-internal that does a local
  > > diff where possible seems to have disappeared in the 2007-10-10 change
  > > "rewrite for filesets".
  > 
  > That's right, because the front end was the wrong place to do it.  
  > Modern VCS diff tools automatically stay local when they can.
  > Subversion diff does the right thing, and it's not even an issue
  > in the 3G systems (they keep local repositories).  It's the
  > kind of variation that should be handled in the VCS-specific back ends.
  > 
  > Actually CVS is the only awkward case.  So what ought to happen is
  > for something like the old stay-local logic to be implemented in the CVS 
  > back end.
  > 
  > This is one of my two remaining to-do items for the mode.  But someone
  > who is more of a CVS expert than I might get it done faster.

I adapted the old vc-diff-internal logic to vc-cvs-diff. After the
patch below the CVS repository is not needed if backup files are
present.

Not sure what is the "correct" return value for vc-cvs-diff in the
case when the CVS repository is not accessed (i.e. when using plain
"diff" instead of "cvs diff").

*** vc-cvs.el.~1.93.~	Fri Nov  9 22:02:03 2007
--- vc-cvs.el	Sun Nov 11 12:38:37 2007
***************
*** 515,526 ****
    "Get a difference report using CVS between two revisions of FILE."
      (let* ((async (and (not vc-disable-async-diff) 
  		       (vc-stay-local-p files)))
! 	   (status (apply 'vc-cvs-command (or buffer "*vc-diff*")
! 			  (if async 'async 1)
! 			  files "diff"
! 			  (and oldvers (concat "-r" oldvers))
! 			  (and newvers (concat "-r" newvers))
! 			  (vc-switches 'CVS 'diff))))
        (if async 1 status)))		; async diff, pessimistic assumption
  
  (defun vc-cvs-diff-tree (dir &optional rev1 rev2)
--- 515,554 ----
    "Get a difference report using CVS between two revisions of FILE."
      (let* ((async (and (not vc-disable-async-diff) 
  		       (vc-stay-local-p files)))
! 	   (invoke-cvs-diff-list nil)
! 	   status)
!       ;; Look through the file list and see if any files have backups
!       ;; that can be used to do a plain "diff" instead of "cvs diff".
!       (dolist (file files)
! 	(let ((ov oldvers)
! 	      (nv newvers))
! 	  (when (or (not ov) (string-equal ov ""))
! 	    (setq ov (vc-workfile-version file)))
! 	  (when (string-equal nv "")
! 	    (setq nv nil))
! 	  (let ((file-oldvers (vc-version-backup-file file ov))
! 		(file-newvers (if (not nv)
! 				  file
! 				(vc-version-backup-file file nv)))
! 		(coding-system-for-read (vc-coding-system-for-diff file)))
! 	    (if (and file-oldvers file-newvers)
! 		(apply 'vc-do-command (or buffer "*vc-diff*") 1 "diff" nil
! 		       (append (if (listp diff-switches)
! 				   diff-switches
! 				 (list diff-switches))
! 			       (if (listp vc-diff-switches)
! 				   vc-diff-switches
! 				 (list vc-diff-switches))
! 			       (list (file-relative-name file-oldvers)
! 				     (file-relative-name file-newvers))))
! 	      (push file invoke-cvs-diff-list)))))
!       (when invoke-cvs-diff-list
! 	(setq status (apply 'vc-cvs-command (or buffer "*vc-diff*")
! 			    (if async 'async 1)
! 			    invoke-cvs-diff-list "diff"
! 			    (and oldvers (concat "-r" oldvers))
! 			    (and newvers (concat "-r" newvers))
! 			    (vc-switches 'CVS 'diff))))
        (if async 1 status)))		; async diff, pessimistic assumption
  
  (defun vc-cvs-diff-tree (dir &optional rev1 rev2)

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-11 20:54     ` Dan Nicolaescu
@ 2007-11-11 21:06       ` Eric S. Raymond
  2007-11-12  3:56         ` Dan Nicolaescu
  2007-11-12 20:05       ` Nick Roberts
  2007-11-12 21:47       ` Stefan Monnier
  2 siblings, 1 reply; 11+ messages in thread
From: Eric S. Raymond @ 2007-11-11 21:06 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: Glenn Morris, Nick Roberts, Emacs developers

Dan Nicolaescu <dann@ics.uci.edu>:
> I adapted the old vc-diff-internal logic to vc-cvs-diff. After the
> patch below the CVS repository is not needed if backup files are
> present.

Thanks, Dan.  If you have not yet committed this, you have my encouragement
to do so.
 
> Not sure what is the "correct" return value for vc-cvs-diff in the
> case when the CVS repository is not accessed (i.e. when using plain
> "diff" instead of "cvs diff").

Hmmm...probably 0.  But even if that's wrong, your logic is still an
improvement.
-- 
		<a href="http://www.catb.org/~esr/">Eric S. Raymond</a>

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-11 21:06       ` Eric S. Raymond
@ 2007-11-12  3:56         ` Dan Nicolaescu
  0 siblings, 0 replies; 11+ messages in thread
From: Dan Nicolaescu @ 2007-11-12  3:56 UTC (permalink / raw)
  To: esr; +Cc: Glenn Morris, Nick Roberts, Emacs developers

"Eric S. Raymond" <esr@thyrsus.com> writes:

  > Dan Nicolaescu <dann@ics.uci.edu>:
  > > I adapted the old vc-diff-internal logic to vc-cvs-diff. After the
  > > patch below the CVS repository is not needed if backup files are
  > > present.
  > 
  > Thanks, Dan.  If you have not yet committed this, you have my encouragement
  > to do so.

Thanks. Done.

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-11 20:54     ` Dan Nicolaescu
  2007-11-11 21:06       ` Eric S. Raymond
@ 2007-11-12 20:05       ` Nick Roberts
  2007-11-12 21:47       ` Stefan Monnier
  2 siblings, 0 replies; 11+ messages in thread
From: Nick Roberts @ 2007-11-12 20:05 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: esr, Glenn Morris, Emacs developers

 > I adapted the old vc-diff-internal logic to vc-cvs-diff. After the
 > patch below the CVS repository is not needed if backup files are
 > present.

This change looks good to me, so I've removed the corresponding item in
FOR-RELEASE.

-- 
Nick                                           http://www.inet.net.nz/~nickrob

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

* Re: 23.0.50; vc-diff needs CVS repository now
  2007-11-11 20:54     ` Dan Nicolaescu
  2007-11-11 21:06       ` Eric S. Raymond
  2007-11-12 20:05       ` Nick Roberts
@ 2007-11-12 21:47       ` Stefan Monnier
  2 siblings, 0 replies; 11+ messages in thread
From: Stefan Monnier @ 2007-11-12 21:47 UTC (permalink / raw)
  To: Dan Nicolaescu; +Cc: esr, Glenn Morris, Nick Roberts, Emacs developers

> I adapted the old vc-diff-internal logic to vc-cvs-diff.  After the
> patch below the CVS repository is not needed if backup files are
> present.

Thanks.  It might be a good occasion to fix a bug in VC: backup files
should only be used this way if they refer to a precise revision number,
but not if they refer to a tag or (worse) a branch, since tags and
branches can change.


        Stefan

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

end of thread, other threads:[~2007-11-12 21:47 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-27  9:05 23.0.50; vc-diff needs CVS repository now Nick Roberts
2007-11-04  1:58 ` Glenn Morris
2007-11-04 17:43   ` Eric S. Raymond
2007-11-05  4:37     ` Nick Roberts
2007-11-05 16:18       ` Eric S. Raymond
2007-11-06  8:38         ` Richard Stallman
2007-11-11 20:54     ` Dan Nicolaescu
2007-11-11 21:06       ` Eric S. Raymond
2007-11-12  3:56         ` Dan Nicolaescu
2007-11-12 20:05       ` Nick Roberts
2007-11-12 21:47       ` Stefan Monnier

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.