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

Resolve Travis/Windows/pip issues with upgrade to cryptography 3.0 #1016

Merged
merged 8 commits into from
Aug 1, 2020

Conversation

micahellison
Copy link
Member

@micahellison micahellison commented Aug 1, 2020

This PR includes the upgrade to cryptography 3.0 (#1012) and fixes the Windows build issues that resulted from that upgrade.

It turns out that, for some reason, telling poetry to store its virtual environment inside the project folder prevents the Windows access issues that prevented poetry (and the version of pip it's running) from working elsewhere within the user's folder.

Checklist

  • I have read the contributing doc.
  • I have included a link to the relevant issue number.
  • [n/a] I have tested this code locally.
  • I have checked to ensure there aren't other open pull requests
    for the same issue. (note: this is based on Update dependency (cryptography) #1012)
  • [n/a] I have written new tests for these changes, as needed.
  • All tests pass.

@micahellison micahellison added windows build Issues related to the build pipeline labels Aug 1, 2020
@micahellison micahellison changed the title [WIP] [build testing] Resolving Travis/Windows/pip issues Resolving Travis/Windows/pip issues Aug 1, 2020
@micahellison micahellison changed the title Resolving Travis/Windows/pip issues Resolvie Travis/Windows/pip issues with upgrade to cryptography 3.0 Aug 1, 2020
@micahellison micahellison changed the title Resolvie Travis/Windows/pip issues with upgrade to cryptography 3.0 Resolve Travis/Windows/pip issues with upgrade to cryptography 3.0 Aug 1, 2020
Copy link
Member

@wren wren left a comment

Choose a reason for hiding this comment

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

Thanks for sorting this out! 🥇

@wren wren merged commit 64a31e3 into jrnl-org:develop Aug 1, 2020
@wren wren mentioned this pull request Aug 1, 2020
6 tasks
wren added a commit that referenced this pull request Oct 17, 2020
…1016)

* add missing dependency
* update cryptography dependency
* Installing poetry with pipx instead of pip
* Fix bad pipx install command
* Remove pipx commands and attempt to solve issue through poetry configuration
* Fix pip install command to include --upgrade as before

Co-authored-by: Jonathan Wren <jonathan@nowandwren.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues related to the build pipeline windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants