feat: add RAYLIB_PLATFORM env var for cross-platform targeting by tonyfettes · Pull Request #15 · moonbit-community/tonyfettes-raylib

tonyfettes · 2026-02-27T09:51:16Z

Summary

Adds RAYLIB_PLATFORM env var to build.js so the target platform can be overridden instead of always auto-detecting from os.platform()
Extends the platform if/else chain with android, drm, and web branches with correct link libs and stub_cc_flags
Guards the entire body of internal/raylib/rglfw.c with #if defined(PLATFORM_DESKTOP_GLFW) so it compiles to nothing on non-GLFW platforms, preventing errors when GLFW sources are unavailable

Supported `RAYLIB_PLATFORM` values

Value	Raylib platform	Backend
`darwin`	`PLATFORM_DESKTOP_GLFW`	GLFW + Cocoa
`linux`	`PLATFORM_DESKTOP_GLFW`	GLFW + X11
`win32`	`PLATFORM_DESKTOP_GLFW`	GLFW + Win32
`android`	`PLATFORM_ANDROID`	Android NDK EGL/GLES
`drm`	`PLATFORM_DRM`	Linux DRM/KMS
`web`	`PLATFORM_WEB`	Emscripten
(unset)	Auto-detect via `os.platform()`

Test plan

node build.js — auto-detect macOS, output unchanged (-DPLATFORM_DESKTOP_GLFW -ObjC)
RAYLIB_PLATFORM=linux node build.js — correct GL/X11 libs
RAYLIB_PLATFORM=android node build.js — correct NDK libs, -DPLATFORM_ANDROID
RAYLIB_PLATFORM=drm node build.js — correct DRM/EGL libs, -DPLATFORM_DRM
RAYLIB_PLATFORM=web node build.js — no link_libs, -DPLATFORM_WEB
RAYLIB_PLATFORM=freebsd node build.js — throws Unsupported platform error
moon -C examples build --target native raylib_demo/ — builds successfully on macOS

🤖 Generated with Claude Code

tonyfettes · 2026-02-27T10:09:20Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6b8ae2e54e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

build.js

tonyfettes · 2026-02-28T02:06:11Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 68e6a2669b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-02-28T02:09:49Z

build.js

