Macros do not appear to work when crossover namespace is granular about lein-cljsbuild HOT 7 CLOSED

In your CLJS code, what does the (ns) form look like where the basil.test.test-macros namespace is :required or :used? The goog.require error makes me think that maybe you used :require or :use instead of :require-macros or :use-macros.

from lein-cljsbuild.

The CLJS code that uses basil.test.test-macros is a CLJ file (softlinked as a CLJS) and has the following:

(ns basil.test.core
  (:require [basil.types :as types]
            [basil.core  :as core]
            [basil.util  :as util])
  (:use;*CLJSBUILD-REMOVE*;-macros
    [basil.test.test-macros :only [deftest is is-thrown testing slot-compiler
                                   ;*CLJSBUILD-REMOVE*;RuntimeException
                                   ]]))

from lein-cljsbuild.

Hmm, when you say softlinked, do you mean literally a symbolic link to the file?

from lein-cljsbuild.

Sorry, it was another file that is symlinked – the test_macros.clj file was never symlinked. I have now renamed this file so that they are different namespaces now – macros for Clojure and macros-cljs for CLJS, controlled as follows (in file basil/test/core.clj that is also symlinked as basil/test/core.cljs in another directory).

(:use;*CLJSBUILD-REMOVE*;-macros
  [basil.test.macros;*CLJSBUILD-REMOVE*;-cljs
   :only [deftest is is-thrown testing slot-compiler
          ;*CLJSBUILD-REMOVE*;RuntimeException
          ]])

However, the error message still persists:
goog.require could not find: basil.test.macros

from lein-cljsbuild.

Hmm, without seeing the source for the project, it's hard for me to tell exactly what's going wrong. I can mention a few guidelines, though, that I think might help track down the problem:

1. The crossover setup should not include any symbolic links at all.
2. None of the sources for the crossover namespaces should exist in the :cljsbuild/:source-path directory in a clean checkout of your project.
3. The *.clj file providing the macros should be referred to by the same name in both clj and cljs code.

Other than that, you might try changing the :crossovers config in the advanced example to use finer granularity. If you can reproduce the problem there, I'll be better able to help track down the cause.

from lein-cljsbuild.

I followed your guidelines and the issue is resolved. Thanks!

I also realized that I need to build one JS file for distribution, and one for testing. The testing JS file needs to include extra .cljs files, hence it may be useful to have :src-cljs accept a vector of directory locations instead of just one location. Please let me know if there is already a solution for that, or else I will file an issue requesting that feature.

from lein-cljsbuild.

I'm glad you have things working!

I don't think there's a good way to do that right now. The 0.1.0 release that I'm working on, though, has built-in support for running tests, and I'm working on adding an example of this to the advanced project right now. I think this new feature should allow you do do what you want (although perhaps in a somewhat different manner).

I hope to get 0.1.0 released in the next 1-3 days.

from lein-cljsbuild.