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

if in cljs repl, will have to 'lein cljsbuild clean' before 'lein cljx once' #38

Open
ccfontes opened this issue May 8, 2014 · 3 comments
Milestone

Comments

@ccfontes
Copy link

ccfontes commented May 8, 2014

When running lein repl followed by Austin, will have to lein cljsbuild clean before lein cljx once to be able to have new code changes available in the cljs repl.

Is this an expected behavior?

@cemerick
Copy link
Collaborator

cemerick commented May 9, 2014

If a .cljs resource is available for a given namespace, then it will be given preference over a .cljx resource. So, yes, this is expected behaviour. Maybe it shouldn't be (I can imagine the opposite arrangement, but I was attempting to be conservative?), but that's how it is now. Perhaps your apparently having the opposite expectation is an indication that changing things might be reasonable…

@ccfontes
Copy link
Author

Forget about the specific order of commands I said before. I cannot reproduce the issue like that anymore.
Regarding an exception I have related to meta that is present in the ns declaration (but removed from the .cljs resource), what I notice is, regardless if I do lein cljx once or not, it will always yield that exception, so it seems it will always use the .cljx resource somehow. When I remove the meta from the cljx resource, it works, of course.

The project with this issue
The stack trace

@cemerick cemerick added this to the 0.5.0 milestone May 27, 2014
@cemerick
Copy link
Collaborator

cemerick commented Dec 8, 2014

Trimming the scope of 0.5.0 so as to cut a release addressing pressing issues. Will circle back later.

@cemerick cemerick modified the milestones: 0.6.0, 0.5.0 Dec 8, 2014
@cemerick cemerick modified the milestones: NEXT, 0.6.0 Feb 13, 2015
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

2 participants