Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

Don't require Python 3.5+ #133

Merged
merged 1 commit into from Mar 15, 2018
Merged

Don't require Python 3.5+ #133

merged 1 commit into from Mar 15, 2018

Conversation

Ichimonji10
Copy link

@Ichimonji10 Ichimonji10 commented Mar 12, 2018

It's entirely possible to install Pulp 3 on a host for which the
ansible_python_interpreter is older than Python 3.5. What matters is
that Python 3.5+ can be made available. For example, on
RHEL 7, the default Python interpreter is Python 2.7, but Python 3.5 can
be installed via an SCL, and Pulp can be installed into a virtualenv
that uses the SCL Python.

@Ichimonji10
Copy link
Author

Ichimonji10 commented Mar 12, 2018

This change should have no impact on Fedora installs, and is necessary in order to enable RHEL 7 installs. This and #130 are the only PRs remaining before I can submit a PR that will enable support for RHEL 7.

Tested against non-Vagrant F26 and F27 VMs.

@asmacdo
Copy link
Contributor

asmacdo commented Mar 13, 2018

The original reason for requiring Python 3.5+ was some stuff in pulp_facts.py, which is apparently gone :)

It's entirely possible to install Pulp 3 on a host for which the
`ansible_python_interpreter` is older than Python 3.5. What matters is
that Python 3.5+ can be made available. For example, on RHEL 7, the
default Python interpreter is Python 2.7, but Python 3.5 can be
installed via an SCL, and Pulp can be installed into a virtualenv that
uses the SCL Python.
@Ichimonji10 Ichimonji10 merged commit 67eef58 into pulp:3.0-dev Mar 15, 2018
@Ichimonji10 Ichimonji10 deleted the drop-min-python branch March 15, 2018 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants