Remove setup_syspath entirely
authorEli Bendersky <eliben@gmail.com>
Sun, 26 Feb 2017 14:26:33 +0000 (06:26 -0800)
committerEli Bendersky <eliben@gmail.com>
Sun, 26 Feb 2017 14:26:33 +0000 (06:26 -0800)
It's vestigial remains from the early days of pyelftools which was originally
developed on Windows, running the tests through an editor rather than on the
command line.

Not setting the run path in the code is safer - it's more reliable to control where the code is found outside the code itself now

test/run_examples_test.py
test/run_readelf_tests.py
test/utils.py

index 2fd3ad1ab429f6e6990af00bd596df75709c4565..9bec604b2ebc7fd6d3740c7b7d331fb317c734ba 100755 (executable)
@@ -9,7 +9,6 @@
 #-------------------------------------------------------------------------------
 import os, sys
 import logging
-from utils import setup_syspath; setup_syspath()
 from utils import run_exe, is_in_rootdir, dump_output_to_temp_files
 
 
index f8990e104e4d5324e1136e012e6012c328c19a96..9f5a4f7dc31e0d57d56b3fd69cf46f88ccca903a 100755 (executable)
@@ -13,7 +13,6 @@ from difflib import SequenceMatcher
 from optparse import OptionParser
 import logging
 import platform
-from utils import setup_syspath; setup_syspath()
 from utils import run_exe, is_in_rootdir, dump_output_to_temp_files
 
 
index 0425f4594c7b5b3ec0bd76a9d5bc97b4ce73df14..7f896858be9b9bf6cdcafbd06488f8b61870ccfb 100644 (file)
@@ -9,16 +9,6 @@
 from __future__ import print_function
 import os, sys, subprocess, tempfile
 
-# This module should not import elftools before setup_syspath() is called!
-# See the Hacking Guide in the documentation for more details.
-
-def setup_syspath():
-    """ Setup sys.path so that tests pick up local pyelftools before the
-        installed one when run from development directory.
-    """
-    if sys.path[0] != '.':
-        sys.path.insert(0, '.')
-
 
 def run_exe(exe_path, args=[], echo=False):
     """ Runs the given executable as a subprocess, given the
@@ -54,4 +44,3 @@ def dump_output_to_temp_files(testlog, *args):
         file.write(s)
         file.close()
         testlog.info('@@ Output #%s dumped to file: %s' % (i + 1, path))
-