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: build on windows in CI #2184

Open
SgtPooki opened this issue Dec 2, 2023 · 3 comments · May be fixed by #2208
Open

fix: build on windows in CI #2184

SgtPooki opened this issue Dec 2, 2023 · 3 comments · May be fixed by #2208
Assignees
Labels
exp/expert Having worked on the specific codebase is important need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s)

Comments

@SgtPooki
Copy link
Member

SgtPooki commented Dec 2, 2023

see https://github.com/ipfs/ipfs-webui/actions/runs/7067401834/job/19240720367

Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'd:'

@SgtPooki SgtPooki added the need/triage Needs initial labeling and prioritization label Dec 2, 2023
@whizzzkid
Copy link
Contributor

Looks like a bunch of other things are failing as well: https://github.com/ipfs/ipfs-webui/actions/runs/7076694600

@whizzzkid whizzzkid added exp/expert Having worked on the specific codebase is important need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) and removed need/triage Needs initial labeling and prioritization labels Dec 7, 2023
@sumingcheng
Copy link

I also encountered the same issue when launching the webui project on Windows. How should I solve it

> ipfs-webui@4.2.0 start:win32
> @powershell -Command $env:REACT_APP_GIT_REV=(git rev-parse --short HEAD); react-app-rewired-esm start

node:internal/errors:496
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'd:'
    at new NodeError (node:internal/errors:405:5)
    at throwIfUnsupportedURLScheme (node:internal/modules/esm/load:149:11)
    at defaultLoad (node:internal/modules/esm/load:99:3)
    at ModuleLoader.load (node:internal/modules/esm/loader:416:13)
    at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:287:22)
    at new ModuleJob (node:internal/modules/esm/module_job:63:26)
    at #createModuleJob (node:internal/modules/esm/loader:311:17)
    at ModuleLoader.getJobFromResolveResult (node:internal/modules/esm/loader:264:34)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:250:17)
    at async ModuleLoader.import (node:internal/modules/esm/loader:335:23) {
  code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
}

Node.js v18.19.0

@SgtPooki SgtPooki self-assigned this Feb 7, 2024
@SgtPooki
Copy link
Member Author

SgtPooki commented Feb 7, 2024

@sumingcheng not sure what the fix is just yet, but i'm going to start looking into this now

@SgtPooki SgtPooki linked a pull request Feb 7, 2024 that will close this issue
@SgtPooki SgtPooki linked a pull request Feb 7, 2024 that will close this issue
SgtPooki added a commit to ipfs/ipld-explorer-components that referenced this issue Feb 13, 2024
SgtPooki added a commit to ipfs/ipld-explorer-components that referenced this issue Feb 13, 2024
* feat: migrate to @helia/http

* chore: deps and storybook fixes

* deps: more dep cleanup

* deps: eslint-config-ipfs

* deps: babel

* deps: filesize update

* deps: ipfs-unixfs update

* deps: rollup update

* deps: cytoscape update

* deps: i18next update

* deps: @types/* update

* deps: @ipld deps update

* deps: @multiformats/sha3 update

* test: test consumers

* test: ensure test is ran in CI

* test: remove consumer test for now

webui windows is failing, see ipfs/ipfs-webui#2184
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exp/expert Having worked on the specific codebase is important need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s)
Projects
Status: 🏃‍♀️ In Progress
Development

Successfully merging a pull request may close this issue.

3 participants