Merged James Rowe's arch-shouldn't-create-userwide-config-file patch.
authorJames Rowe <jnrowe@ukfsn.org>
Fri, 19 Jun 2009 19:10:34 +0000 (15:10 -0400)
committerW. Trevor King <wking@drexel.edu>
Fri, 19 Jun 2009 19:10:34 +0000 (15:10 -0400)
From his email to the be-devel list:

From: James Rowe <jnrowe@ukfsn.org>
Date: Tue, 3 May 2009 11:44:41 +0000
Subject: [PATCH] Don't create config file unless we're using arch.

Hi,

  I find the current behaviour of creating a config file simply to set
a default for a revision control system I'm never going to use to be
a little annoying, the attached patch changes this behaviour to only
set the default in the config file if you're actually using arch.

Thanks,

James

libbe/arch.py

index 11735359d378e5401f971da8f20f38f226119344..b6d01e1c32d149103cc96990104ff90bcb446db8 100644 (file)
@@ -29,7 +29,6 @@ from rcs import RCS, RCStestCase, CommandError
 client = config.get_val("arch_client")
 if client is None:
     client = "tla"
-    config.set_val("arch_client", client)
 
 def new():
     return Arch()
@@ -50,6 +49,7 @@ class Arch(RCS):
     def _rcs_detect(self, path):
         """Detect whether a directory is revision-controlled using Arch"""
         if self._u_search_parent_directories(path, "{arch}") != None :
+            config.set_val("arch_client", client)
             return True
         return False
     def _rcs_init(self, path):