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

Fix abi3 packaging #785

Merged
merged 1 commit into from
Apr 4, 2018
Merged

Conversation

stealthycoin
Copy link
Contributor

Python 3 has an ABI called abi3 which is the CPython 3 Stable ABI which
is compatible with any version of Python 3 after 3.2. The packager
previously did not recognize these as being compatible with the Lambda
abi of cp36m and would not use them. This fix adds abi3 as a valid abi
for python 3.

This fixes automatic packaging of cryptography 2.2.x since it removed
its specific python 3 wheel files in favor of a single cp34 abi3 wheel.

fixes #764

Python 3 has an ABI called abi3 which is the CPython 3 Stable ABI which
is compatible with any version of Python 3 after 3.2. The packager
previously did not recognize these as being compatible with the Lambda
abi of cp36m and would not use them. This fix adds abi3 as a valid abi
for python 3.

This fixes automatic packaging of cryptography 2.2.x since it removed
its specific python 3 wheel files in favor of a single cp34 abi3 wheel.
@codecov-io
Copy link

Codecov Report

Merging #785 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #785   +/-   ##
=======================================
  Coverage   94.91%   94.91%           
=======================================
  Files          21       21           
  Lines        3661     3661           
  Branches      470      470           
=======================================
  Hits         3475     3475           
  Misses        124      124           
  Partials       62       62
Impacted Files Coverage Δ
chalice/deploy/packager.py 95.6% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c4a4f1...a9895db. Read the comment docs.

@stealthycoin stealthycoin merged commit 7a793a9 into aws:master Apr 4, 2018
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.

Latest version of cryptography 2.2.1 fails on module import
4 participants