-
Notifications
You must be signed in to change notification settings - Fork 309
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
Dockerfile: Temporary adaptation for Ubuntu 20.04 #3613
Conversation
This is a WIP. |
I'll convert it to a draft PR consequently.
Probably, it makes more sense to switch to ScanCode v21.2.9 first, as it has full support for recent Python 3 versions. |
|
Actually not that simple. Scancode new binaries are python 3.6, and bionic is 3.8. I will check that later |
At least the release notes say "Add support for Python 3.6, 37, 3.8 and 3.9 on Linux". But I'm not sure whether that refers to the package released on PyPI, or only to builds from source. Maybe @pombredanne can explain? |
cda3c04
to
885e3dc
Compare
Updated:
|
The PyPI package that you install with pip runs on 3.6, 37, 3.8 and 3.9. If you want to install from PyPI you can either use the wheel or source distribution and must use |
I needed Scancode for a project as well. I had some troubles when using the requirements.txt above, so I've installed the full scancode pip package and did a Now its dependency versions are fixed and it's running in python 3.9. See: https://github.com/philips-software/license-scanner/pull/48/files |
885e3dc
to
57d0a24
Compare
09bbe92
to
bb89a2c
Compare
bb89a2c
to
40ca253
Compare
Ubuntu 20.04 jumped to python3 mostly, some adaptation are needed to run basic docker. This is far to be production ready, as it requires scancode been compiled from source code instead of releases code but properly works for Ort been ready to run TODO: * Ort requirements test will fail, so it need to be fixe. Commented now * Dependencies installation inside docker can be done in a smar way, with independent installation instead of heavily depends on Ubuntu packages Signed-off-by: Helio Chissini de Castro <helio@kde.org>
46bea22
to
8b8527b
Compare
I tried a similar conversion myself, but then the
When upgrading the Docker image, do we need to consider these requirements as well? |
With this particular configuration of packages I get errors from ScanCode. For example:
Outside of the container image I can reproduce the error with this version of ScanCode using the provided command. It is resolved with the newest version in Pypi |
Superseded by #3902. |
Ubuntu 20.04 jumped to python3 mostly, some adaptation are
needed to run basic docker.
This is far to be production ready, as it requires scancode been
compiled from source code instead of releases code but properly
works for Ort been ready to run
TODO:
with independent installation instead of heavily depends on Ubuntu packages
Signed-off-by: Helio Chissini de Castro helio@kde.org
Please ensure that your pull request adheres to our contribution guidelines. Thank you!