Use the proper sys.path in build/print_python_deps.py

build/print_python_deps.py

Index: build/print_python_deps.py
diff --git a/build/print_python_deps.py b/build/print_python_deps.py
index 3d0c9a8e140795010476c86158a04094834abdd5..50a1f08f4bf6e87a780ac3a173b48c3e41c7705d 100755
--- a/build/print_python_deps.py
+++ b/build/print_python_deps.py
@@ -82,7 +82,9 @@ def main():
                       help='Recursively include all non-test python files '
                       'within this directory. May be specified multiple times.')
   options = parser.parse_args()
-  sys.path.append(os.path.dirname(options.module))
+  # Replace the path entry for print_python_deps.py with the one for the given
+  # module.
+  sys.path[0] = os.path.dirname(options.module)

[jbudorick, 2016/11/10 19:41:02]

Do we want anything else in sys.path at all? I'm concerned that this won't work
in some cases if the modules don't resolve to something in the options.module
path.

[agrieve, 2016/11/10 19:45:35]

If you clear out sys.path, then normal system import don't work (e.g. import
os). I did a local test with a file that just prints sys.path:

PYTHONPATH=a python hi.py
['/usr/local/google/home/agrieve/ssd/git/clankium1/src',
'/usr/local/google/home/agrieve/ssd/git/clankium1/src/a', '/usr/lib/python2.7',
'/usr/lib/python2.7/plat-x86_64-linux-gnu', '/usr/lib/python2.7/lib-tk',
'/usr/lib/python2.7/lib-old', '/usr/lib/python2.7/lib-dynload',
'/usr/local/lib/python2.7/dist-packages', '/usr/lib/python2.7/dist-packages',
'/usr/lib/python2.7/dist-packages/PILcompat',
'/usr/lib/python2.7/dist-packages/gtk-2.0', '/usr/lib/pymodules/python2.7',
'/usr/lib/python2.7/dist-packages/ubuntu-sso-client']

1st entry is module dir, 2nd is PYTHONPATH, remaining is system paths

[jbudorick, 2016/11/10 19:49:12]

I guess I'm concerned about the case where a module appends to sys.path then
imports something from that, but it conflicts with something on PYTHONPATH.
Maybe that's too much of a corner case for this script to worry about, though --
perhaps the responsibility for not passing the infra PYTHONPATH contents falls
on the infra side of the fence.

   imp.load_source('NAME', options.module)
 
   paths_set = _ComputePythonDependencies()