+} else if (platform === 'web') {
+  // Emscripten handles linking; no link_libs needed
+  stub_cc_flags = '-DPLATFORM_WEB';


Enable Emscripten GLFW shim for web target

When RAYLIB_PLATFORM=web, this branch emits no link flags, but rglfw.c is now compiled out unless PLATFORM_DESKTOP_GLFW is defined, so the glfw* symbols used by internal/raylib/platforms/rcore_web.c must come from Emscripten’s GLFW port. The repository’s own web build script (tools/build_web_native_emcc.sh) passes -sUSE_GLFW=3 for this reason; without emitting an equivalent flag here, builds that rely on build.js config will hit unresolved GLFW symbols at link time.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-02-28T02:09:49Z

build.js

+  stub_cc_flags = '-DPLATFORM_DRM';
+} else if (platform === 'web') {
+  // Emscripten handles linking; no link_libs needed
+  stub_cc_flags = '-DPLATFORM_WEB';


Define WebGL graphics API for web builds

This sets only -DPLATFORM_WEB, but rlgl.h defaults to GRAPHICS_API_OPENGL_33 when no graphics API macro is provided, which drives the desktop GL path instead of WebGL/ES2. The repo’s validated web pipeline (tools/build_web_native_emcc.sh) explicitly adds -DGRAPHICS_API_OPENGL_ES2; without that define here, the new web platform mode can compile/link against the wrong GL backend and fail at runtime or during compilation depending on toolchain settings.

Useful? React with 👍 / 👎.

…eting Allow overriding the target platform via RAYLIB_PLATFORM env var instead of always auto-detecting from os.platform(). Adds support for android, drm, and web platforms with appropriate link_libs and stub_cc_flags. Guards rglfw.c body with #if defined(PLATFORM_DESKTOP_GLFW) so it compiles to nothing on non-GLFW platforms (android/drm/web), preventing link errors when GLFW sources are absent. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

Adds an override (RAYLIB_PLATFORM) to the prebuild/link configuration so the raylib native build can target non-host platforms (and avoids compiling bundled GLFW sources when GLFW isn’t the selected backend).

Changes:

Allow overriding os.platform() detection via process.env.RAYLIB_PLATFORM in build.js.
Add android, drm, and web platform branches with updated link libraries and stub compile flags.
Wrap all of internal/raylib/rglfw.c in #if defined(PLATFORM_DESKTOP_GLFW) to compile it out on non-GLFW platforms.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
build.js	Adds `RAYLIB_PLATFORM` override and new platform branches for link/stub flags.
internal/raylib/rglfw.c	Compiles bundled GLFW aggregator only when `PLATFORM_DESKTOP_GLFW` is defined.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

build.js

Without this flag, rlgl.h defaults to GRAPHICS_API_OPENGL_33 (desktop OpenGL), which is incorrect for these platforms that require OpenGL ES. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-04T06:33:19Z

build.js

+} else if (platform === 'android') {
+  link_config.link_libs = ['log', 'android', 'OpenSLES', 'EGL', 'GLESv2', 'm', 'dl'];
+  stub_cc_flags = '-DPLATFORM_ANDROID -DGRAPHICS_API_OPENGL_ES2';
+} else if (platform === 'drm') {
+  link_config.link_libs = ['drm', 'gbm', 'EGL', 'GLESv2', 'm', 'pthread', 'dl', 'rt'];
+  stub_cc_flags = '-DPLATFORM_DRM -DGRAPHICS_API_OPENGL_ES2';
+} else if (platform === 'web') {
+  // Emscripten handles linking; no link_libs needed
+  stub_cc_flags = '-DPLATFORM_WEB -DGRAPHICS_API_OPENGL_ES2';


For the new non-desktop branches, stub_cc_flags is reassigned (dropping -DPLATFORM_DESKTOP_GLFW). That matches the intent here, but it makes existing repo documentation inaccurate (e.g. CLAUDE.md says stub-cc-flags always includes -DPLATFORM_DESKTOP_GLFW). Please update the docs to reflect the new platform-dependent behavior so contributors don’t rely on the old invariant.

…forms The previous description said stub-cc-flags always includes -DPLATFORM_DESKTOP_GLFW, which is no longer true after adding android, drm, and web platform support. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

chatgpt-codex-connector bot reviewed Feb 27, 2026

View reviewed changes

build.js Outdated Show resolved Hide resolved

tonyfettes force-pushed the worktree-build-js branch from 6b8ae2e to 1e4ae83 Compare February 27, 2026 16:37

chatgpt-codex-connector bot reviewed Feb 28, 2026

View reviewed changes

tonyfettes force-pushed the worktree-build-js branch from 68e6a26 to eb69d02 Compare March 4, 2026 04:55

tonyfettes and others added 2 commits March 4, 2026 12:56

fix: link gbm for drm raylib platform

bdfb27f

tonyfettes force-pushed the worktree-build-js branch from eb69d02 to bdfb27f Compare March 4, 2026 04:56

tonyfettes requested a review from Copilot March 4, 2026 05:01

Copilot started reviewing on behalf of tonyfettes March 4, 2026 05:02 View session

Copilot AI reviewed Mar 4, 2026

View reviewed changes

build.js Outdated Show resolved Hide resolved

build.js Outdated Show resolved Hide resolved

build.js Outdated Show resolved Hide resolved

fix: define GRAPHICS_API_OPENGL_ES2 for android, drm, and web platforms

22b407f

Without this flag, rlgl.h defaults to GRAPHICS_API_OPENGL_33 (desktop OpenGL), which is incorrect for these platforms that require OpenGL ES. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

tonyfettes requested a review from Copilot March 4, 2026 06:29

Copilot started reviewing on behalf of tonyfettes March 4, 2026 06:29 View session

Copilot AI reviewed Mar 4, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add RAYLIB_PLATFORM env var for cross-platform targeting#15

feat: add RAYLIB_PLATFORM env var for cross-platform targeting#15
tonyfettes wants to merge 4 commits intomainfrom
worktree-build-js

tonyfettes commented Feb 27, 2026

Uh oh!

tonyfettes commented Feb 27, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

Uh oh!

tonyfettes commented Feb 28, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

tonyfettes commented Feb 27, 2026

Summary

Supported RAYLIB_PLATFORM values

Test plan

Uh oh!

tonyfettes commented Feb 27, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

tonyfettes commented Feb 28, 2026

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector bot Feb 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Supported `RAYLIB_PLATFORM` values