QuTech-Delft / QuTech-Delft/libqasm

[Infra] Update to `emsdk/3.1.51` or newer

Open
#206 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
23
Forks
18
PR merge metrics
No merged PRs in 30d

Description

There is an issue when running the JS code generated from emscripten builds in a Firefox browser that should be fixed by emsdk/3.1.51.

That code modifies Error.stack which is read-only as per spec, but only Firefox follows this spec.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the repository configuration that selects emsdk and check how the current version is used in the build. Update it to emsdk/3.1.51 or newer, then run the relevant build and verify the generated JavaScript in Firefox; done means the Error.stack issue no longer occurs.

Written by the indexing model from the issue text.

Assessment

Domain
build-system
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.