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: Closed (submitted)

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. 
      
Loading...