Jsonnet: Demo is broken on the website

Created on 23 Jul 2018 · 11Comments · Source: google/jsonnet

I wanted to try out the interactive demo on the website but it appears the demo is broken.

Steps to Reproduce

Visit https://jsonnet.org/
Try editing one of the input boxes

I would expect that the output box would update but nothing changes.

Error Messages

I'm getting the following error

Failed to load resource: the server responded with a status of 404 (Not Found)

It appears that the link to https://jsonnet.org/js/libjsonnet.js is dead. When I try to visit that link I get a 404.

Source

lukesmurray

Most helpful comment

Well, it still doesn't work for me (fails in a different way, tho) STATIC ERROR: std.jsonnet:1249:1: did not expect: (NUMBER, "3")

sbarzowski on 24 Jul 2018

👍2

All 11 comments

Looks like something went wrong during the last deployment. I have the same issue. @sparkprime

sbarzowski on 23 Jul 2018

I fixed it, and can now curl the URL. But every browser I try still
reports 404. I suspect it's a cacheing issue and will resolve itself.

On Mon, Jul 23, 2018 at 12:08 PM, Stanisław Barzowski <
[email protected]> wrote:

Looks like something went wrong during the last deployment. I have the
same issue. @sparkprime https://github.com/sparkprime

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/jsonnet/issues/545#issuecomment-407112656, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKQ3gokq0H_ZKlx9j1m71zUm_QZdV56ks5uJfUSgaJpZM4VbN7h
.

sparkprime on 23 Jul 2018

The 404 appears to be fixed but I'm getting two new errors.

Error 1

Uncaught TypeError: Module.cwrap is not a function
at demo.js:2

The line where it fails is

/** Bind the functions as variables in Javascript. */
let jsonnet_make = Module.cwrap(
    'jsonnet_make', 'number', [])

Error 2

Uncaught ReferenceError: counter is not defined
at aux_demo (demo.js:207)
at demo (demo.js:379)
at (index):202

It appears that counter is not defined in the function aux_demo and the code fails on the line

let id = 'editor-' + counter++;

lukesmurray on 23 Jul 2018

Looks like there has been a breaking change in emscripten. I'll have to investigate this properly.

sparkprime on 23 Jul 2018

Working for me again now.

sparkprime on 23 Jul 2018

Well, it still doesn't work for me (fails in a different way, tho) STATIC ERROR: std.jsonnet:1249:1: did not expect: (NUMBER, "3")

sbarzowski on 24 Jul 2018

👍2

I'm getting the same result.

lukesmurray on 24 Jul 2018

I guess I didn't actually try it, I just checked the javascript console for errors!

This must be a bug in emscripten as the native executable is fine. I'll try different versions of emscripten.

This is also making me think we need to run the tests on the Javascript build.

sparkprime on 24 Jul 2018

Ok this is a good one actually. When I did the release, I checked the cmake build. cmake has its own little executable for building core/std.jsonnet.h (for embedding stdlib in the executable vvia an array of numbers). This writes to the same header file as the Makefile rule that does the same thing, but the Makefile rule terminates the array with 0. Having built the unterminated one with cmake, the timestamps were fresh so running make again did not rebuild it, so then when I build the js with emscripten, it triggers this behavior. I guess the regular native C++ build happens to work OK despite the lack of '0' because of padding or whatever.

sparkprime on 24 Jul 2018

👍1

Hi, I am still getting this issue while running on macOS Catalina with jsonnet=0.11.2