Skip to content

Resolve meos_catalog.c through MDB_SRC_ROOT for the installed-headers path#74

Merged
estebanzimanyi merged 1 commit into
MobilityDB:masterfrom
estebanzimanyi:fix/type-relations-mdb-src-root
Jul 23, 2026
Merged

Resolve meos_catalog.c through MDB_SRC_ROOT for the installed-headers path#74
estebanzimanyi merged 1 commit into
MobilityDB:masterfrom
estebanzimanyi:fix/type-relations-mdb-src-root

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

The object-model resolver returns the meos/src directory only when a source tree sits beside the headers. On the installed-headers build path the headers carry no source, so it returns none and the type-relation registry drops out — even though the provisioning checks out the full repository under MDB_SRC_ROOT.

This locates meos_catalog.c from the resolved source root when present, otherwise from MDB_SRC_ROOT/meos/src, so the registry attaches on both the source-headers and installed-headers build paths. A test covers the MDB_SRC_ROOT fallback, and the degrade test clears the env var so it still proves honest absence when neither source is available.

… path

The object-model resolver returns the meos/src directory only when a source
tree sits beside the headers. On the installed-headers build path the headers
carry no source, so it returns none and the type-relation registry drops out —
even though the provisioning checks out the full repository under MDB_SRC_ROOT.

Locate meos_catalog.c from the resolved source root when present, otherwise from
MDB_SRC_ROOT/meos/src, so the registry attaches on both build paths. A test
covers the MDB_SRC_ROOT fallback, and the degrade test clears the env var so it
still proves honest absence when neither source is available.
@estebanzimanyi
estebanzimanyi merged commit f5afcf8 into MobilityDB:master Jul 23, 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