Skip to content

Sync Linux devcontainer package requirements#127460

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/sync-packages-in-dockerfile
Draft

Sync Linux devcontainer package requirements#127460
Copilot wants to merge 2 commits intomainfrom
copilot/sync-packages-in-dockerfile

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

The Linux devcontainer package list and Debian/Ubuntu setup docs had drifted. Both locations should describe the same build dependency set and remind maintainers to keep them aligned.

  • Package list sync

    • Updated .devcontainer/Dockerfile to include the union of packages from both sources.
    • Updated docs/workflow/requirements/linux-requirements.md Debian/Ubuntu package list and apt install command to match.
  • Maintenance note

    • Added cross-reference notes in both files to keep the package lists synchronized.

Agent-Logs-Url: https://github.com/dotnet/runtime/sessions/e01dfde2-e3fd-478e-aa3c-c270ffb6f75b

Co-authored-by: jtschuster <36744439+jtschuster@users.noreply.github.com>
Copilot AI requested review from Copilot and removed request for Copilot April 27, 2026 15:59
Copilot AI changed the title [WIP] Fix package mismatch between Dockerfile and requirements documentation Sync Linux devcontainer package requirements Apr 27, 2026
Copilot AI requested a review from jtschuster April 27, 2026 16:04
Comment thread .devcontainer/Dockerfile
# Keep this package list in sync with docs/workflow/requirements/linux-requirements.md.
RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
&& apt-get -y install --no-install-recommends \
build-essential \
Copy link
Copy Markdown
Member

@am11 am11 Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can just call eng/common/native/install-dependencies.sh to avoid duplications. We already use it in some build containers and pipelines.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot please update the Dockerfile to use the script mentioned above.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has the repo been cloned at this stage of the Dev Container build? I don’t think the script will be available.

@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-meta
See info in area-owners.md if you want to be subscribed.


The packages you need to install are shown in the following list:

**NOTE:** Keep this package list in sync with the Linux-based *Dockerfile* [over here](/.devcontainer/Dockerfile).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is user-facing workflow instructions. I don’t think it makes sense to put this comment in a document that is largely aimed at external contributors.

- `cmake` (version 3.26 or newer)
- `cpio`
- `curl`
- `gettext`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need these packages?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Packages installed in .devcontainer/Dockerfile do not match docs/workflow/requirements/linux-requirements.md

5 participants