#1449 Update README.rst, use python2-virtualenv instead python-virtualenv
Merged by alunux. Opened by alunux.
alunux/pagure readme_python2  into  master

Download 1449.patch
no initial comment

Note that this won't work on older Fedora (23 for example) or CentOS/RHEL

Maybe we could mention it?

Thanks for re-opening the PR here btw :)

The python-virtualenv alias should work everywhere. I suggest keeping it the way it is due to the python2- package providing the python- package. Note that I'm not -1'ing this PR, so if @pingou wants it, feel free to ignore me ☺

@bowlofeggs I would agree with you if it wasn't for https://bugzilla.redhat.com/show_bug.cgi?id=1096506

@alunux did you see my comment?

I'm sorry, I didn't try it on F23 or Centos. I'm just confused with the package name and what python version is used when I want to set up my pagure instance :)

We can simply say something like:

     sudo dnf install git python2-virtualenv libgit2-devel \
                       libjpeg-devel gcc libffi-devel redhat-rpm-config
.. note: On Fedora 23 and earlier or on RHEL and derivative (CentOS, Scientific Linux) 
      the package `python2-virtualenv` is named `python-virtualenv`

What do you think?

Would you like to do it? Otherwise I can open a new PR with that wording on the top of your commit, as you prefer :)

Commit 835397c0 fixes this pull-request

Pull-Request has been merged by fadlun.net@gmail.com

Manually adjusted and merged, thanks!

Metadata