Skip to content

Update Dockerfile #278

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

Closed
wants to merge 1 commit into from
Closed

Update Dockerfile #278

wants to merge 1 commit into from

Conversation

djacobs7
Copy link

Dockerfile did not work out of the box. These were necessary changes to get editly running in Docker as far as I could tell. It's quite possible I was doing something wrong, but I thought it might save some you or someone else the trouble of having to debug this again

ERROR 1
First error is that you see this when you try to run editly:

ERROR Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only file and data URLs are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'

Fix:
Put in a minimum NODE_VERSION ( as documented here: https://stackoverflow.com/questions/69665780/error-err-unsupported-esm-url-scheme-only-file-and-data-urls-are-supported-by )

Error 2

internal/modules/cjs/loader.js:1250

return process.dlopen(module, path.toNamespacedPath(filename));

Fix:

Dockerfile did not work out of the box.  These were necessary changes to get editly running

* Put in a minimum NODE_VERSION  for ESM support
* Add LD_LIBRARY_PATH for canvas support
@bkeepers bkeepers mentioned this pull request Jan 18, 2025
3 tasks
@bkeepers
Copy link
Collaborator

@djacobs7 thanks for your pull request. I did a bigger rewrite of the Dockerfile in #312 to use the official node images, and to add a build stage so it doesn't depend on precompiled binaries for canvas/gl.

Woudl you be willing to give it a try and let me know if it works for you now?

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