Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compilation for PLATFORM_WEB #3328

Closed
wants to merge 0 commits into from
Closed

Fix compilation for PLATFORM_WEB #3328

wants to merge 0 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Sep 20, 2023

PR changes

  1. Fixes compilation for PLATFORM_WEB (R201). #include "GLFW/glfw3.h" (R204) is necessary for CoreData's GLFWwindow *handle; (R282) on PLATFORM_WEB.
  2. Temporarily removes EM_ASM_INT from core_input_gestures_web example (R311-R318). Doing this so it doesn't block compiling the examples/core. I'll come back and review this specific example later.

Current PLATFORM_WEB status

  • Compiling raylib
  • Compiling examples/audio
  • Compiling examples/core
  • Compiling examples/models
  • Compiling examples/others
  • Compiling examples/shaders
  • Compiling examples/shapes
  • Compiling examples/text
  • Compiling examples/textures

Notes

  • examples/others has some very singular/specific examples. I'll review them last.
  • I'll now start reviewing each function for web.

Environment

  • Compiled with emscripten/emsdk on Linux (Mint 21.1 64-bit).
  • Tested on Firefox (115.1.0esr 64-bit) and Chromium (115.0.5790.170 64-bit) both running on Linux Mint (21.1 64-bit).

Edit 1: added line marks.
Edit 2: added core_input_gestures_web example temporary fix; added current web status, environment and notes; formatting.

@@ -308,13 +308,14 @@ int main(void)
{
// Initialization
//--------------------------------------------------------------------------------------
#if defined( PLATFORM_WEB )
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It can probably be removed, it shouldn't be here.

@ghost ghost closed this Sep 20, 2023
@ghost
Copy link
Author

ghost commented Sep 20, 2023

@raysan5 Sorry. I synced my repo and ended up messing this PR. I will resend it shortly.

This pull request was closed.
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.

1 participant