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

Add documentation tell users to enable EPEL on RedHat/CentOS 6 #1241

Merged
merged 1 commit into from Nov 16, 2015

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Oct 30, 2015

This PR replaces PR #1232 to fix #1204

@pde

This comment has been minimized.

Show comment
Hide comment
@pde

pde Oct 30, 2015

Member

Can we just do this for the user, instead of telling them to do it? Or failing that, detect its absence and give them a command to run?

Member

pde commented Oct 30, 2015

Can we just do this for the user, instead of telling them to do it? Or failing that, detect its absence and give them a command to run?

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Oct 30, 2015

Because we definitely do not want to enable EPEL for an user who has Python 2.7 installed (not necessarily via rpm, maybe they compiled it themselves), the reason is the python-virtualenv provided in EPEL is for Python 2.6

ghost commented Oct 30, 2015

Because we definitely do not want to enable EPEL for an user who has Python 2.7 installed (not necessarily via rpm, maybe they compiled it themselves), the reason is the python-virtualenv provided in EPEL is for Python 2.6

@pde

This comment has been minimized.

Show comment
Hide comment
@pde

pde Nov 2, 2015

Member

OK. I'd still prefer to automate detection of those circumstances if possible (after all, the sys admin may naively follow our instructions even if Python 2.7 is installed). But will merge this if you don't think an automated approach is feasible.

Member

pde commented Nov 2, 2015

OK. I'd still prefer to automate detection of those circumstances if possible (after all, the sys admin may naively follow our instructions even if Python 2.7 is installed). But will merge this if you don't think an automated approach is feasible.

@kuba

This comment has been minimized.

Show comment
Hide comment
@kuba

kuba Nov 2, 2015

Contributor

Why do you need to install this EPEL repo? A couple of people managed to get away without EPEL when using #1257.

Contributor

kuba commented Nov 2, 2015

Why do you need to install this EPEL repo? A couple of people managed to get away without EPEL when using #1257.

@@ -21,7 +21,6 @@ $tool install -y \
python \
python-devel \
python-virtualenv \
python-devel \

This comment has been minimized.

@kuba

kuba Nov 2, 2015

Contributor

@devnsec-com: since we might not merge this PR at all, can you submit another with this change only?

@kuba

kuba Nov 2, 2015

Contributor

@devnsec-com: since we might not merge this PR at all, can you submit another with this change only?

@pde

This comment has been minimized.

Show comment
Hide comment
@pde

pde Nov 12, 2015

Member

@devnsec-com did #1257 being merged resolve the need for this? Or should we still figure out how to land it?

Member

pde commented Nov 12, 2015

@devnsec-com did #1257 being merged resolve the need for this? Or should we still figure out how to land it?

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Nov 12, 2015

@pde I don't think #1257 solves this issue(but I didn't test it), because the CentOS 6 does not have the virtualenv in the offcial repos, without EPEL the bootstrap script will not be able to install virtualenv.

ghost commented Nov 12, 2015

@pde I don't think #1257 solves this issue(but I didn't test it), because the CentOS 6 does not have the virtualenv in the offcial repos, without EPEL the bootstrap script will not be able to install virtualenv.

@centminmod

This comment has been minimized.

Show comment
Hide comment
@centminmod

centminmod Nov 13, 2015

yes centos 6 doesn't have yum virtualenv package hence requires EPEL https://community.letsencrypt.org/t/redhat-centos-6-x-users-need-python-2-7/2190/7

centminmod commented Nov 13, 2015

yes centos 6 doesn't have yum virtualenv package hence requires EPEL https://community.letsencrypt.org/t/redhat-centos-6-x-users-need-python-2-7/2190/7

@pde

This comment has been minimized.

Show comment
Hide comment
@pde

pde Nov 16, 2015

Member

OK, merging in that case.

Member

pde commented Nov 16, 2015

OK, merging in that case.

pde added a commit that referenced this pull request Nov 16, 2015

Merge pull request #1241 from devnsec-com/pr7
Add documentation tell users to enable EPEL on RedHat/CentOS 6

@pde pde merged commit f265179 into certbot:master Nov 16, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment