Add notice about Python 3 #1807
Conversation
filips123
force-pushed
the
python3-notice
branch
2 times, most recently
from
6675289
to
0b40ea7
Compare
|
|
||
| * On RPM distros use: | ||
| ``sudo yum install python-devel xz-libs zlib libxml2-devel libxslt-devel bzip2-libs`` | ||
| ``sudo yum install python3.6-devel xz-libs zlib libxml2-devel libxslt-devel bzip2-libs`` |
There was a problem hiding this comment.
Same as above, are you sure that python3.6-devel is the correct package name and is available?
There was a problem hiding this comment.
Unfourtunately, Yum handles varsions a bit diffrent. Because of that, python36-devel does not exist and only Python 3.7 is availible.
(Tested on Fedora in Docker)
There was a problem hiding this comment.
You rock 🙇♂️ ... but what about EPEL?
There was a problem hiding this comment.
I think that EPEL also does not have Python 3.6 version of python-devel...
There was a problem hiding this comment.
In case if Python 2.7 is still supported, I can update instructions to only use Python 2.7 on RPM.
But the real solution would be to update ScanCode to support newer versions of Python and then also update instructions to just install any (newer) Python 3 version.
There was a problem hiding this comment.
to just install any (newer) Python 3 version.
Yes, but we will never support anything before Python 3.6
There was a problem hiding this comment.
But is now Python 3.7 supported?
|
I will check if that package names are correct tomorrow. |
Signed-off-by: Filip Š <filip.stamcar@hotmail.com>
Signed-off-by: Filip Š <filip.stamcar@hotmail.com>
Signed-off-by: Filip Š <filip.stamcar@hotmail.com>
filips123
force-pushed
the
python3-notice
branch
from
f1ec360
to
fcc661d
Compare
|
@filips123 are we all set? Do you mind to rebase so I can review and merge? |
|
@filips123 ping |
|
The problem is Python 3.6 handling in distributions that use Yum. I think that they just have 3.7+ available. The solution for this is to check if ScanCode supports 3.7 and newer versions (and obviously support them if they are not supported already), and then fix instructions to install any newer version. |
|
Recent README and documentation changes were already updated to recommend Python 3.6.8 so this PR is not needed anymore. |
Add notice that Python 2.7 up to 3.6.8 is supported to README. This should be correct, but please review if any of installation methods don't work.
Also, this should also be added to repository wiki. I will do this after I'm sure that all changes here are correct.
@pombredanne @Abhishek-Dev09