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.

Sign up for GitHub

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

Jump to bottom

wasm-opt exits with signal: 11 #781

Closed
dsherret opened this issue Feb 1, 2020 · 3 comments · Fixed by #687
Closed

wasm-opt exits with signal: 11 #781

dsherret opened this issue Feb 1, 2020 · 3 comments · Fixed by #687
Assignees
@ashleygwilliams
Labels
bug Something isn't working

Comments

@dsherret
Copy link

dsherret commented Feb 1, 2020

🐛 Bug description

The latest 0.9.0 release fails to build with the following error in my scenario:

Error: failed to execute `wasm-opt`: exited with signal: 11
  full command: "/home/travis/.cache/.wasm-pack/wasm-opt-37dc6b0d9960f622/wasm-opt" "wasm/../src/wasm/dprint_rust_printer_bg.wasm" "-o" "wasm/../src/wasm/dprint_rust_printer_bg.wasm-opt.wasm" "-O"
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.

This only happens on my CI. Builds fine locally on my windows machine so this might just be an issue with the CI? Not sure as it's using linux.

Related issue: #696

👟 Steps to reproduce

Sorry, I don't have time to submit an MCVE at the moment, but the following will probably reproduce the problem:

git clone https://github.com/dsherret/dprint.git
cd dprint
git reset --hard 11e617fabef70260b46060e5823ff247323e3ba8
yarn install
yarn bootstrap
yarn build

Fails in packages/core/wasm.

https://travis-ci.org/dsherret/dprint/builds/644649601?utm_medium=notification&utm_source=github_status

🌍 Your environment

Ubuntu 16.04.6 LTS
$ rustc --version
rustc 1.41.0 (5e1a79984 2020-01-27)
$ rustup --version
rustup 1.21.1 (7832b2ebe 2019-12-20)
$ cargo --version
cargo 1.41.0 (626f0f40e 2019-12-03)

Workarounds

Either:

  1. Install wasm-pack via npm and specify 0.8.1 as pointed out here: Ability to provide version to init.sh #779 (comment)
  2. Update cargo.toml to disable wasm-opt (I didn't try this, but I'm assuming it might work): 
    [package.metadata.wasm-pack.profile.release]
wasm-opt = false

...so this is not a blocker for me. Anyway, thanks for the great project!
@dsherret dsherret mentioned this issue Feb 1, 2020
Closed
@ashleygwilliams
Copy link
Member

really sorry you are running into this! we knew that wasm-opt had some issues on Linux.. many folks seem to be running into issues specifically with CI- i'll be looking into this today to see if we can't improve it! (thanks for the really awesome+detailed bug report by the way<3)

@ashleygwilliams ashleygwilliams self-assigned this Feb 3, 2020
@ashleygwilliams ashleygwilliams added the bug Something isn't working label Feb 3, 2020
@jshrake jshrake mentioned this issue Feb 3, 2020
Closed
@coolreader18
Copy link

coolreader18 commented Feb 6, 2020 •
edited
Loading

I have the same problem, and it takes at least 5 minutes for wasm-opt to fail. See here inside the "run test" step; in our setup, the keepalive line gets printed every 5 minutes.

@ashleygwilliams ashleygwilliams mentioned this issue Feb 7, 2020
Merged
@ashleygwilliams ashleygwilliams linked a pull request Feb 7, 2020 that will close this issue
refactor(tool): wasm-opt is a tool variant #687
Merged
@expenses expenses mentioned this issue Feb 17, 2020
Merged
@ijjk ijjk mentioned this issue May 24, 2022
Merged
Marcel-G added a commit to Marcel-G/sobaka-sample that referenced this issue Jul 18, 2022
@Marcel-G
Switch to macos for wasm-opt
e571de3 
Currently wasm-opt has issues in ubuntu

rustwasm/wasm-pack#781
Marcel-G added a commit to Marcel-G/sobaka-sample that referenced this issue Sep 5, 2022
@Marcel-G
fundsp Migration
a7dc036 
    - Move DSP to fundsp (& recreate all DSP modules)
    - Rework RPC messaging
    - Parameter stabilisation fixes
    - Add Clock and Sequencer
    - Add VCA module
    - Add Reverb module
    - Add Noise module
    - Add Envelope module
    - Add Delay module
    - Add LFO module
    - Add StepSequencer module
    - Add Midi module
    - Add Filter module
    - Add Sample&Hold module
    - Add Scope module
    - Add String Oscillator (Experimental)
    - Update clock divider
    - Update LED styles
    - Use 1v/octave for frequencies
    - Implement up/down arrow keys on Toolbox
    - Make Quantiser keyboard more stylish
    - Move to Svelte-Kit
    - Offline storage for media assets
    - Basic routing added
    - Templating system added (replaces examples)
    - Add MIDI Synth (2 Oscillator) template
    - Kick, Snare, Hi-hat templates added
    - Sampler added with onset detection
    - Add editable workspace/template title
    - Index page with list of workspaces & templates
    - Error pages added (in progress)
    - Add ADSR envelope
    - Switch to macos in ci resolving wasm-opt issue (rustwasm/wasm-pack#781)
@frewsxcv
Copy link

If anyone is wondering how to resolve the sigsegv going forward: #823 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashleygwilliams ashleygwilliams

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor(tool): wasm-opt is a tool variant rustwasm/wasm-pack
4 participants
@frewsxcv @ashleygwilliams @dsherret @coolreader18