all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Andre Spiegel <spiegel@gnu.org>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: vc-cvs-global-switches and -f
Date: Wed, 22 Jun 2005 12:08:53 +0200	[thread overview]
Message-ID: <1119434933.4412.276.camel@localhost> (raw)
In-Reply-To: <f7ccd24b05062120511dfe0dab@mail.gmail.com>

On Wed, 2005-06-22 at 05:51 +0200, Juanma Barranquero wrote:

> Let's put it other way: even on normal situation, it seems safer to
> make VC *not* use customized defaults it knows nothing about (and
> which can cause trouble), and let the user who uses a .cvsrc and knows
> what he's doing to put a simple
> 
>   (setq vc-cvs-global-switches nil)
> 
> on their .emacs.

If we put implementation issues aside for the moment, I think the
following behaviour would be desirable for VC:

(1) Settings from .cvsrc should be effective during VC commands, unless
    a particular VC command would not work with that setting, in 
    which case VC must override it

(2) Customizations that the user makes inside Emacs should override
    settings in .cvsrc.  In some cases, it may be good or tolerable to 
    see a warning message if the settings are inconsistent, e.g. in 
    your case where Emacs and .cvsrc don't agree on the diff format.

(3) Generally, users who customize VC settings or who create .cvsrc 
    files can be considered advanced users, and they can be bothered 
    with some effort to make sure that both kinds of settings are
    consistent.

Given this, I don't like the idea of making "-f" the default in VC,
because it violates (1).

A real solution would require something along the lines of Stefan's
parsing of .cvsrc from within Emacs.  Stefan, do you see a possibility
that VC could use your code from pcvs.el to do this?

However, given that we are in a feature freeze, and given (3), and given
the fact that this is a rather exotic situation, I don't think we should
change anything right now.

  reply	other threads:[~2005-06-22 10:08 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-21  1:28 vc-cvs-global-switches and -f Juanma Barranquero
2005-06-21 20:49 ` Stefan Monnier
2005-06-21 22:24   ` Juanma Barranquero
2005-06-22  3:31     ` Stefan Monnier
2005-06-22  3:51       ` Juanma Barranquero
2005-06-22 10:08         ` Andre Spiegel [this message]
2005-06-22 10:41           ` Juanma Barranquero
2005-06-22 11:48             ` Andre Spiegel
2005-06-22 11:48             ` Stefan Monnier
2005-06-22 13:03               ` Juanma Barranquero

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=1119434933.4412.276.camel@localhost \
    --to=spiegel@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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.