Skip to content

Commit c49ef8d

Browse files
committed
replace git and git-lfs features with apt calls
this dramatically speeds up the builds, and the exact tool versions probably do not matter that much.
1 parent 8aead37 commit c49ef8d

6 files changed

Lines changed: 14 additions & 24 deletions

File tree

src/s-core-devcontainer/.devcontainer/devcontainer-lock.json

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,6 @@
1010
"resolved": "ghcr.io/devcontainers/features/common-utils@sha256:00fd45550f578d9d515044d9e2226e908dbc3d7aa6fcb9dee4d8bdb60be114cf",
1111
"integrity": "sha256:00fd45550f578d9d515044d9e2226e908dbc3d7aa6fcb9dee4d8bdb60be114cf"
1212
},
13-
"ghcr.io/devcontainers/features/git": {
14-
"version": "1.3.4",
15-
"resolved": "ghcr.io/devcontainers/features/git@sha256:f24645e64ad39a596131a50ec96f7d5cf7a2a87544cce772dd4b7182a233e98a",
16-
"integrity": "sha256:f24645e64ad39a596131a50ec96f7d5cf7a2a87544cce772dd4b7182a233e98a"
17-
},
18-
"ghcr.io/devcontainers/features/git-lfs": {
19-
"version": "1.2.5",
20-
"resolved": "ghcr.io/devcontainers/features/git-lfs@sha256:71c2b371cf12ab7fcec47cf17369c6f59156100dad9abf9e4c593049d789de72",
21-
"integrity": "sha256:71c2b371cf12ab7fcec47cf17369c6f59156100dad9abf9e4c593049d789de72"
22-
},
2313
"ghcr.io/devcontainers/features/rust": {
2414
"version": "1.5.0",
2515
"resolved": "ghcr.io/devcontainers/features/rust@sha256:0c55e65f2e3df736e478f26ee4d5ed41bae6b54dac1318c443e31444c8ed283c",

src/s-core-devcontainer/.devcontainer/devcontainer.json

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,6 @@
55
"context": "."
66
},
77
"features": {
8-
"ghcr.io/devcontainers/features/git": {
9-
"version": "2.49.0",
10-
"ppa": "true"
11-
},
12-
"ghcr.io/devcontainers/features/git-lfs": {
13-
"version": "3.7.0",
14-
"autoPull": "false" // do not automatically pull LFS files when creating the container
15-
},
168
"ghcr.io/devcontainers/features/common-utils": {
179
// Installs latest version from the Distribution
1810
"installZsh": "false",
@@ -42,8 +34,6 @@
4234
"overrideFeatureInstallOrder": [
4335
// this order makes it more convenient to develop the s-core-local feature, since it will be installed last
4436
// which means changes to it will be applied without needing to rebuild all other features
45-
"ghcr.io/devcontainers/features/git",
46-
"ghcr.io/devcontainers/features/git-lfs",
4737
"ghcr.io/devcontainers/features/common-utils",
4838
"ghcr.io/devcontainers-community/features/llvm",
4939
"ghcr.io/devcontainers/features/rust",

src/s-core-devcontainer/.devcontainer/s-core-local/install.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,10 @@ apt-get install -y graphviz="${graphviz_version}*"
3030
# Protobuf compiler, via APT (needed by FEO)
3131
apt-get install -y protobuf-compiler="${protobuf_compiler_version}*"
3232

33+
# Git and Git LFS, via APT
34+
apt-get install -y git
35+
apt-get install -y git-lfs
36+
3337
# Python, via APT
3438
apt-get install -y python${python_version} python3-pip python3-venv
3539
# The following packages correspond to the list of packages installed by the

src/s-core-devcontainer/.devcontainer/s-core-local/tests/test_default.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ check "validate graphviz has the correct version" bash -c "dpkg -s graphviz | gr
1313
# Other build-related tools
1414
check "validate protoc is working and has the correct version" bash -c "protoc --version | grep 'libprotoc ${protobuf_compiler_version}'"
1515

16+
# Common tooling
17+
check "validate git is working and has the correct version" bash -c "git --version | grep '${git_version}'"
18+
check "validate git-lfs is working and has the correct version" bash -c "git lfs version | grep '${git_lfs_version}'"
19+
1620
# Python-related tools (a selected sub-set; others may be added later)
1721
check "validate python3 is working and has the correct version" bash -c "python3 --version | grep '${python_version}'"
1822
check "validate pip3 is working and has the correct version" bash -c "pip3 --version | grep '${python_version}'"

src/s-core-devcontainer/.devcontainer/s-core-local/versions.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,12 @@ qemu_system_arm:
1010
sshpass:
1111
version: 1.09
1212

13+
git:
14+
version: "2.43.0"
15+
16+
git_lfs:
17+
version: "3.4.1"
18+
1319
python:
1420
version: "3.12"
1521

src/s-core-devcontainer/test-project/test.sh

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,6 @@ SCRIPT_PATH=$(readlink -f "$0")
55
SCRIPT_DIR=$(dirname -- "${SCRIPT_PATH}")
66
source "${SCRIPT_DIR}/../../../scripts/test-utils.sh" vscode
77

8-
# Common tooling
9-
check "validate git is working and has the correct version" bash -c "git --version | grep '2.49.0'"
10-
check "validate git-lfs is working and has the correct version" bash -c "git lfs version | grep '3.7.0'"
11-
128
# C++ tooling
139
check "validate clangd is working and has the correct version" bash -c "clangd --version | grep '20.1.8'"
1410
check "validate clang-format is working and has the correct version" bash -c "clang-format --version | grep '20.1.8'"

0 commit comments

Comments
 (0)