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

Not working on clojurescript 1.10.439 #189

Closed
galdolber opened this issue Nov 11, 2018 · 8 comments
Closed

Not working on clojurescript 1.10.439 #189

galdolber opened this issue Nov 11, 2018 · 8 comments

Comments

@galdolber
Copy link

One error I get is this:

screen shot 2018-11-11 at 2 56 21 pm

But even when I fixed that, the parser function is failing.

@mtruyens
Copy link

I wanted to submit the same error. So far, the parsing does continue to work, despite the compile warning.

@Engelberg
Copy link
Owner

@galdolber Can you describe the way it is failing? It appears that Clojurescript made a change to the public visibility of some functions, and added a new set of compiler warnings, but the warning should not break the function. I would expect it to still work, as @mtruyens described.

@galdolber
Copy link
Author

galdolber commented Nov 12, 2018

(instaparse.core/parser "S = AB* AB = A B A = 'a'+ B = 'b'+")

I get:
screen shot 2018-11-12 at 1 17 30 am

Rolling back to clojurescript 1.10.339 works fine.

@galdolber
Copy link
Author

This is the commit that breaks the code:
clojure/clojurescript@9a8196e

@galdolber
Copy link
Author

I was able to fix the issues here: #190

@Engelberg
Copy link
Owner

I fixed the warning that clojurescript was giving about the use of the private var read-string* from tools.reader. But I am unable to duplicate the bug you are seeing. The test suite is passing under the latest version of clojurescript, and I entered in your sample bug by hand into a clojurescript REPL and it worked fine.

I bumped up the version of tools.reader listed in the dependencies, so perhaps that helped. Please check again against instaparse 1.4.10 and let me know whether you still see a problem. Thanks for your assistance.

@jeroenvandijk
Copy link

I can confirm that upgrading from 1.4.9 to 1.4.10 got rid of the warning:

WARNING: var: cljs.tools.reader/read-string* is not public

Which blocked figwheel to compile my code.

Thanks! Btw the README still says 1.4.9 (hence my trial)

@Engelberg
Copy link
Owner

Just updated README. Thanks for catching that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants