From: Dan Nicolaescu <dann@ics.uci.edu>
To: "Marshall\, Simon" <Simon.Marshall@misys.com>
Cc: Andre Spiegel <spiegel@gnu.org>, emacs-devel@gnu.org
Subject: Re: vc-revert-hook?
Date: Thu, 08 May 2008 08:51:12 -0700 [thread overview]
Message-ID: <200805081551.m48FpDlm012001@sallyv1.ics.uci.edu> (raw)
In-Reply-To: <6EE216E1AA959543A555C60FF34FB767040F7D64@maileube01.misys.global.ad> (Simon Marshall's message of "Thu, 8 May 2008 09:09:42 +0100")
"Marshall, Simon" <Simon.Marshall@misys.com> writes:
> > > Without this, CVS (my backend VCS) or vc.el would leave the
> reverted source
> > > file with the timestamp of the reverted revision, ie, older than
> the modified
> > > file, and a dependency-based recompilation would be fooled into
> thinking that
> > > the source file's compiled object file was up-to-date.
> >
> > Are you sure? A quick experiment showed that and after C-x v u the
> file
> > on disk had the time stamp set to the current time...
> > Can you replicate the problem with emacs -Q?
>
> Sure, with CVS:
>
> risksun2 104> ls -l Makefile
> -r--r--r-- 1 marshals gcms 707 May 2 11:33 Makefile
> risksun2 105> pemacs -Q Makefile&
> [1] 12298
>
> Where pemacs is my 22.2 + patches to cc-mode from Alan.
> In emacs, do C-x v v on Makefile, edit, save and:
>
> risksun2 106> ls -l Makefile
> -rw-r--r-- 1 marshals gcms 708 May 8 08:50 Makefile
>
> No surprise there.
> In emacs, do C-x v u on Makefile, confirm and:
>
> risksun2 107> ls -l Makefile
> -r--r--r-- 1 marshals gcms 707 May 2 11:33 Makefile
>
> As you can see, the original timestamp is restored on reversion.
>
> risksun2 108> env|grep CVS
> CVSROOT=/repo/cvs
> CVSREAD=read-only
> risksun2 109> cvs -v
I followed the same steps and for me the time stamp got set to the
current time after revert. The only difference is the cvs version:
cvs -v
Concurrent Versions System (CVS) 1.11.22 (client/server)
Copyright (C) 2006 Free Software Foundation, Inc.
This is what comes by default with Fedora 8.
next prev parent reply other threads:[~2008-05-08 15:51 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-07 9:12 vc-revert-hook? Marshall, Simon
2008-05-07 18:29 ` vc-revert-hook? Dan Nicolaescu
2008-05-08 8:09 ` vc-revert-hook? Marshall, Simon
2008-05-08 15:51 ` Dan Nicolaescu [this message]
2008-05-08 19:16 ` vc-revert-hook? Marshall, Simon
2008-05-09 1:09 ` vc-revert-hook? Stefan Monnier
2008-05-12 10:14 ` vc-revert-hook? Marshall, Simon
2008-05-12 14:33 ` vc-revert-hook? Stefan Monnier
2008-05-12 16:18 ` vc-revert-hook? Marshall, Simon
2008-05-12 17:03 ` vc-revert-hook? Stefan Monnier
2008-05-13 9:06 ` vc-revert-hook? Marshall, Simon
2008-05-13 14:19 ` vc-revert-hook? Stefan Monnier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200805081551.m48FpDlm012001@sallyv1.ics.uci.edu \
--to=dann@ics.uci.edu \
--cc=Simon.Marshall@misys.com \
--cc=emacs-devel@gnu.org \
--cc=spiegel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.