Skip to content

Drop Priority: optional, which is the default in Trixie#291

Merged
ottok merged 1 commit intoDebian:masterfrom
elboulangero:drop-priority-optional
Jan 12, 2026
Merged

Drop Priority: optional, which is the default in Trixie#291
ottok merged 1 commit intoDebian:masterfrom
elboulangero:drop-priority-optional

Conversation

@elboulangero
Copy link
Contributor

When omitted, the priority defaults to optional (since dpkg 1.22.13).

Cf. man deb-src-control.

Copy link
Contributor

@ottok ottok left a comment

Choose a reason for hiding this comment

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

Thanks, we should indeed do this!

@elboulangero
Copy link
Contributor Author

Let's hold on a bit for this one. I tried, and noticed that Lintian complains:

Running lintian...
W: golang-github-oschwald-maxminddb-golang-v2 source: recommended-field (in section for source) Priority [debian/control:1]
$ lintian-explain-tags recommended-field
N:
W: recommended-field
N: 
N:   The named field is recommended by policy but not present in the package's primary control file.
N: 
N:   Please refer to Source package control files – debian/control (Section 5.2) in the Debian Policy Manual, Binary package control files – DEBIAN/control (Section 5.3) in the Debian Policy Manual, and Debian changes files – .changes (Section
N:   5.5) in the Debian Policy Manual for details.
N: 
N:   Visibility: warning
N:   Show-Always: no
N:   Check: fields/recommended
N:   Renamed from: no-priority-field no-section-field-for-source no-section-field no-urgency-in-changes-file
N: 

Indeed, the Debian policy was not yet updated to reflect this change: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1110017

@ottok
Copy link
Contributor

ottok commented Jan 11, 2026

Debian policy 4.7.3.0 was released and it includes this, so we should merge this now.

When omitted, the priority defaults to optional (since dpkg 1.22.13).

Cf. `man deb-src-control`.
@elboulangero elboulangero force-pushed the drop-priority-optional branch from 188f8d3 to 6f05437 Compare January 12, 2026 03:55
@elboulangero
Copy link
Contributor Author

I tried to build a package without Priority: optional: no warning from Lintian. I think it can be merged now.

@ottok ottok merged commit 4033630 into Debian:master Jan 12, 2026
2 checks passed
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.

2 participants