FS#58568 - [devtools] host_mirror parsing fails on flat mirror

Attached to Project: Arch Linux
Opened by deusstultus (deusstultus) - Saturday, 12 May 2018, 21:21 GMT
Last edited by Eli Schwartz (eschwartz) - Tuesday, 29 May 2018, 00:59 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Eli Schwartz (eschwartz)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

When establishing the mirror to use within the chroot, the expression used assumes that all mirrors will be of the form "/$repo/os/$arch". If using custom local serving of a package cache from /var/cache/pacman/pkg, the current sed operation won't do anything and propagates the mirror including devtools.*.tar.xz

Attached patch is a quick approach to eliminate the tarball while preserving the desired replacement. adds support for a flat mirror approach, but maintains the other assumptions on the mirror topology which could break in alternate configurations.

Additional info:
* Version: 20171108-2

Steps to reproduce:
* pacman -Syw base-devel #Ensure package cache is up to date
* Add Server=http:/127.0.0.1:8080/archlinux to head of /etc/pacman.d/mirrorlists on host
* setup new chroot
* cat "${chroot_path}"/etc/pacman.d/mirrorlist
This task depends upon

Closed by  Eli Schwartz (eschwartz)
Tuesday, 29 May 2018, 00:59 GMT
Reason for closing:  Fixed
Additional comments about closing:  testing/devtools 20180528-1
Comment by deusstultus (deusstultus) - Saturday, 12 May 2018, 21:25 GMT Comment by Eli Schwartz (eschwartz) - Sunday, 13 May 2018, 07:23 GMT
Shared pacman cache has nothing to do with modifying the repo url topology. The cache isn't what pacman downloads from...

Also the patch is horribly broken with the very very closely approaching pacman 5.1 release, for the same reason devtools itself is broken. See https://patchwork.archlinux.org/patch/348/

Note: using pacman-conf, which is the *right* technical solution, means deriving the url directly from pacman.conf, and the sed will either undo the $repo/$arch variable resolution, or be a no-op.
Comment by deusstultus (deusstultus) - Tuesday, 29 May 2018, 00:46 GMT
Eli's patchwork was merged May 28th, resolving this issue. May raise secondary concerns for those using such mirrors which are not sufficiently complete, but failing out due to sources not being available from the specified mirror is a valid error and easier to trace.

Loading...