search

josephburnett / jd

JSON diff and patch
MIT License
826 stars 38 forks source link

the web UI wasn't include in this build: use `make release` to include it #77

Open arranf opened 2 months ago

arranf commented 2 months ago

The install instructions say you can install with brew install however this build does not seem to include the web ui.

Running jd -port 8080 returns the error message

the web UI wasn't include in this build: use make release to include it

It'd be good to highlight this and/or give more complete instructions for installing a build with access to the web ui

josephburnett commented 1 month ago

Looks like brew is build with go build: https://github.com/Homebrew/homebrew-core/blob/e68e8c162d12a4227b5ceb3104eebb3b1fc1ecbd/Formula/j/jd.rb#L21

Maybe we should change that to make release so it includes the UI!

@arranf do you want to make the upstream change? Or shall I?

josephburnett commented 1 month ago

Or maybe @chenrui333 would like to help, since they last updated the formula.