Skip to content

Fix definition of OAPV_STATIC_DEFINE in the pkg-config for static only builds#103

Merged
dariusz-f merged 1 commit intoAcademySoftwareFoundation:mainfrom
jamrial:exports
Jun 11, 2025
Merged

Fix definition of OAPV_STATIC_DEFINE in the pkg-config for static only builds#103
dariusz-f merged 1 commit intoAcademySoftwareFoundation:mainfrom
jamrial:exports

Conversation

@jamrial
Copy link
Contributor

@jamrial jamrial commented Jun 7, 2025

The check in oapv.h will fail otherwise on builds with no shared libs, as oapv_exports.h is not installed and OAPV_STATIC_DEFINE will be defined only if pkg-config is invoked with --static.

…y builds

The check in oapv.h will fail otherwise on builds with no shared libs, as
oapv_exports.h is not installed and OAPV_STATIC_DEFINE will be defined only if
pkg-config is invoked with --static.

Signed-off-by: James Almer <jamrial@gmail.com>
Copy link
Collaborator

@dkozinski dkozinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. It has been tested.

@dariusz-f dariusz-f requested a review from kpchoi June 10, 2025 06:34
Copy link
Collaborator

@kpchoi kpchoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dariusz-f dariusz-f merged commit bbc4316 into AcademySoftwareFoundation:main Jun 11, 2025
5 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.

4 participants