• 
      

    Fixed overzealous initialization of PYTHONPATH.

    Review Request #4027 — Created April 7, 2013 and submitted

    Information

    Review Board
    master

    Reviewers

    This is a bugfix over __init__.py that was inititalliy changed at https://github.com/reviewboard/reviewboard/commit/915452d483a812ee34006741a6f5c23a4d03a5b1
    
    The current initialization is overzealous and conflicts with the installation call from extensionbrowser's easy_install usage (with the current initialization, an easy_install call would nuke all entries in the easy_install.pth file and overwrite previous entries).
    
     
    Description From Last Updated

    As briefly discussed in IRC, you were right before in that settings_local.__file__ is correct. Also, make sure both parameters align.

    chipx86chipx86
    reviewbot
    1. This is a review from Review Bot.
        Tool: PEP8 Style Checker
        Processed Files:
          reviewboard/__init__.py
        Ignored Files:
      
      
    2. 
        
    AY
    reviewbot
    1. This is a review from Review Bot.
        Tool: PEP8 Style Checker
        Processed Files:
          reviewboard/__init__.py
        Ignored Files:
      
      
    2. 
        
    chipx86
    1. 
        
    2. reviewboard/__init__.py (Diff revision 2)
       
       
      For future reference, settings must always be imported as:
      
          from django.conf import settings
      
      Of course, in this case, we'll actually want settings_local, as you had before. See below:
    3. reviewboard/__init__.py (Diff revision 2)
       
       
      Show all issues
      As briefly discussed in IRC, you were right before in that settings_local.__file__ is correct.
      
      Also, make sure both parameters align.
    4. 
        
    AY
    reviewbot
    1. This is a review from Review Bot.
        Tool: PEP8 Style Checker
        Processed Files:
          reviewboard/__init__.py
        Ignored Files:
      
      
    2. 
        
    chipx86
    1. Ship It!
    2. 
        
    AY
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-1.7.x (996026c)