Skip to content

Add Debian standards version metadata - #1222

Merged
cpuguy83 merged 2 commits into
project-dalec:mainfrom
cpuguy83:cpuguy83-add-debian-standards-version
Sep 3, 2026
Merged

Add Debian standards version metadata#1222
cpuguy83 merged 2 commits into
project-dalec:mainfrom
cpuguy83:cpuguy83-add-debian-standards-version

Conversation

@cpuguy83

@cpuguy83 cpuguy83 commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Debian source ingestion requires Standards-Version, but DALEC-generated control metadata currently omits it.

  • define Debian Standards-Version 4.7.4 in the packaging package
  • emit it in the generated control source stanza
  • test the exact value, stanza placement, and single occurrence

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Copilot AI balanced review requested due to automatic review settings September 3, 2026 16:45
Signed-off-by: Brian Goff <cpuguy83@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

馃煝 Approval recommended

The metadata is correctly placed and covered by focused assertions.

Pull request overview

Adds Debian Standards-Version metadata to generated source control stanzas for PMC ingestion.

Changes:

  • Defines Standards-Version 4.7.4.
  • Emits it once in the source stanza.
  • Tests its value and placement.
File summaries
File Description
packaging/linux/deb/debroot.go Defines the standards version.
packaging/linux/deb/template_control.go Exposes the version to the template.
packaging/linux/deb/templates/debian_control.tmpl Emits the control field.
packaging/linux/deb/template_control_test.go Verifies value, placement, and occurrence count.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Balanced

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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