Added virtualenv detection to better deal with requirements installat…

…ion.
morpheus65535 · Jul 27, 2021 · d1f86a3 · d1f86a3
1 parent aca9941
commit d1f86a3
Showing 1 changed file with 14 additions and 4 deletions.
diff --git a/bazarr/init.py b/bazarr/init.py
@@ -42,6 +42,13 @@
 configure_logging(settings.general.getboolean('debug') or args.debug)
 import logging
 
+
+def is_virtualenv():
+    # return True if Bazarr have been start from within a virtualenv or venv
+    base_prefix = getattr(sys, "base_prefix", None) or getattr(sys, "real_prefix", None) or sys.prefix
+    return base_prefix != sys.prefix
+
+
 # deploy requirements.txt
 if not args.no_update:
     try:
@@ -57,10 +64,13 @@
             else:
                 logging.info('BAZARR installing requirements...')
                 try:
-                    subprocess.check_output([sys.executable, '-m', 'pip', 'install', '--user', '-qq',
-                                             '--disable-pip-version-check', '--no-color', '-r',
-                                             os.path.join(os.path.dirname(__file__), '..', 'requirements.txt')],
-                                            stderr=subprocess.STDOUT)
+                    pip_command = [sys.executable, '-m', 'pip', 'install', '-qq', '--disable-pip-version-check',
+                                   '--no-color', '-r', os.path.join(os.path.dirname(os.path.dirname(__file__)),
+                                                                    'requirements.txt')]
+                    if not is_virtualenv():
+                        # --user only make sense if not running under venv
+                        pip_command.insert(4, '--user')
+                    subprocess.check_output(pip_command, stderr=subprocess.STDOUT)
                 except subprocess.CalledProcessError as e:
                     logging.exception('BAZARR requirements.txt installation result: {}'.format(e.stdout))
                     os._exit(1)