On Mon, Oct 18, 2021 at 10:33:59AM +0200, Kamil Dudka wrote:
Here dracut allows ftp://, by calling curl. If curl does not support
the protocol, this will fail… I think this is what we want.
(BTW, I think the dracut repo on
git.kernel.org seems stale…
https://github.com/dracutdevs/dracut has commits from this week, but
kernel.org ends at 2021-05-27.)
I
think the story would be the same here: if an url with ftp:// was actually
configured somewhere, the download would fail. But I don't think we have many
such urls...
This calls %_urlhelper, i.e. '/usr/bin/curl --silent --show-error --fail
--globoff --location -o',
so it will fail on ftp://.
Anyway, with current libcurl-minimal, dnf and rpm both seem to download
ftp:// urls just fine… (I used [2] for testing since we don't advertise ftp
anywhere.)
--
To move this along, I pushed Suggests:libcurl-minimal to systemd-journal-remote
and systemd-container [1], because those two systemd subpackages Require libcurl.so.4.
This saves 5MB (out of 105MB) when installing systemd-journal-remote into an
empty chroot. But those two subpackages are optional so we might get full libcurl
when something else pulls it in… I think it'd make sense to do this at some lower
level so that we always get libcurl-minimal by default.
If we added Suggests:libcurl-minimal in filesystem.rpm, it'd be prefered
everywhere. But we might also get libcurl-minimal if somebody installs
filesystem.rpm with weak dependencies, which we don't want. So I think this
should be only added in packages that actually pull in libcurl. But I don't
see any great candidate, since none of the low-level packages use libcurl.
[1]
https://src.fedoraproject.org/rpms/systemd/c/2731a22179be6db678bde386ae3f...
[2]
ftp://ftp.pbone.net/mirror/archive.fedoraproject.org/fedora/linux/release...
Zbyszek