Skip to content

Upgrade to CMocka 1.1.8#2

Open
DaStoned wants to merge 1 commit into
OlivierLDff:masterfrom
DaStoned:master
Open

Upgrade to CMocka 1.1.8#2
DaStoned wants to merge 1 commit into
OlivierLDff:masterfrom
DaStoned:master

Conversation

@DaStoned

Copy link
Copy Markdown

Upgrade to v1.1.8 where they've renamed both the option to build a static lib and the name of library target

@DaStoned

DaStoned commented Dec 9, 2025

Copy link
Copy Markdown
Author

Hi @OlivierLDff - Proposing an update to latest CMocka v1 revision 1.1.8.
I see they've released v2 last week, but I'll leave that upgrade to someone else.

@OlivierLDff

Copy link
Copy Markdown
Owner

Yes better wait for a v2 upgrade, with appropriate CI, I can't take PR in without a working CI backing up the fact it's working

@DaStoned

DaStoned commented Dec 9, 2025

Copy link
Copy Markdown
Author

Fair enough. CMocka folks renamed two things since 1.1.5 - the option for static builds and the library itself. Public documentation only mentions the first change (WITH_STATIC_LIB ON -> BUILD_SHARED_LIBS OFF). It took me a good hour of debugging to discover the second, undocumented change (cmocka-static -> cmocka). Maybe the next guy will find this PR helpful.

@OlivierLDff

Copy link
Copy Markdown
Owner

yes I'm leaving your PR open on purpose for that

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.

2 participants