search

martinklepsch / boot-garden

Boot task to compile Garden stylesheets.
Eclipse Public License 1.0
37 stars 6 forks source link

updates to garden 1.3.0-SNAPSHOT, workifies garden pod. #14

Closed livtanong closed 9 years ago

livtanong commented 9 years ago

Based on #11 Entirely unsure about the change I made to...

(let [c-pod (:refresh garden-pods)] ... )

The above shows the change I made. Before, it was (garden-pods :refresh). Afaik, the two are equivalent in clojure, but for some reason I get a Java.io error if I keep it the way it is in current master of boot garden. Hoping you could investigate, @martinklepsch.

martinklepsch commented 9 years ago

Using keywords as argument and as function arguments is only supported with maps afaik. The correct way to call the pod pool would thus be (garden-pods :refresh). It's puzzling that this throws an exception. Have you reproduced that in multiple garden projects? (maybe try the example/ for a minimal setup)

Will give it a spin now.

livtanong commented 9 years ago

Silly me! I didn't realize that garden-pods is a function. That makes more sense. I suppose, then, that it's important to note that simply removing :refresh makes it work. I'm just about to test it on another project now. Will report my findings.

martinklepsch commented 9 years ago

Works for me in the example project when calling garden-pods with :refresh :)

livtanong commented 9 years ago

Jesus. I now know what the problem was! I had inserted the garden task before the hoplon task. This is embarrassing. I'll create a separate pull request without the "fix".