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

CLI hangs #2

Closed
smani opened this issue Jan 26, 2016 · 6 comments
Closed

CLI hangs #2

smani opened this issue Jan 26, 2016 · 6 comments
Labels
Milestone

Comments

@smani
Copy link

smani commented Jan 26, 2016

I followed the steps below:

  1. Install PhantomJS version 2.0 from http://phantomjs.org/download.html
  2. Install version 4.2.4 from https://nodejs.org/en/download/ to obtain the npm.
  3. Run npm i wavedrom-cli --save-dev
  4. Run phantomjs node_modules\.bin\wavedrom -i path\to\sample.json -p sample.png -s sample.svg

The phantomjs command never terminates (I waited for more than 30mins).

I am using Windows 7 64 bit operating system.

@drom
Copy link
Member

drom commented Jan 27, 2016

@smani could you try with PhantomJS v2.1.1 ?

@drom drom added the bug label Jan 27, 2016
@drom drom added this to the v1 milestone Jan 29, 2016
@drom
Copy link
Member

drom commented Jan 29, 2016

Looks like symbolic links on Windows don't work as expected. Please try:
phantomjs node_modules\wavedrom-cli\bin\wavedrom-cli.js -i path\to\sample.json -p sample.png -s sample.svg

@smani
Copy link
Author

smani commented Jan 29, 2016

This command worked but the output PNG/SVG is no way related to the input. It seems to have some sample waveform (clk, Data and Request lines -- the Data line has in different colors).

BTW, I tried phantomjs node_modules\wavedrom-cli\bin\wavedrom-cli.js -p sample.png and it produces the same output as phantomjs node_modules\wavedrom-cli\bin\wavedrom-cli.js -i path\to\sample.json -p sample.png. It looks as if the sample.json isn't used by the tool.

@drom
Copy link
Member

drom commented Jan 29, 2016

@smani Your are absolutely right! My bad. Please try v0.3.0

@smani
Copy link
Author

smani commented Jan 29, 2016

Wow! It works now. Thank you.

@drom
Copy link
Member

drom commented Jan 29, 2016

great. closing.

@drom drom closed this as completed Jan 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants