• 
      

    Fix "environment can only contain strings" errors.

    Review Request #7522 — Created July 14, 2015 and submitted

    Information

    Review Board
    release-2.0.x
    3640e81...

    Reviewers

    On Windows hosts, setting things into the environment requires passing in bytes
    rather than unicode. Normally I wouldn't really care, but this does also affect
    power pack installations.

    Ran unit tests. There are a few failures with rbssh, but those were already
    present, and are related to the python path.

    Description From Last Updated

    'reviewboard' imported but unused

    reviewbotreviewbot

    Col: 80 E501 line too long (81 > 79 characters)

    reviewbotreviewbot

    I think this might cause upgrade issue with python 2.5, or i am missing something here.

    AM Aman

    'reviewboard' imported but unused

    reviewbotreviewbot

    Missing a bytestring here.

    chipx86chipx86
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/admin/siteconfig.py
          reviewboard/ssh/utils.py
          reviewboard/extensions/packaging.py
          reviewboard/__init__.py
          reviewboard/scmtools/core.py
          reviewboard/cmdline/rbssh.py
          reviewboard/manage.py
          reviewboard/cmdline/rbsite.py
          reviewboard/admin/tests.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/admin/siteconfig.py
          reviewboard/ssh/utils.py
          reviewboard/extensions/packaging.py
          reviewboard/__init__.py
          reviewboard/scmtools/core.py
          reviewboard/cmdline/rbssh.py
          reviewboard/manage.py
          reviewboard/cmdline/rbsite.py
          reviewboard/admin/tests.py
      
      
    2. reviewboard/__init__.py (Diff revision 1)
       
       
      Show all issues
       'reviewboard' imported but unused
      
    3. reviewboard/manage.py (Diff revision 1)
       
       
      Show all issues
      Col: 80
       E501 line too long (81 > 79 characters)
      
    4. 
        
    david
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/admin/siteconfig.py
          reviewboard/ssh/utils.py
          reviewboard/extensions/packaging.py
          reviewboard/__init__.py
          reviewboard/scmtools/core.py
          reviewboard/cmdline/rbssh.py
          reviewboard/manage.py
          reviewboard/cmdline/rbsite.py
          reviewboard/admin/tests.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/admin/siteconfig.py
          reviewboard/ssh/utils.py
          reviewboard/extensions/packaging.py
          reviewboard/__init__.py
          reviewboard/scmtools/core.py
          reviewboard/cmdline/rbssh.py
          reviewboard/manage.py
          reviewboard/cmdline/rbsite.py
          reviewboard/admin/tests.py
      
      
    2. reviewboard/__init__.py (Diff revision 2)
       
       
      Show all issues
       'reviewboard' imported but unused
      
    3. 
        
    chipx86
    1. 
        
    2. reviewboard/cmdline/rbsite.py (Diff revision 2)
       
       
       
      Show all issues

      Missing a bytestring here.

    3. 
        
    chipx86
    1. Meant to say, once that's fixed, Ship It :)

    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.0.x (bf3eea7)
    AM
    1. 
        
    2. reviewboard/__init__.py (Diff revision 2)
       
       
      Show all issues

      I think this might cause upgrade issue with python 2.5, or i am missing something here.

      1. Review Board 2.0+ requires Python 2.6 at a minimum.

      2. Ahh I see.. Thanks for the info.

    3.