GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||||||
GeezerVison is showing github.com — GH-102973: add a dev container by brettcannon · Pull Request #102975 · python/cpython Conversation
|
|
@frenzymadness, do you want to take a look? (I wont't get to it this week) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks @encukou for the ping. I like it. You might want to use |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Co-authored-by: Ronald Oussoren <[email protected]>
|
Thanks @frenzymadness ! PTAL and let me know if I got everything you were suggesting. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Looks perfect to me now. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
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 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
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! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Turns out |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
I decided to not go with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I decided to go ahead and merge this as we can easily update it if there's anything we want to tweak. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
It takes 20min to start Codespace. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Already working on it; have to get budget from the PSF first. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
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. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Yep, I believe you can also use
True, although once we get prebuilds up and running that will be automatic after every commit to |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Prebuilds have been turned on! https://github.com/python/cpython/actions/workflows/codespaces/create_codespaces_prebuilds /cc @methane |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| 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.
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.)
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem 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.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
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 <[email protected]> Co-authored-by: Dusty Phillips <[email protected]>
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.