all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Sascha Wilde <wilde@sha-bang.de>
To: emacs-devel@gnu.org
Subject: Un-setting HGRCPATH in vc-hg.el problematic
Date: Tue, 15 Jun 2010 13:38:00 +0200	[thread overview]
Message-ID: <m2sk4ofrvr.fsf@kenny.sha-bang.de> (raw)

Hi*,

in vc-hg-state and vc-hg-working-revision the hg binary is run with
HGRCPATH set to an empty string in its environment.  What is the
rational of doing so?  The comment states "Avoid localization of
messages so we can parse the output." but to my best knowledge the hgrc
files are not used to set the localization...

The problem with this is, that in general the hgrc files are used by
users/administrators to make configurations they actually want to use --
disabling this configurations unconditionally will be a source of
endless surprise and misbehavior.

I stumbled about it, when I suddenly noticed, that I got "Not trusting
file /foo/bar/stuff/.hg/hgrc from untrusted user bob, group users
Not trusting file /foo/bar/stuff/.hg/hgrc from untrusted user bob, group users
156" instead of the revision number in my mode line and was no longer
able to check in the altered file with C-v C-v.

The user bob is configured as trusted in our hg setup so this should not
happen -- but with HGRCPATH='' our setup is simply ignored -- that's
bad.

I'd suggest removing the HGRCPATH='' from the environment.  If there
are really configuration options that should be disabled for good
reasons in this functions, I'd suggest using the --config option of hg
to set them specifically.

cheers
sascha
-- 
Sascha Wilde

"Unix was the first OS where you could carry the media and system
documentation around in a briefcase. This was fixed in BSD4.2."



             reply	other threads:[~2010-06-15 11:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-15 11:38 Sascha Wilde [this message]
2010-06-28  4:23 ` Un-setting HGRCPATH in vc-hg.el problematic Dan Nicolaescu
2010-06-28 11:21   ` Sascha Wilde
2010-06-28 15:22     ` Dan Nicolaescu
2010-08-01 15:32       ` Sascha Wilde
2010-08-04 16:58         ` Dan Nicolaescu
2010-08-04 17:26           ` Sascha Wilde
2010-08-05  6:33             ` Dan Nicolaescu
2010-08-06 16:14               ` Sascha Wilde
2010-08-22 13:26                 ` Sascha Wilde
2010-09-11 13:48                   ` Stefan Monnier
2010-09-17  7:35                     ` Sascha Wilde
2010-06-28 19:39     ` Chong Yidong

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=m2sk4ofrvr.fsf@kenny.sha-bang.de \
    --to=wilde@sha-bang.de \
    --cc=emacs-devel@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.