]> git.ipfire.org Git - thirdparty/kmod.git/commit
Swap rsync for cp --archive for module sources copying
authorEmil Velikov <emil.l.velikov@gmail.com>
Thu, 17 Oct 2024 20:30:08 +0000 (21:30 +0100)
committerLucas De Marchi <lucas.de.marchi@gmail.com>
Thu, 17 Oct 2024 21:57:43 +0000 (16:57 -0500)
commite16d92b429fe01bfd50468b35bbf39fef6b678c1
tree61ad4c74d82aaea56b181b637e45824eed41baad
parent7b416d59e4505b8f0a59dd0111ed738b79de2629
Swap rsync for cp --archive for module sources copying

As Tobias reported, rsync is a bit of heavyweight dependency. We
introduced it, as a replacement for the rm/cp -r previously used.

The rsync was inspired since, unlike make, meson will build all the test
binaries/artefacts even without calling "meson test".

We can go back to cp with --archive (--preserve=timestamps at least),
which will ensure we don't get stale files. To ensure the second run
doesn't copy the source folder as _subfolder_ of the dest we need to
wildcard the copy... Plus we need a proper destination folder in the
first place.

With this, we get a no-op second+ builds - be that with meson or make.
Since the explicit always-dirty state is by design, drop the meson TODO
and document the output variable.

Confirmed by comparing both the `make --debug` output and the execution
times.

Reported-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Link: https://github.com/kmod-project/kmod/pull/192
Signed-off-by: Lucas De Marchi <lucas.de.marchi@gmail.com>
.github/actions/setup-alpine/action.yml
.github/actions/setup-archlinux/action.yml
.github/actions/setup-debian/action.yml
.github/actions/setup-fedora/action.yml
.github/actions/setup-ubuntu/action.yml
README.md
scripts/setup-modules.sh
testsuite/meson.build