Skip to content

Add devcontainer integration - #544

Merged
jackpot51 merged 2 commits into
masterfrom
devcontainer
Sep 23, 2026
Merged

jackpot51 merged 2 commits into
masterfrom
devcontainer

Conversation

@crawfxrd

@crawfxrd crawfxrd commented Nov 24, 2025

Copy link
Copy Markdown
Member

Basic devcontainer integration and config for VSCode.

Installed VSCode to make sure it works, but I don't use it. Don't know if adding other extensions would be useful, like a Makefile and C/C++ extension (esp. since we use SDCC dialect of C for 8051).

@crawfxrd
crawfxrd force-pushed the devcontainer branch 2 times, most recently from c985211 to f4582dc Compare November 24, 2025 17:33
@crawfxrd
crawfxrd marked this pull request as ready for review November 25, 2025 15:50
@crawfxrd
crawfxrd requested review from a team November 25, 2025 15:50
leviport
leviport previously approved these changes Nov 25, 2025
@crawfxrd

Copy link
Copy Markdown
Member Author
  • SDCC toolchain is now provided as a separate container that must be built first.
  • Fixes compiling SDCC 4.4.0 by using gcc-13 instead of default (gcc-14).
  • Updates the base Debian image.

leviport
leviport previously approved these changes Jan 6, 2026
@crawfxrd

crawfxrd commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

Debian Trixie has SDCC 4.5.0, so drop building 4.4.0 from source and use the distro-provided package.

Without having to build SDCC, the local Containerfile is viable to use without having to publish and maintain images.

- Update base image to debian:trixie-20260713-slim
- Use manifest digest as part of image ID
- Drop building SDCC and use distro-provided package
  - Updates SDCC from 4.4.0 to 4.5.0

Signed-off-by: Tim Crawford <tcrawford@system76.com>
Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd
crawfxrd requested a review from a team September 23, 2026 19:37
@jackpot51
jackpot51 merged commit 8faf1c2 into master Sep 23, 2026
49 checks passed
@jackpot51
jackpot51 deleted the devcontainer branch September 23, 2026 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants