-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
GH-102973: add a dev container #102975
GH-102973: add a dev container #102975
Conversation
On content update, builds `python` and the docs.
…into dev-container
The extension itself is left out until the Jupyter soft dependency is dropped to minimize installation cruft for this repo.
Seems to not like `Py_VAR_HEAD`.
@frenzymadness, do you want to take a look? (I wont't get to it this week) |
Thanks @encukou for the ping. I like it. You might want to use |
Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
Thanks @frenzymadness ! PTAL and let me know if I got everything you were suggesting. |
Looks perfect to me now. |
I'm not familiar with devcontainers, let me throw ideas around for consideration. I don't understand the You might want to install You might want to install There's also a |
It is, but I view the dev container details as specific to editing via GitHub Codespaces while with the Dockerfile I tried to take a less opinionated approach.
👍
I'm up for it!
With GitHub Codespaces prebuilds the save is minimal since the assumption is everyone will start from the container already created and all of those commands already run (takes less then 45 seconds to get to a terminal prompt). But I'm up for trying it out! |
Turns out |
I decided to not go with |
I decided to go ahead and merge this as we can easily update it if there's anything we want to tweak. |
It takes 20min to start Codespace. |
Already working on it; have to get budget from the PSF first. |
In case you want to revisit
Fair. I use it (one less venv → one less update command to forget to run, and with docs previews a slightly outdated version doesn't matter that much), but, it's not for everyone. |
Yep, I believe you can also use
True, although once we get prebuilds up and running that will be automatic after every commit to |
Prebuilds have been turned on! https://github.com/python/cpython/actions/workflows/codespaces/create_codespaces_prebuilds /cc @methane |
ENV WASMTIME_VERSION=7.0.0 | ||
ENV WASMTIME_CPU_ARCH=x86_64 | ||
|
||
RUN dnf -y --nodocs install git clang xz python3-blurb dnf-plugins-core && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I realize I am late to the party, but you might save space by installing git-core instead of git, which should avoid Perl (unless something else brings it in).
(dnf builddep python3
will bring in git-core as well, because we use git to apply patches, but that is probably not explicit enough.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Invoking dnf with --setopt=install_weak_deps=False
will also save space. Let me try sending a PR and see if it works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On content update, builds `python` and the docs. Also adds a Dockerfile that should include everything but autoconf 2.69 that's necessary to build CPython and the entire stdlib on Fedora. Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com> Co-authored-by: Dusty Phillips <dusty@phillips.codes>
Defines a Dockerfile based on Fedora along with common tooling (e.g. various shells people may want to use). It also includes the tooling necessary to do a cross-build for WASI.
Co-authored-by: Dusty Phillips dusty@phillips.codes