Fix prepare-dev.py hook installation on Python 3

Review Request #10680 — Created Sept. 5, 2019 and submitted — Latest diff uploaded

Information

Review Board
master
9f69293...

Reviewers

There were two issues in the implementation of install_git_hooks that
prevented it from working on Python 3:
- an old-style (0777) octal literal was used instead of a new-style one
(0o777): old-style literals are not supported on Python 3; and
- bytes and unicode were being mixed in os.path.join, which is not an
issue in Python 2 but is on Python 3.

Since prepare-dev.py is intended to install dependencies, we can't
assume the presence of six, so we determine the correct string type
manually.

Ran prepare-dev.py under Python 2 and Python 3 and it created git
hooks successfully.

contrib/internal/build-media.py
contrib/internal/prepare-dev.py
reviewboard/dependencies.py
reviewboard/staticbundles.py
reviewboard/admin/security_checks.py
reviewboard/static/lib/js/csshover2.htc
reviewboard/static/lib/js/pngfix.htc
reviewboard/static/rb/css/defs.less
reviewboard/static/rb/css/mixins/style.less
reviewboard/static/rb/css/pages/admin/django.less
reviewboard/static/rb/css/pages/admin/dashboard.less
reviewboard/static/rb/css/pages/admin/integrations.less
reviewboard/static/rb/css/pages/base.less
reviewboard/static/rb/css/pages/admin/auth.less
reviewboard/static/rb/css/pages/admin/defs.less
reviewboard/static/rb/css/pages/admin/extensions.less
reviewboard/static/rb/css/pages/admin/filediffs.less
reviewboard/static/rb/css/pages/admin/log-viewer.less
reviewboard/static/rb/css/pages/admin/oauth2.less
reviewboard/static/rb/css/pages/admin/repositories.less
This diff has been split across 2 pages: 1 2 >
Loading...