Skip to content

Linux: enable HTML5 and Android builds, fix display.save() colours - #936

Open
chkuendig wants to merge 5 commits into
coronalabs:masterfrom
chkuendig:linux-builder-fixes
Open

Linux: enable HTML5 and Android builds, fix display.save() colours#936
chkuendig wants to merge 5 commits into
coronalabs:masterfrom
chkuendig:linux-builder-fixes

Conversation

@chkuendig

@chkuendig chkuendig commented Aug 14, 2026

Copy link
Copy Markdown

Everything Solar2DBuilder needs to build HTML5 and Android apps on Linux, plus
the a fix for broken colours in display.save() on the Simulator in Linux.

This supersedes #891, which enables the Android half.

I tested this using Docker on Linux and macOS hosts.

@chkuendig
chkuendig requested a review from Shchvova as a code owner August 14, 2026 16:15
@CLAassistant

CLAassistant commented Aug 14, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

The packager is already compiled in; only CORONABUILDER_HTML5 was missing, so
the binary answered that HTML5 builds are not supported on this operating
system.
Defines CORONABUILDER_ANDROID, adds the Rtt_AndroidSupportTools.c the factory
calls from inside that gate, gives the AndroidValidation path and
GetResourceDirectory() a Linux arm — the latter ran off the end of a function
returning const char*, killing the build in free() with no diagnostic.
The Android packager reads it from disk rather than through LUA_SOURCES, so
without it every build stops at "Could not find script file".
The paths start empty, so appending left the builder writing to /Documents and
its three siblings, which only root can create; they now go under TMPDIR, one
directory per uid (first reported in coronalabs#891).
A capture is tagged kBGRA but the packed readback lands in memory as A,R,G,B,
so the byte-order writers swapped red and green and took blue from the alpha
byte; the Linux writer now reports kARGB, which savePNG and saveJPG grow a case
for.
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