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

[Examples] Added single file option to examples #4755

Closed

Conversation

CrackedPixel
Copy link
Contributor

No description provided.

@CrackedPixel CrackedPixel marked this pull request as draft February 7, 2025 11:43
@CrackedPixel
Copy link
Contributor Author

@raysan5 what do you think of this? it will make single .html files that can be viewed without a web server

Comment on lines +62 to +63
BUILD_WEB_SINGLE_FILE ?= TRUE
BUILD_WEB_FILE_MODE ?= --preload-file
Copy link
Contributor Author

Choose a reason for hiding this comment

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

a single file requires --embed-file instead

@CrackedPixel CrackedPixel force-pushed the add-single-file-option branch from 4b77bd5 to c145587 Compare February 7, 2025 11:45
@@ -59,6 +59,8 @@ USE_WAYLAND_DISPLAY ?= FALSE
BUILD_WEB_ASYNCIFY ?= TRUE
BUILD_WEB_SHELL ?= $(RAYLIB_PATH)/src/shell.html
BUILD_WEB_HEAP_SIZE ?= 134217728
BUILD_WEB_SINGLE_FILE ?= TRUE
Copy link
Contributor Author

Choose a reason for hiding this comment

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

default to false?

@raysan5
Copy link
Owner

raysan5 commented Feb 8, 2025

@CrackedPixel Definitely single-html is an interesting option... but it also seems a non-standard approach. I prefer to not expose that option for the users for now... It seems very user-specific for certain situations...

Still, I think some note/comment could be added somewhere, maybe on emscripten parameters details section, to explain users this is a possibility.

@CrackedPixel CrackedPixel deleted the add-single-file-option branch February 11, 2025 07:34
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