all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* vc.el no longer recognizes files in an arch working directory
@ 2007-05-25  3:38 Miles Bader
  2007-05-25  4:01 ` Chong Yidong
  0 siblings, 1 reply; 11+ messages in thread
From: Miles Bader @ 2007-05-25  3:38 UTC (permalink / raw)
  To: emacs-devel

Using current CVS (emacs -Q), if I visit a file in an arch working
directory, vc.el doesn't seem to recognize that:  no modeline indicator,
vc commands give an error "File ... is not under version control".

This used to work just fine of course.

It still works OK with files in a CVS working directory however.

-Miles
-- 
If you can't beat them, arrange to have them beaten.  [George Carlin]

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  3:38 vc.el no longer recognizes files in an arch working directory Miles Bader
@ 2007-05-25  4:01 ` Chong Yidong
  2007-05-25  4:03   ` Miles Bader
  0 siblings, 1 reply; 11+ messages in thread
From: Chong Yidong @ 2007-05-25  4:01 UTC (permalink / raw)
  To: Miles Bader; +Cc: emacs-devel

Miles Bader <miles.bader@necel.com> writes:

> Using current CVS (emacs -Q), if I visit a file in an arch working
> directory, vc.el doesn't seem to recognize that:  no modeline indicator,
> vc commands give an error "File ... is not under version control".
>
> This used to work just fine of course.
>
> It still works OK with files in a CVS working directory however.

Is this the branch or trunk?

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:01 ` Chong Yidong
@ 2007-05-25  4:03   ` Miles Bader
  2007-05-25  4:25     ` Miles Bader
  0 siblings, 1 reply; 11+ messages in thread
From: Miles Bader @ 2007-05-25  4:03 UTC (permalink / raw)
  To: Chong Yidong; +Cc: emacs-devel

Chong Yidong <cyd@stupidchicken.com> writes:
>> Using current CVS (emacs -Q), if I visit a file in an arch working
>> directory, vc.el doesn't seem to recognize that:  no modeline indicator,
>> vc commands give an error "File ... is not under version control".
>
> Is this the branch or trunk?

trunk

-miles
-- 
自らを空にして、心を開く時、道は開かれる

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:03   ` Miles Bader
@ 2007-05-25  4:25     ` Miles Bader
  2007-05-25  4:32       ` Chong Yidong
  2007-05-25  4:37       ` Miles Bader
  0 siblings, 2 replies; 11+ messages in thread
From: Miles Bader @ 2007-05-25  4:25 UTC (permalink / raw)
  To: Chong Yidong; +Cc: emacs-devel

Miles Bader <miles.bader@necel.com> writes:
>>> Using current CVS (emacs -Q), if I visit a file in an arch working
>>> directory, vc.el doesn't seem to recognize that:  no modeline indicator,
>>> vc commands give an error "File ... is not under version control".
>>
>> Is this the branch or trunk?
>
> trunk

I just tried the same thing using the release branch -- and it has the
same problem!!

