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

autoconf now prefers Python 3 over 2 #1614

Closed
wants to merge 2 commits into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Feb 22, 2018

The configure script now looks for Python 3.6 or newer, then falls back to Python 2. All Makefile default to Python 3 if Python 3 is available.

Move logic for installing just the Python packages out of the spec file and into our root Makefile. It removes code duplication to simplify a spec file that supports building without Python 2

https://pagure.io/freeipa/issue/7131

@tiran tiran added the py3 label Feb 22, 2018
@tiran tiran force-pushed the python3_configure branch 2 times, most recently from d0ad500 to d97d5b5 Compare February 23, 2018 08:47
@tiran tiran requested a review from stlaz February 23, 2018 11:17
@tiran tiran added the needs review Pull Request is waiting for a review label Feb 23, 2018
Move logic for installing just the Python packages out of the spec file
and into our root Makefile. It removes code duplication to simplify a
spec file that supports building without Python 2.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
The configure script now looks for Python 3.6 or newer, then falls back
to Python 2. All Makefile default to Python 3 if Python 3 is available.

See: pagure.io/freeipa/issue/7131
Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran
Copy link
Member Author

tiran commented Mar 14, 2018

ping The PR is waiting for a review.

@tiran tiran added the re-run Trigger a new run of PR-CI label Mar 14, 2018
@freeipa-pr-ci freeipa-pr-ci removed the re-run Trigger a new run of PR-CI label Mar 14, 2018
@abbra abbra self-assigned this Mar 15, 2018
@abbra
Copy link
Contributor

abbra commented Mar 15, 2018

ACK, looks good and passes tests.

@abbra abbra added ack Pull Request approved, can be merged and removed needs review Pull Request is waiting for a review labels Mar 15, 2018
@tiran tiran added the pushed Pull Request has already been pushed label Mar 15, 2018
@tiran
Copy link
Member Author

tiran commented Mar 15, 2018

master:

  • fd9ede5 Simplify Python package installation
  • fa94ef0 autoconf prefers Python 3 over 2

@tiran tiran closed this Mar 15, 2018
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
Development

Successfully merging this pull request may close these issues.

3 participants