Revert "ci: cache packages"
This reverts commit e348ba3881
.
With the above commit we started caching the downloaded packages. Based
on some testing and, it saves ~30s in the "step_script" stage while
adding 18s for "Restoring/Saving cache". A net saving of ~10s.
With earlier commit, we no longer use an ancient image which also pulls
base-devel - thus the packages we have to download is minimal.
Now comparing the uncached "step_script", vs the cached one - it is
slowed by 2-3 seconds (1:01 -> 1:03), while we eliminate the 18s (and
growing) caching.
Tl:Dr: With up-to date image, package caching in not worth it - be that
time, disk or network wise.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Signed-off-by: Allan McRae <allan@archlinux.org>
This commit is contained in:
parent
cc1d23d333
commit
17d3da4777
@ -1,14 +1,6 @@
|
|||||||
variables:
|
variables:
|
||||||
MAKEFLAGS: "-j10"
|
MAKEFLAGS: "-j10"
|
||||||
VERBOSE: 1
|
VERBOSE: 1
|
||||||
PACMAN_OPTS: --needed --noconfirm --cachedir .pkg-cache
|
|
||||||
PACTEST_OPTS: --review --editor=../build-aux/cat-test-file
|
|
||||||
|
|
||||||
cache:
|
|
||||||
key: pkgs-v1
|
|
||||||
paths:
|
|
||||||
# For some reason Gitlab CI only supports storing cache/artifacts in a path relative to the build directory
|
|
||||||
- .pkg-cache
|
|
||||||
|
|
||||||
default:
|
default:
|
||||||
after_script:
|
after_script:
|
||||||
@ -18,7 +10,7 @@ default:
|
|||||||
image: archlinux:base-devel
|
image: archlinux:base-devel
|
||||||
before_script:
|
before_script:
|
||||||
- >
|
- >
|
||||||
pacman -Syu $PACMAN_OPTS
|
pacman -Syu --needed --noconfirm
|
||||||
git
|
git
|
||||||
gpgme libarchive curl
|
gpgme libarchive curl
|
||||||
python
|
python
|
||||||
@ -42,14 +34,14 @@ arch-debug:
|
|||||||
arch-docs:
|
arch-docs:
|
||||||
extends: .arch-test
|
extends: .arch-test
|
||||||
script:
|
script:
|
||||||
- pacman -Syu $PACMAN_OPTS asciidoc
|
- pacman -Syu --needed --noconfirm asciidoc
|
||||||
- meson -Ddoc=enabled build
|
- meson -Ddoc=enabled build
|
||||||
- ninja -C build
|
- ninja -C build
|
||||||
|
|
||||||
arch-clang:
|
arch-clang:
|
||||||
extends: .arch-test
|
extends: .arch-test
|
||||||
script:
|
script:
|
||||||
- pacman -Syu $PACMAN_OPTS clang
|
- pacman -Syu --needed --noconfirm clang
|
||||||
- CC=clang meson build
|
- CC=clang meson build
|
||||||
- ninja -C build
|
- ninja -C build
|
||||||
- fakechroot meson test -C build
|
- fakechroot meson test -C build
|
||||||
@ -92,7 +84,6 @@ arch-no-nls:
|
|||||||
|
|
||||||
debian:
|
debian:
|
||||||
image: debian:bullseye
|
image: debian:bullseye
|
||||||
cache: {}
|
|
||||||
before_script:
|
before_script:
|
||||||
- apt update
|
- apt update
|
||||||
- >
|
- >
|
||||||
@ -108,7 +99,6 @@ debian:
|
|||||||
|
|
||||||
fedora:
|
fedora:
|
||||||
image: fedora
|
image: fedora
|
||||||
cache: {}
|
|
||||||
before_script:
|
before_script:
|
||||||
- >
|
- >
|
||||||
dnf -y install
|
dnf -y install
|
||||||
|
Loading…
Reference in New Issue
Block a user