Skip to content

ci/circle: add ffi cdecls check#2039

Merged
Frenzie merged 6 commits into
koreader:masterfrom
benoit-pierre:pr/ci_ffi_cdecls_check
Mar 31, 2025
Merged

ci/circle: add ffi cdecls check#2039
Frenzie merged 6 commits into
koreader:masterfrom
benoit-pierre:pr/ci_ffi_cdecls_check

Conversation

@benoit-pierre

@benoit-pierre benoit-pierre commented Feb 20, 2025

Copy link
Copy Markdown
Member

Verify (most) ffi cdecls are up-to-date.


This change is Reviewable

@benoit-pierre benoit-pierre force-pushed the pr/ci_ffi_cdecls_check branch 3 times, most recently from 83838a3 to 2ce3048 Compare February 21, 2025 18:45
To avoid differences if the build directory or arguments to ffi-cdecl change.
To avoid differences when the build directory or arguments to ffi-cdecl
change, and since rules for easily updating most ffi cdecls have been
added to the build system.
@benoit-pierre benoit-pierre marked this pull request as ready for review February 26, 2025 22:05
@NiLuJe

NiLuJe commented Mar 5, 2025

Copy link
Copy Markdown
Member

Looks good at a quick glance, but on mobile right now.

@Frenzie Frenzie merged commit 686541a into koreader:master Mar 31, 2025
@benoit-pierre benoit-pierre deleted the pr/ci_ffi_cdecls_check branch March 31, 2025 20:22
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.

3 participants