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 2, 2017, 9:43 AM

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

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

Blocked until some issues are sorted upstream in TRDR

Nicola Mometto
August 3, 2017, 8:18 PM

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 4, 2017, 3:06 AM

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

David Nolen
August 4, 2017, 8:22 AM
Completed

Assignee

David Nolen

Reporter

Mike Fikes

Labels

Approval

Accepted

Patch

Code and Test

Fix versions

Affects versions

Priority

Major