Issue using interop record constructors in macros namespaces

Description

Repro:

Expected:

Master produces:

Not a regression as far as I can tell, and also affects regular JVM ClojureScript.

Environment

None

Activity

Show:
António Monteiro
August 3, 2017, 5:06 PM

Attached CLJS-2261-2.patch which also bumps tools.reader to 1.0.5

Nicola Mometto
August 3, 2017, 10:18 AM

I just release 1.0.5 which now requires cljs to resolve `Foo.` symbols to `their.ns/Foo.` in `resolve-symbol`. I realize this might be a bit annoying but it's the only way to support both clojure's behaviour of not including namespace segments and cljs' of requiring it

António Monteiro
August 2, 2017, 12:04 AM

Blocked until some issues are sorted upstream in TRDR

António Monteiro
August 1, 2017, 11:43 PM

Attached patch with Mike's tests + tools.reader bump (which is where the fix is).

Completed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

David Nolen

Reporter

Mike Fikes

Labels

Approval

Accepted

Patch

Code and Test