Skip to content

Configure the install prefix, not only the install - #93

Merged
estebanzimanyi merged 1 commit into
MobilityDB:masterfrom
estebanzimanyi:fix/provision-configures-the-install-prefix
Aug 29, 2026
Merged

Configure the install prefix, not only the install#93
estebanzimanyi merged 1 commit into
MobilityDB:masterfrom
estebanzimanyi:fix/provision-configures-the-install-prefix

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

Configure the install prefix, not only the install. meos.pc is generated from CMAKE_INSTALL_PREFIX at configure time, so a build configured for the default prefix names /usr/local wherever the files are later relocated to, and a consumer that follows the installed library's own pkg-config file reaches whatever occupies the machine-wide directory instead of the prefix this script installs into. That file is how a generated binding discovers both the include dir and the family macros the library carries, so a wrong prefix costs the macros too: the POINTCLOUD-guarded declarations stay invisible and the build reports undeclared symbols the library exports. A clean run of the patched script writes the parse prefix into meos.pc, pkg-config --cflags resolves to that prefix with all ten family macros, and the GoMEOS functions package builds against it driven by PKG_CONFIG_PATH alone.

meos.pc is generated from CMAKE_INSTALL_PREFIX at configure time, so a
build configured for the default prefix names /usr/local wherever the
files are later relocated to. A consumer that follows the installed
library's own pkg-config file, which is how a generated binding
discovers the include dir and the family macros it was built with, is
then sent to whatever occupies the machine-wide directory instead of the
prefix this script installed into. Configuring with the parse prefix
makes the file describe the tree it belongs to.
@estebanzimanyi
estebanzimanyi merged commit bad14f0 into MobilityDB:master Aug 29, 2026
1 check passed
@estebanzimanyi
estebanzimanyi deleted the fix/provision-configures-the-install-prefix branch August 29, 2026 12:59
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