FS#67167 - [archiso] cosmetic changes to archiso

Attached to Project: Release Engineering
Opened by AMM (amish) - Wednesday, 01 July 2020, 15:02 GMT
Last edited by David Runge (dvzrv) - Saturday, 11 July 2020, 13:52 GMT
Task Type Bug Report
Category ArchISO
Status Closed
Assigned To David Runge (dvzrv)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
I appreciate the recent changes being made to archiso package. Happy with switch to systemd-networkd and addition of reflector service and other cool changes.

I would like to mention few cosmetic changes to archiso (as currently there is no way to send PR on gitlab)

1) packages.x86_64 file entries should be sorted. Currently lvm2 and reflector package are off by a line.

2) customize_airootfs.sh should keep permission of /root as 750 instead of 700. Because 750 is the permission as per "filesystem" package. No idea why it is being changed unnecessarily.

3) pacman.conf should be as similar as /etc/pacman.conf.
i.e. change XferCommand line to
#XferCommand = /usr/bin/curl -L -C - -f -o %o %u

Remove UseDelta line as pacman no longer supports it.

Thank you

Additional info:
* package version(s)
45
This task depends upon

Closed by  David Runge (dvzrv)
Saturday, 11 July 2020, 13:52 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed with https://gitlab.archlinux.org/archlinux/a rchiso/-/merge_requests/33
Comment by Eli Schwartz (eschwartz) - Wednesday, 01 July 2020, 15:39 GMT
  • Field changed: Attached to Project (Arch Linux → Release Engineering)
Project bug, not package bug.
Comment by David Runge (dvzrv) - Wednesday, 01 July 2020, 21:36 GMT
@amish: Thanks for the report!

1) is not super important, but will probably be cleaned up in a future merge request :)
2) we actually have the plan to remove the script altogether, if we can [1]
3) that's correct. I think in the future it would be best to use a packaged version of the configuration file and rely on pacman's "--cachedir" option [2].

[1] https://gitlab.archlinux.org/archlinux/archiso/-/issues/21
[2] https://jlk.fjfi.cvut.cz/arch/manpages/man/pacman.8#OPTIONS

Loading...