Try hard to get PYTHONPATH right for subprocesses.

Review Request #3896 — Created Feb. 20, 2013 and submitted

Information

Review Board
release-1.6.x

Reviewers

Try hard to get PYTHONPATH right for subprocesses.

This sets PYTHONPATH in two places, in an attempt to get it right for
all cases where we invoke subprocesses.

Newly generated reviewboard.wsgi files will have PYTHONPATH explicitly
set to include the conf directory.

We also set PYTHONPATH in initialize to that of sys.path, which should
ensure that, at least after the first URL access, we have all the
modules Review Board has access to available to the subprocess.
Accessed the site and saw no problems.

This general issue is limited to certain systems. I'm going by the
feedback from people who tried the workaround on bug 2834.
Description From Last Updated

testing

SA satish3

testing

SA satish3
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/__init__.py
      Ignored Files:
        reviewboard/cmdline/conf/reviewboard.wsgi.in
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
SA
  1. 
      
  2. Show all issues
    testing
  3. 
      
SA
  1. 
      
  2. Show all issues
    testing
  3. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...