Skip to content

Fix: Remove NDK build-id#59

Merged
naltatis merged 3 commits intoevcc-io:mainfrom
Maschga:remove-ndk-build-id
Apr 9, 2025
Merged

Fix: Remove NDK build-id#59
naltatis merged 3 commits intoevcc-io:mainfrom
Maschga:remove-ndk-build-id

Conversation

@Maschga
Copy link
Copy Markdown
Collaborator

@Maschga Maschga commented Apr 9, 2025

Ref: #57, #58

TODO:

  • npx expo prebuild runs successfully

\cc @naltatis

@naltatis
Copy link
Copy Markdown
Member

naltatis commented Apr 9, 2025

Does not work for me.
Clean install (no ios, android folder)

npm run android

> evcc@1.0.2 android
> expo run:android

SyntaxError: Unexpected identifier 'ConfigPlugin'
SyntaxError: Unexpected identifier 'ConfigPlugin'
    at compileSourceTextModule (node:internal/modules/esm/utils:338:16)
    at ModuleLoader.importSyncForRequire (node:internal/modules/esm/loader:353:18)
    at loadESMFromCJS (node:internal/modules/cjs/loader:1385:24)
    at Module._compile (node:internal/modules/cjs/loader:1536:5)
    at Object..js (node:internal/modules/cjs/loader:1706:10)
    at Module.load (node:internal/modules/cjs/loader:1289:32)
    at Function._load (node:internal/modules/cjs/loader:1108:12)
    at TracingChannel.traceSync (node:diagnostics_channel:322:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:220:24)
    at Module.require (node:internal/modules/cjs/loader:1311:12)

@naltatis
Copy link
Copy Markdown
Member

naltatis commented Apr 9, 2025

Scheint ein Problem mit TS gewesen zu sein. Ich habs mal in js konvertiert. Dann läufts auch bei mir. Komisch dass das bei dir ging,

@naltatis naltatis merged commit 3a2acd8 into evcc-io:main Apr 9, 2025
2 checks passed
@Maschga
Copy link
Copy Markdown
Collaborator Author

Maschga commented Apr 9, 2025

Das ist schon lustig.
Hatte den gleichen Fehler und habe dann den Deep Import eingeführt.
Dann ging es bei mir auf Windows und Linux. Hast du es auf einem Apple Gerät ausgeführt?

JS ist auch super. 👍

@Maschga Maschga deleted the remove-ndk-build-id branch April 9, 2025 21:05
@naltatis
Copy link
Copy Markdown
Member

Ja, macOS.

@Maschga Maschga mentioned this pull request Apr 11, 2025
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