search

bbodi / notecalc3

NoteCalc is a handy calculator trying to bring the advantages of Soulver to the web.
https://bbodi.github.io/notecalc3/
GNU Affero General Public License v3.0
1.16k stars 43 forks source link

Unable to clone & run locally #1

Closed alechp closed 3 years ago

alechp commented 3 years ago

Summary

  1. Need wasm-pack and serve binaries
  2. #![feature] may not be used on the stable release channel
  3. When running, it seems to launch the notecalc home page instead of the calculator itself

1. Resolving wasm-pack and serve binaries missing

Screenshots

Without wasm-pack
With wasm-pack, without serve

Update README

# 1. clone repo
git clone https://github.com/bbodi/notecalc3.git

# 2. install global binaries required for compile_and_run
curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh

# 3. compile and run
cd ../
./compile_and_run.bat

Update compile_and_run.bat

Use npx for serve to avoid global install

From... 

cd frontend-web
wasm-pack build --debug --target no-modules
cd ..
serve .
To... 

cd frontend-web
wasm-pack build --debug --target no-modules
cd ..
npx serve .
```

2. Stable release channel

Screenshots

Stable release channel error
Re-running rustup

Questions

This is obviously not a new issue https://github.com/rust-lang/rust/issues/37877

Despite re-running rustup, there's still an issue. I'm a rust novice and I'm confident this is an issue with my system, not notecalc.

Is there a rust RC (eg. like a nightly release) that I need to be using?

3. Showing website instead of calculator

Screenshots

Serve working (expected)
Notecalc home page showing (unexpected)
Calculator I expected to see when accessing localhost:5000

Expected

I would open localhost:5000 and see an error

Actual

Surprisingly, I was able to access localhost:5000 despite the compilation errors. However, the site is showing notecalc3's website instead of the calculator.

bbodi commented 3 years ago

Nice description, thanks :) You can access notecalc via localhost:5000/notecalc

You are right the README is not up to date, I will update it.

Thanks for your report!

alechp commented 3 years ago

I think useful changes would be:

  1. Update readme to show install
  2. Add npx serve to ./compile_and_run.bat
  3. Add a comment about rust version required to run this (to avoid the notecalc-lib errors)

Thanks for the quick response. That worked :)