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

[Bug]: Error [ERR_REQUIRE_ESM] #640

Closed
szkrstms opened this issue Jan 21, 2024 · 6 comments · Fixed by #651
Closed

[Bug]: Error [ERR_REQUIRE_ESM] #640

szkrstms opened this issue Jan 21, 2024 · 6 comments · Fixed by #651
Assignees
Labels
bug Something isn't working

Comments

@szkrstms
Copy link

What happened?

I installed from CLI (https://josephdadams.github.io/TallyArbiter/docs/installation/cli), but but it doesn't want to start on Raspberry Pi (Linux magic-pi 6.1.21-v7+ #1642 SMP Mon Apr 3 17:20:52 BST 2023 armv7l).
~ $ sudo n 20.10.0
installing : node-v20.10.0
mkdir : /usr/local/n/versions/node/20.10.0
fetch : https://nodejs.org/dist/v20.10.0/node-v20.10.0-linux-armv7l.tar.xz
copying : node/20.10.0
installed : v20.10.0 (with npm 10.2.3)

~ $ node --version
v20.10.0

~ $ sudo npm i -g tallyarbiter
npm WARN deprecated [email protected]: npm is holding this package for security reasons. As it's a core Node module, we will not transfer it over to other users. You may safely remove the package from your dependencies.

changed 332 packages in 5m

35 packages are looking for funding
run npm fund for details

~ $ tallyarbiter
[2024-01-21 10:29:37] info: Starting OSC Setup.
/usr/local/lib/node_modules/tallyarbiter/node_modules/esm/esm.js:1
const global = this;(function (require, module, shared) { var shared;.... (very long source code)

[/usr/local/lib/node_modules/tallyarbiter/node_modules/axios/index.js:1
Error [ERR_REQUIRE_ESM]: require() of ES Module /usr/local/lib/node_modules/tallyarbiter/node_modules/axios/index.js not supported.
Instead change the require of index.js in null to a dynamic import() which is available in all CommonJS modules.] {
code: 'ERR_REQUIRE_ESM'
}

Node.js v20.10.0

What do I wrong, or what missing?

Version

3.0.7

Distribution

CLI

OS

Linux

What browsers are you seeing the problem on?

Safari

If applicable, What Listener Clients are You Using?

GPI

TallyArbiter configuration

No response

Relevant log output

No response

Error stacktrace (if applicable)

No response

@szkrstms szkrstms added the bug Something isn't working label Jan 21, 2024
Copy link

Hello there szkrstms 👋

Welcome to TallyArbiter!

Thank you for opening your first issue for the Tally Arbiter project. Tally Arbiter fosters an open and welcoming environment for all our contributors. Please adhere to our Code Of Conduct.

If you have more to contribute to this issue, please comment down below! We will try to get back to you as soon as we can.

@Jocke4f
Copy link
Collaborator

Jocke4f commented Jan 23, 2024

The issue with

~ $ sudo npm i -g tallyarbiter
npm WARN deprecated [email protected]: npm is holding this package for security reasons. As it's a core Node module, we will not transfer it over to other users. You may safely remove the package from your dependencies.

is already handled.

The issue with

~ $ tallyarbiter
[2024-01-21 10:29:37] info: Starting OSC Setup.
/usr/local/lib/node_modules/tallyarbiter/node_modules/esm/esm.js:1
const global = this;(function (require, module, shared) { var shared;.... (very long source code)

[/usr/local/lib/node_modules/tallyarbiter/node_modules/axios/index.js:1
Error [ERR_REQUIRE_ESM]: require() of ES Module /usr/local/lib/node_modules/tallyarbiter/node_modules/axios/index.js not supported.
Instead change the require of index.js in null to a dynamic import() which is available in all CommonJS modules.] {
code: 'ERR_REQUIRE_ESM'
}

is caused by the change of axios from 0.27.2 to 1.6.0. If I understand it correctly is it an issue since later axios versions don't support require(). I don't know what is the correct way of solving this.

@Jocke4f
Copy link
Collaborator

Jocke4f commented Jan 26, 2024

I looked around for a short online summary of what needs to be done. One summary was this: https://www.sobyte.net/post/2022-07/es-module-error/

@Jocke4f
Copy link
Collaborator

Jocke4f commented Feb 14, 2024

Should we as a temporary solution revert #607?

This way we would use an axios version that can be used the old way. If we makes this revert we can created a new TA release so a number of fixes since 3.0.7 are included in an official release. hopefully even the docker package and once again installation via npm.

@josephdadams
Copy link
Owner

Works for me

@szkrstms
Copy link
Author

Not for me. Or later, in the next version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants