-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Interactive installation of cython via the pip package manager inside the alpine container in the "Creating Your Own Container Images" does not work anymore #209
Comments
Thanks for spotting this and raising the issue. We need to have a think about what the example should be - Python was chosen for a reason as it is something that users often want to install but if it no longer works in a sensible way then a rethink is needed. Potential options:
|
OK - recent PRs have removed the Cython install via pip to get rid of the rror in the short term. We should review if this is the correct approach long term |
How about invoking |
Possibly. Most course attendees do not use compilers on a day-to-day basis so this may not be a good choice as you will likely need to explain what a compiler is for attendees which is a rabbit hole we probably do not want to go down. |
I think the example we have at the moment (install Python but not Cython) is fine for the purpose of the lesson. Does anyone else think we need to do more here or can we close this issue now? |
I am happy with this example. |
Likewise, I think what we have now is fine. I've removed a remaining outdated reference to cython via #249 and verified that the current example works as expected. |
)In the lesson on Creating Your Own Container Images, in the interactive installation section, we currently have the command to install
cython
within the container asAs of May 2023, pip version 23.3.1 (which gets installed by the previous command in the lesson
$ apt add --update python3 py3-pip python3-dev
, this does not work anymore!When we run the above command, we get the following error:
Cython does not get installed. I am not quite sure if it is acceptable to teach/encourage students to pass the
--break-system-packages
flag topip
just to get around this issue (we'll have to update the lessons in multiple places as well as the two Dockerfiles located within thedocker-intro.zip
file). Even when passing this flag, after the installation ofcython
, we get a strong warning:Perhaps a different example (maybe a non-python one) shall be helpful to avoid such confusions (esp to beginners)?
The text was updated successfully, but these errors were encountered: