Fix some lists used as default parameters

Author: timj

python/lsst/sconsUtils/dependencies.py

@@ -167,9 +167,9 @@ class Configuration:
         name ``<root>/doc/<name>.inc``.
     hasDoxygenTag : `bool`, optional
         If True, the package generates a Doxygen TAG file.
-    includeFileDirs : `list`, optional
+    includeFileDirs : `collections.abc.Sequence`, optional
         List of directories that should be searched for include files.
-    libFileDirs : `list`, optional
+    libFileDirs : `collections.abs.Sequence`, optional
         List of directories that should be searched for libraries.
     eupsProduct : `str`
         Name of the EUPS product for the package, if different from the name
@@ -223,8 +223,8 @@ def __init__(
         headers=(),
         libs=None,
         hasSwigFiles=True,
-        includeFileDirs=["include"],
-        libFileDirs=["lib"],
+        includeFileDirs=("include",),
+        libFileDirs=("lib",),
         hasDoxygenInclude=False,
         hasDoxygenTag=True,
         eupsProduct=None,

python/lsst/sconsUtils/installation.py

@@ -359,7 +359,7 @@ def InstallDir(self, prefix, dir, ignoreRegex=r"(~$|\.pyc$|\.os?$)", recursive=T
 
 
 @memberOf(SConsEnvironment)
-def InstallEups(env, dest, files=[], presetup=""):
+def InstallEups(env, dest, files=(), presetup=""):
     """Install a ups directory, setting absolute versions as appropriate
     (unless you're installing from the trunk, in which case no versions
     are expanded).
@@ -370,7 +370,7 @@ def InstallEups(env, dest, files=[], presetup=""):
         Environment to use.
     dest : `str`
         Destination directory.
-    files : `list`, optional
+    files : `collections.abc.Sequence`, optional
         List of files to install.  Any build/table files present in ``./ups``
         are automatically added to this list.
     presetup : `dict`, optional

python/lsst/sconsUtils/scripts.py

@@ -455,7 +455,7 @@ def python(module=None, src=None, extra=(), libs="main python"):
         return result
 
     @staticmethod
-    def pybind11(nameList=[], libs="main python", extraSrc=None, addUnderscore=True):
+    def pybind11(nameList=(), libs="main python", extraSrc=None, addUnderscore=True):
         """Convenience function to replace standard ``python/*/SConscript``
         boilerplate.
 
@@ -467,7 +467,7 @@ def pybind11(nameList=[], libs="main python", extraSrc=None, addUnderscore=True)
 
         Parameters
         ----------
-        nameList : `list`, optional
+        nameList : `collections.abc.Sequence`, optional
             Sequence of pybind11 modules to be built (does not include the
             file extensions).
         libs : `str` or `list`, optional