Skip to content

Link all Debian mirrors from the same mirror+file source together#836

Open
mgedmin wants to merge 3 commits into
furlongm:mainfrom
mgedmin:link-debian-mirrors
Open

Link all Debian mirrors from the same mirror+file source together#836
mgedmin wants to merge 3 commits into
furlongm:mainfrom
mgedmin:link-debian-mirrors

Conversation

@mgedmin
Copy link
Copy Markdown

@mgedmin mgedmin commented Jun 1, 2026

Rework the Debian repository determination algorithm to link mirrors together when they come from the same source.

The dist_repos/nondist_repos logic was moved into a case statement based on $arch, and is now also applied to repos that use mirror+file.

The title of a repo now bases the architecture on the repo itself, rather than hardcoding it to host_arch. The architecture is omitted entirely when it's 'all' or missing.

Closes #834.

Rework the Debian repository determination algorithm to link mirrors
together when they come from the same source.

The dist_repos/nondist_repos logic was moved into a case statement based
on $arch, and is now also applied to repos that use mirror+file.

The title of a repo now bases the architecture on the repo itself,
rather than hardcoding it to host_arch.  The architecture is omitted
entirely when it's 'all' or missing.

Closes furlongm#834.
@mgedmin mgedmin marked this pull request as draft June 1, 2026 08:24
@mgedmin mgedmin marked this pull request as ready for review June 1, 2026 08:27
Comment thread client/patchman-client
Gedminas, Marius added 2 commits June 1, 2026 13:41
This was a regression introduced in the previous commit, where a
mirror+file containing

   http://archive.ubuntu.com/ubuntu/

(with a trailing slash) would result in mirror URLs looking like

   http://archive.ubuntu.com/ubuntu//dists/noble/main/binary-amd64

instead of

   http://archive.ubuntu.com/ubuntu/dists/noble/main/binary-amd64
Two lines starting with `local urls=...` looked too much like a
copy/paste error.
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.

Automatic linking of Debian mirrors that use mirror+file

1 participant