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 Fedora rawhide builders #127

Merged
merged 2 commits into from
Sep 14, 2017

Conversation

frozencemetery
Copy link
Member

This seems to happen roughly every time it branches; see build log for #124.

@frozencemetery frozencemetery force-pushed the fedora_gpg branch 2 times, most recently from 5f51be3 to b39fa50 Compare September 5, 2017 16:48
@DirectXMan12
Copy link
Member

seems reasonable-ish. Would prefer to have GPG checks, but if they break when fedora branches, I'm not sure there's much we can do about it...

Copy link
Member

@DirectXMan12 DirectXMan12 left a comment

Choose a reason for hiding this comment

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

Consider this good to merge once we're sure that the failure is not a bug on our end.

@frozencemetery
Copy link
Member Author

Agreed... it's probably not something we want in our release builds if we can avoid it, but on the other hand, release builds aren't working right now and haven't been for a while.

@frozencemetery
Copy link
Member Author

@DirectXMan12 Please review added commit. Thanks!

@frozencemetery frozencemetery changed the title [travis] Tolerate GPG failures in Fedora rawhide Fix Fedora rawhide builders Sep 8, 2017
@DirectXMan12
Copy link
Member

DirectXMan12 commented Sep 13, 2017

can you put the RFC number after is permitted by RFC (I looked at 2743/2744, but couldn't find exactly what you were referring to), and also put a rough version near in newever versions for posterity? Otherwise, looks good.

@DirectXMan12
Copy link
Member

(also possibly add a more elaborate comment than # clockskew)

@frozencemetery
Copy link
Member Author

Added version to commit and clarified language. Elaborated on comment inline.

krb5 >= 1.14 adds the clockskew to the server context lifetime.  Allow
this behavior in test_high_level.py.
@DirectXMan12 DirectXMan12 merged commit bd61844 into pythongssapi:master Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants