Skip to content

Commit 8d2654a

Browse files
committed
CI: remove dbf from image/env — installed via auto-derive now
With ci.yml's auto-derive step in place (previous commit), dbf is installed at CI runtime from TMGimporter's .gpr.py requires_mod list. Keeping it baked into the Dockerfile and environment.yml in parallel would defeat the "single source of truth = .gpr.py" goal and drift the moment a new addon declares an additional dep. Remove dbf from both; leave the stable base (PyGObject, pycairo, Gramps, orjson, ruff) since those are not addon deps. Add a comment pointing readers at the auto-derive step so future edits do not re-bake runtime deps back in.
1 parent c6aa10e commit 8d2654a

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

.github/docker/gramps-ci/Dockerfile

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,17 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
3737
xauth \
3838
&& rm -rf /var/lib/apt/lists/*
3939

40+
# Addon runtime deps (dbf, networkx, lxml, svgwrite, boto3, etc.) are
41+
# NOT baked in here — ci.yml's "Install addon runtime deps (derived from
42+
# requires_mod)" step pip-installs them at CI runtime from every
43+
# .gpr.py's requires_mod list, matching what Gramps' Addon Manager does
44+
# for an end user. Keeps .gpr.py the single source of truth.
4045
RUN pip install --no-cache-dir \
4146
PyGObject \
4247
pycairo \
4348
"gramps>=6.0,<6.1" \
4449
orjson \
45-
ruff \
46-
dbf
50+
ruff
4751

4852
RUN apt-get purge -y gcc python3-dev pkg-config && apt-get autoremove -y
4953

.github/environment.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,10 @@ dependencies:
66
- pygobject
77
- gtk3
88
- pip
9+
# Addon runtime deps (dbf, networkx, lxml, svgwrite, boto3, etc.) are
10+
# installed at CI runtime by ci.yml's auto-derive step from .gpr.py
11+
# requires_mod — single source of truth. Keep only the stable base
12+
# here (Gramps + orjson for plugin registration).
913
- pip:
1014
- "gramps>=6.0,<6.1"
1115
- orjson
12-
- dbf

0 commit comments

Comments
 (0)