search

pangloss / clang

Don't use it!
228 stars 18 forks source link

Update to work with Clojure 1.6 #17

Open pharcosyle opened 10 years ago

pharcosyle commented 10 years ago

Works great in 1.5.1, but I get this error with 1.6:

No protocol method ITransientAssociative.-assoc! defined for type object: [object Object]

ccfontes commented 10 years ago

@pharcosyle I cannot reproduce your success with Clojure 1.5.1. I still have this error: screen shot 2014-04-24 at 21 36 38

Do you also have warning error with lein cljsbuild auto?:

WARNING: No such namespace: goog.object at line 30 file:/Users/me/.m2/repository/xhh/clang/0.1.0-SNAPSHOT/clang-0.1.0-SNAPSHOT.jar!/clang/util.cljs

tried lein-cljsbuild 1.0.1 as well as 1.0.3, but to no avail.

My cljs:

(ns app.clang
  (:require-macros [clang.angular :refer [def.controller defn.scope]])
  (:require [clang.util :refer [module]]))

(def m (module "app" []))

(def.controller m login [$scope]
  (assoc! $scope :route [{:bla "foo"}]))
pharcosyle commented 10 years ago

I'm pretty sure I got that warning with 1.6. Are you totally sure your project is using 1.5.1 instead of latest?

ccfontes commented 10 years ago

I'm sure, but maybe it's possible I got the same warning for a different reason.

pharcosyle commented 10 years ago

Okay, I checked again: I get that warning in both 1.6 and 1.5.1 but the error ONLY in 1.5.1. I also had to do a lein cljsbuild clean between changing the version in my project.clj.

If this still doesn't work for you I can post my source

On Thu, Apr 24, 2014 at 8:13 PM, Carlos Cunha notifications@github.comwrote:

I'm sure, but maybe it's possible I got the same warning for a different reason.

— Reply to this email directly or view it on GitHubhttps://github.com/pangloss/clang/issues/17#issuecomment-41347605 .

ccfontes commented 10 years ago

@pharcosyle, unfortunately I'm not using Clang anymore in this project.