[It's a fresh checkout from CVS, built using "make bootstrap"]

-Miles

-- 
"Suppose He doesn't give a shit?  Suppose there is a God but He
just doesn't give a shit?"  [George Carlin]

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:25     ` Miles Bader
@ 2007-05-25  4:32       ` Chong Yidong
  2007-05-25  4:37       ` Miles Bader
  1 sibling, 0 replies; 11+ messages in thread
From: Chong Yidong @ 2007-05-25  4:32 UTC (permalink / raw)
  To: Miles Bader; +Cc: emacs-devel

Miles Bader <miles.bader@necel.com> writes:

> Miles Bader <miles.bader@necel.com> writes:
>>>> Using current CVS (emacs -Q), if I visit a file in an arch working
>>>> directory, vc.el doesn't seem to recognize that:  no modeline indicator,
>>>> vc commands give an error "File ... is not under version control".
>>>
>>> Is this the branch or trunk?
>>
>> trunk
>
> I just tried the same thing using the release branch -- and it has the
> same problem!!

Please try to backtrack, and find out where the problem first
appeared.

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:25     ` Miles Bader
  2007-05-25  4:32       ` Chong Yidong
@ 2007-05-25  4:37       ` Miles Bader
  2007-05-25  7:30         ` Glenn Morris
  2007-05-25 14:47         ` Stefan Monnier
  1 sibling, 2 replies; 11+ messages in thread
From: Miles Bader @ 2007-05-25  4:37 UTC (permalink / raw)
  To: Chong Yidong; +Cc: emacs-devel

... and the following patch seems to fix it:


--- orig/lisp/vc-hooks.el
+++ mod/lisp/vc-hooks.el
@@ -325,7 +325,7 @@
                    ;; to another user.  This should save us from looking in
                    ;; things like /net and /afs.  This assumes that all the
                    ;; files inside a project belong to the same user.
-                   (not (equal user (file-attributes file)))
+                   (not (equal user (nth 2 (file-attributes file))))
                    (string-match vc-ignore-dir-regexp file)))
       (if (file-exists-p (expand-file-name witness file))
          (setq root file)


So I'm going to commit this on the branch/trunk.

[Geez, people, please test stuff you commit on the release branch!]

-Miles

-- 
Saa, shall we dance?  (from a dance-class advertisement)

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:37       ` Miles Bader
@ 2007-05-25  7:30         ` Glenn Morris
  2007-05-25  7:39           ` David Kastrup
  2007-05-25 14:47         ` Stefan Monnier
  1 sibling, 1 reply; 11+ messages in thread
From: Glenn Morris @ 2007-05-25  7:30 UTC (permalink / raw)
  To: Miles Bader; +Cc: Chong Yidong, emacs-devel

Miles Bader wrote:

> [Geez, people, please test stuff you commit on the release branch!]

The responsibility in this case can only lie with the person who added
this non-essential item (originally only on the trunk) to FOR-RELEASE.

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  7:30         ` Glenn Morris
@ 2007-05-25  7:39           ` David Kastrup
  2007-05-25  8:21             ` David Kastrup
  0 siblings, 1 reply; 11+ messages in thread
From: David Kastrup @ 2007-05-25  7:39 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

Glenn Morris <rgm@gnu.org> writes:

> Miles Bader wrote:
>
>> [Geez, people, please test stuff you commit on the release branch!]
>
> The responsibility in this case can only lie with the person who
> added this non-essential item (originally only on the trunk) to
> FOR-RELEASE.

I disagree.  The name of the file is "FOR-RELEASE", not
"FOR-BREAKING".  If I call a taxi, it does not imply that I am
responsible if the driver runs over half a dozen people on the way to
me.  And I don't see how the person putting something into FOR-RELEASE
can be held responsible for the particular code someone then places
into Emacs when he has no chance to _anticipate_ the outcome.

Unless the FOR-RELEASE item is "Please insert some random untested
stuff that looks good but will likely break."  In that case, I would
consider the person to be somewhat responsible.

-- 
David Kastrup

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  7:39           ` David Kastrup
@ 2007-05-25  8:21             ` David Kastrup
  0 siblings, 0 replies; 11+ messages in thread
From: David Kastrup @ 2007-05-25  8:21 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

David Kastrup <dak@gnu.org> writes:

> Unless the FOR-RELEASE item is "Please insert some random untested
> stuff that looks good but will likely break."  In that case, I would
> consider the person to be somewhat responsible.

An afterthought I have not yet seen such an entry.  But then I might
not be checking FOR-RELEASE often enough: it would certainly explain a
lot.

-- 
David Kastrup

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25  4:37       ` Miles Bader
  2007-05-25  7:30         ` Glenn Morris
@ 2007-05-25 14:47         ` Stefan Monnier
  2007-05-27  1:00           ` Richard Stallman
  1 sibling, 1 reply; 11+ messages in thread
From: Stefan Monnier @ 2007-05-25 14:47 UTC (permalink / raw)
  To: Miles Bader; +Cc: Chong Yidong, emacs-devel

> ... and the following patch seems to fix it:
> --- orig/lisp/vc-hooks.el
> +++ mod/lisp/vc-hooks.el
> @@ -325,7 +325,7 @@
>                     ;; to another user.  This should save us from looking in
>                     ;; things like /net and /afs.  This assumes that all the
>                     ;; files inside a project belong to the same user.
> -                   (not (equal user (file-attributes file)))
> +                   (not (equal user (nth 2 (file-attributes file))))
>                     (string-match vc-ignore-dir-regexp file)))
>        (if (file-exists-p (expand-file-name witness file))
>           (setq root file)

> So I'm going to commit this on the branch/trunk.

Oops, thanks.

> [Geez, people, please test stuff you commit on the release branch!]

Sorry 'bout the typo introduce when copy&pasting between the tested code and
the installed code.  But note that I'm surprised to see this code on
the release branch.  I never intended for it to go there, never committed
there AFAIK and when Richard asked me to install it on the release branch
I explicitly said I was not confortable doing it.

Looking at the CVS log I see:

   revision 1.182.2.2
   date: 2007-05-17 03:38:11 -0400;  author: gm;  state: Exp;  lines: +8 -1;  commitid: wzJ1cmOAuE56Mfis;
   Sync change from trunk at request of rms in FOR-RELEASE:
   
   Stefan Monnier  <monnier at iro.umontreal.ca>
   
   (vc-find-root): Stop searching when the user changes.

So it looks like it was Richard's decision.


        Stefan

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

* Re: vc.el no longer recognizes files in an arch working directory
  2007-05-25 14:47         ` Stefan Monnier
@ 2007-05-27  1:00           ` Richard Stallman
  0 siblings, 0 replies; 11+ messages in thread
From: Richard Stallman @ 2007-05-27  1:00 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: cyd, emacs-devel, miles

    Sorry 'bout the typo introduce when copy&pasting between the tested code and
    the installed code.

Such things happen.

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

end of thread, other threads:[~2007-05-27  1:00 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-25  3:38 vc.el no longer recognizes files in an arch working directory Miles Bader
2007-05-25  4:01 ` Chong Yidong
2007-05-25  4:03   ` Miles Bader
2007-05-25  4:25     ` Miles Bader
2007-05-25  4:32       ` Chong Yidong
2007-05-25  4:37       ` Miles Bader
2007-05-25  7:30         ` Glenn Morris
2007-05-25  7:39           ` David Kastrup
2007-05-25  8:21             ` David Kastrup
2007-05-25 14:47         ` Stefan Monnier
2007-05-27  1:00           ` Richard Stallman

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.