Skip to content

Stamp the MobilityDB source commit into meos-idl.json#50

Merged
estebanzimanyi merged 1 commit into
masterfrom
feat/catalog-source-commit-stamp
Jul 11, 2026
Merged

Stamp the MobilityDB source commit into meos-idl.json#50
estebanzimanyi merged 1 commit into
masterfrom
feat/catalog-source-commit-stamp

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

Records the MobilityDB commit the catalog is derived from as a top-level sourceCommit field, making the catalog self-describing about its freshness.

A consumer — a binding, or a staleness check — proves the catalog is current by comparing sourceCommit to live upstream master, rather than inferring freshness from whatever directory a vendored copy sits in. That directory proxy is structurally blind to a catalog vendored inside an unrelated binding repository, whose git HEAD is independent of the catalog's generation.

sourceCommit resolves from the git working tree that provides the source headers, and is null when the source is not a git working tree (e.g. a release tarball) — freshness is then unprovable, the correct safe default.

The change is additive: it introduces one top-level key and alters no existing field or schema.

Record the MobilityDB commit the catalog was derived from as a top-level
`sourceCommit` field, so the catalog is self-describing about its freshness.
A consumer (a binding, or a staleness gate) proves the artifact is current by
comparing sourceCommit to live upstream master, rather than inspecting whatever
directory a vendored copy happens to sit in — a proxy that is blind to a catalog
vendored inside an unrelated binding repository.

Resolved from the git checkout that provided the source; null when the source is
not a git checkout (e.g. a release tarball), in which case freshness is treated
as unprovable — the correct safe default.
@estebanzimanyi estebanzimanyi merged commit 6fc7480 into master Jul 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant