Skip to content

pkg/*: deb: remove redundant override_dh_gencontrol#452

Merged
thaJeztah merged 1 commit into
docker:mainfrom
thaJeztah:rm_remaining_override
Jun 12, 2026
Merged

pkg/*: deb: remove redundant override_dh_gencontrol#452
thaJeztah merged 1 commit into
docker:mainfrom
thaJeztah:rm_remaining_override

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Jun 12, 2026

Copy link
Copy Markdown
Member

pkg/*: deb: remove redundant override_dh_gencontrol

The --remaining-packages option was added in docker-ce-packaging@3e78175,
which manually called dh_gencontrol for a single package, after which it
had to call dh_gencontrol --remaining-packages to process the remaining
packages as usual.

This override was removed, so dh_gencontrol --remaining-packages is no
longer needed, and we can remove the override.

This option was copied to all other packages, which never needed it, so
removing it there as well.

The `--remaining-packages` option was added in [docker-ce-packaging@3e78175],
which manually called `dh_gencontrol` for a single package, after which it
had to call `dh_gencontrol --remaining-packages` to process the remaining
packages as usual.

This override was removed, so `dh_gencontrol --remaining-packages` is no
longer needed, and we can remove the override.

This option was copied to all other packages, which never needed it, so
removing it there as well.

[docker-ce-packaging@3e78175]: docker/docker-ce-packaging@3e78175

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the rm_remaining_override branch from 47ea41a to c16e33a Compare June 12, 2026 17:31
@thaJeztah thaJeztah marked this pull request as ready for review June 12, 2026 17:32
@thaJeztah thaJeztah requested a review from a team as a code owner June 12, 2026 17:32
@thaJeztah thaJeztah merged commit 32868dd into docker:main Jun 12, 2026
386 checks passed
@thaJeztah thaJeztah deleted the rm_remaining_override branch June 12, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants