# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 2019-12-11 19:22:37
# Node ID 86fe85364811ea3b233639dc3525c485d9ffe532
# Parent  40fd1ef4e4c154a4895a494a07f82c663597c44b

rcutil: don't check if defaultrc/ is a directory -- we know it is

`mercurial/defaultrc/` is a directory both in the Mercurial repo and
once installed on a target platform. The directory was created in
c4ce077588d0 (config: introduce "built-in" default configuration
settings in default.d, 2014-09-04). That commit has some more
information, but it still doesn't seem to say that `defaultrc/` (then
called `default.d/`) could be a file. Perhaps the check was there to
allow you to run the same code on an older install/repo?

Differential Revision: https://phab.mercurial-scm.org/D7624

diff --git a/mercurial/rcutil.py b/mercurial/rcutil.py
--- a/mercurial/rcutil.py
+++ b/mercurial/rcutil.py
@@ -63,11 +63,8 @@ def envrcitems(env=None):
 
 def defaultrcpath():
     '''return rc paths in defaultrc'''
-    path = []
     defaultpath = os.path.join(resourceutil.datapath, b'defaultrc')
-    if os.path.isdir(defaultpath):
-        path = _expandrcpath(defaultpath)
-    return path
+    return _expandrcpath(defaultpath)
 
 
 def rccomponents():