Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated --process-dependency-links flag from pip install #755

Merged
merged 1 commit into from
Feb 25, 2019

Conversation

rmartz
Copy link
Contributor

@rmartz rmartz commented Feb 21, 2019

Overview

Removes pip install flag that was removed in pip version 12. It appears to no longer be necessary (Or had been previously unused).

Checklist

  • PR has a descriptive enough title to be useful in changelogs

Testing Instructions

Closes PT163762556

Copy link
Contributor

@pcaisse pcaisse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It looks like the --process-dependency-links flag was added for djsonb (see f018e77), so I just wanted to test that djsonb was working correctly. I deleted the driver-app container and image and reprovisioned. Then I imported djsonb within a Python shell inside of the container using the new image and it seems to be working fine.

@rmartz rmartz merged commit 21c1a14 into develop Feb 25, 2019
@rmartz rmartz deleted the bugfix/deprecated-pip-dep-flag branch February 25, 2019 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants