Skip to content

Feature/enable-standard-install#32

Open
jeanbaptistelab wants to merge 4 commits intomasterfrom
feature/enable-standard-install
Open

Feature/enable-standard-install#32
jeanbaptistelab wants to merge 4 commits intomasterfrom
feature/enable-standard-install

Conversation

@jeanbaptistelab
Copy link

Currently, installing CSP through meson does not allow other modules to use the installed version as it does not generate the required pkg-config files, this PR addresses this

@jeanbaptistelab jeanbaptistelab self-assigned this Dec 4, 2025
@jeanbaptistelab jeanbaptistelab force-pushed the feature/enable-standard-install branch from 99d000b to 2fa48a1 Compare December 4, 2025 08:46
Currently, installing CSP through meson does not allow other
modules to use the installed version as it does not generate
the required pkg-config files.
@jeanbaptistelab jeanbaptistelab force-pushed the feature/enable-standard-install branch from 2fa48a1 to 289cbcd Compare December 4, 2025 08:48
"runtime" will install the shared library only,
"devel" will install the static library, headers and
pkg-config files
@troelsjessen
Copy link

I suppose that is fine...
Modules are building with this variant

@kivkiv12345
Copy link

With this branch I can successfully build libparam as a shared object, with Meson finding the installed libcsp dependency.

But it does point to our (perhaps incorrect) usage of .as_link_whole().
The .as_link_whole() method is only supported on declare_dependency() objects, which the following 'system installed' dependency doesn't appear to be:

csp_dep = dependency('csp', fallback: ['csp', 'csp_dep'], required: true)

So the following is not allowed:

csp_dep = dependency('csp', fallback: ['csp', 'csp_dep'], required: true).as_link_whole()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants