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

Jump to bottom

Adjustments for setup requirements v2 #297

Closed
wants to merge 1 commit into from
Closed

Adjustments for setup requirements v2 #297

wants to merge 1 commit into from

Conversation

pvomacka
Copy link

@pvomacka pvomacka commented Dec 1, 2016 •
edited by martbab

Remove setup requirement on wheel since it triggers download.

https://fedorahosted.org/freeipa/ticket/6468

Adjustments for setup requirements v2
b43f78c 
Remove setup requirement on wheel since it triggers download.

https://fedorahosted.org/freeipa/ticket/6468
@tiran tiran added the ack Pull Request approved, can be merged label Dec 2, 2016
@martbab
Copy link
Contributor

martbab commented Dec 2, 2016

Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/7f301b00ce50814d098e7e42324ef741c71b6853

@martbab martbab added the pushed Pull Request has already been pushed label Dec 2, 2016
@martbab martbab closed this Dec 2, 2016
@tiran
Copy link
Member

tiran commented Dec 2, 2016

The fix was originally a part of #255. Both changes got lost in a rebase. 1a8bf57 was still good. git reflog to the rescue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tiran tiran

Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
Milestone
No milestone
3 participants
@pvomacka @martbab @tiran