Skip to content

luajit: fix library cross-compiling link arguments#1799

Merged
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_libluajit_cross_link_args
May 29, 2024
Merged

luajit: fix library cross-compiling link arguments#1799
Frenzie merged 1 commit into
koreader:masterfrom
benoit-pierre:pr/fix_libluajit_cross_link_args

Conversation

@benoit-pierre

@benoit-pierre benoit-pierre commented May 29, 2024

Copy link
Copy Markdown
Member

No real impact in practice, as we only ship the library on android, and in this case the luajit-launcher version is used.


This change is Reviewable

@benoit-pierre

Copy link
Copy Markdown
Member Author

Which by the way is a situation I would really like to address by using the base library: it does not make sense to compile 2 versions of luajit, and the less time spent in gradle land, the better.

@Frenzie

Frenzie commented May 29, 2024

Copy link
Copy Markdown
Member

Preferably while remaining independently usable, but if you know or want to get to know Gradle better you certainly have my blessing. Pinging @pazos for some tips, thoughts or possibly objections, since he's responsible for the modern Gradle/Kotlin Android renovation.

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