Skip to content

fix(devcontainer): package libgl1 in the image - #8806

Open
sankalpsthakur wants to merge 2 commits into
electricitymaps:masterfrom
sankalpsthakur:fix/devcontainer-libgl-7350
Open

fix(devcontainer): package libgl1 in the image#8806
sankalpsthakur wants to merge 2 commits into
electricitymaps:masterfrom
sankalpsthakur:fix/devcontainer-libgl-7350

Conversation

@sankalpsthakur

@sankalpsthakur sankalpsthakur commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a .devcontainer/Dockerfile that extends the existing Python Bookworm base image and installs libgl1.
  • Switch devcontainer.json from a prebuilt image to building that Dockerfile so LibGL is available without a manual apt-get workaround.

Uses libgl1 (not libgl1-mesa-glx) because the base image is Debian Bookworm and parsers.dockerfile already installs libgl1 for the same OpenCV/LibGL dependency.

Fixes #7350

Test plan

  • Rebuild the dev container (Dev Containers: Rebuild Container)
  • Confirm libgl1 is present (dpkg -l libgl1)
  • Run a parser that previously failed with a LibGL error and confirm it no longer fails for that reason

AI/LLM disclosure

  • AI coding tools (including Grok and/or Codex agent-assisted editing) were used to help draft or modify code and this PR description.
  • I reviewed the complete change, understand the reasoning, and ran the reported local tests before submitting.
  • This submission is original work of authorship under the project CLA / contributor terms; AI output was not pasted unreviewed.

@sankalpsthakur
sankalpsthakur force-pushed the fix/devcontainer-libgl-7350 branch from 7aadebd to cd40485 Compare July 28, 2026 10:12
LibGL is required when running parser code that depends on OpenCV;
package it in the dev container image so contributors don't need a
manual apt install. Use libgl1 to match parsers.dockerfile on Bookworm.

Fixes electricitymaps#7350
@sankalpsthakur
sankalpsthakur force-pushed the fix/devcontainer-libgl-7350 branch from cd40485 to 567c1af Compare August 12, 2026 16:42
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.

LibGL required library should be packaged as part of the dev container

1 participant