Merge pull request #13 from codrsquad/more-lenient-entry-point-checking

Loosen entry point checking to support non-wrapped executables

Author: amjith

src/pickley/package.py

@@ -52,7 +52,7 @@ def __init__(self, venv):
         self.files = None
         self.info = {}
         name = venv.pspec.dashed
-        if runez.DRYRUN and not venv.is_venv_exe("bin/pip"):
+        if runez.DRYRUN and not runez.is_executable("bin/pip"):
             self.bin.files = {venv.pspec.dashed: "dryrun"}  # Pretend an entry point exists in dryrun mode
             return
 
@@ -84,7 +84,7 @@ def __init__(self, venv):
                     if "_completer" in basename:
                         self.completers.add_file(path)
 
-                    elif venv.is_venv_exe(path):
+                    elif runez.is_executable(path):
                         self.bin.add_file(path)
 
                 elif dirname.endswith(".dist-info"):
@@ -210,20 +210,6 @@ def bin_path(self, name, try_variant=False):
             if os.path.exists(path):
                 return path
 
-    def is_venv_exe(self, path):
-        """
-        Args:
-            path (str): Path to file to examine
-
-        Returns:
-            (bool): True if 'path' points to a python executable part of this venv
-        """
-        if runez.is_executable(path):
-            for line in runez.readlines(path, first=1):
-                if line.startswith("#!"):
-                    if path.startswith(self.folder):
-                        return True
-
     def pip_install(self, *args):
         """Allows to not forget to state the -i index..."""
         r = self.run_pip("install", "-i", self.pspec.index, *args, fatal=False)