Make rbintegrations a packaging-only dependency.

Review Request #9249 — Created Oct. 4, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
76a9946...

Reviewers

In 3.0 beta 1, we attempted to make rbintegrations a dependency of
Review Board, which worked fine for the packaging process but created an
issue when building an environment to package rbintegrations. Since
rbintegrations was a hard dependency of Review Board, it created a
circular reference issue.

Now, we only include rbintegrations as a dependency if actively building
a package (through a sdist or bdist command), or if installing
(using install). Setting up a development environment no longer
requires it.

Tested running develop and saw that rbintegrations wasn't a
dependency.

Tested the packaging commands and inspected the package metadata.
rbintegrations was a dependency.

    Loading...