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

Bump required python-cryptography version #464

Closed
wants to merge 1 commit into from

Conversation

stlaz
Copy link
Contributor

@stlaz stlaz commented Feb 14, 2017

Since we started using Certificate.serial_number instead of .serial from python-cryptography (3d9bec2), bump the required version to the one where the above mentioned transition happened (pyca/cryptography@e295f3a).

@stlaz stlaz changed the title ⬆️ Bump required python-cryptography version Bump required python-cryptography version Feb 14, 2017
@HonzaCholasta
Copy link
Contributor

NACK, you didn't update the comments.

@HonzaCholasta HonzaCholasta added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Feb 14, 2017
@HonzaCholasta
Copy link
Contributor

@tiran
Copy link
Member

tiran commented Feb 16, 2017

ipasetup.py.in hasn't been updated.

@stlaz
Copy link
Contributor Author

stlaz commented Feb 16, 2017

Didn't realize we need that as well now, patch is in this PR.

@stlaz stlaz removed ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Feb 16, 2017
@tiran tiran self-assigned this Feb 17, 2017
@tiran tiran added the ack Pull Request approved, can be merged label Feb 17, 2017
@martbab
Copy link
Contributor

martbab commented Feb 17, 2017

@stlaz can you please link https://fedorahosted.org/freeipa/ticket/6631 to the fixup commit?

When bumping version of python-cryptography in freeipa.spec.in,
ipasetup.py.in was forgotten about.

https://fedorahosted.org/freeipa/ticket/6631
@stlaz
Copy link
Contributor Author

stlaz commented Feb 17, 2017

@martbab Sure, done.

@martbab
Copy link
Contributor

martbab commented Feb 17, 2017

@martbab martbab added the pushed Pull Request has already been pushed label Feb 17, 2017
@martbab martbab closed this Feb 17, 2017
@stlaz stlaz deleted the cryptography_bump branch September 11, 2017 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
4 participants