Clean up and document the Review Board setup.py file.

Review Request #8498 — Created Oct. 27, 2016 and submitted

Information

Review Board
release-3.0.x
c592875...

Reviewers

This is a cleanup change for bringing setup.py back to our modern coding
standards and to help with maintenance. It adds docstrings where
missing, removes the outdated ez_setup.py support, switches strings to
single quotes, appends "Command" to all command classes, removes the old
hack for installing data files on macOS (which no longer appears to be
needed for modern versions of Python), and fixes up the setup() call to
be better organized and use a more sane indent level. The package also
has a better description.

Built different package types and inspected them.

Tested that each entrypoint loaded, that all command line scripts were
registered, and that all configuration forms (authentication settings
and repositories) listed each option we support.

Compared the list of installed files before and after the install_data
hack. Verified that Python 2.7 isn't impacted by the old bug that the
classes were for. (Django has also removed their support, and it doesn't
look like Python 2.6 should be impacted either.)

chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (dee4b20)
Loading...