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

Trace conflicts with #'boot.util/trace #186

Closed
arichiardi opened this issue Jun 25, 2017 · 4 comments
Closed

Trace conflicts with #'boot.util/trace #186

arichiardi opened this issue Jun 25, 2017 · 4 comments

Comments

@arichiardi
Copy link
Collaborator

Hello folks!

With the advent of boot 2.7.2-SNAPSHOT a conflict arises:

java.lang.IllegalStateException: trace already refers to: #'boot.util/trace in namespace: boot.user
     clojure.lang.ExceptionInfo: trace already refers to: #'boot.util/trace in namespace: boot.user

This is because someone (myself) added a new method for logging in boot. I did not expect it would conflict with perun.

I have just tried the blog example and indeed it breaks.

A solution would be not to use :refer :all in the build.boot file.

@podviaznikov
Copy link
Member

This seems to be fixed now.

@mprokopov
Copy link

I still have this issue when try to follow README examples
this build.boot

(set-env!
 :source-paths #{"content"}
 :dependencies '[[perun "0.4.2-SNAPSHOT" :scope "test"]
                 [pandeiro/boot-http "0.8.3" :exclusions [org.clojure/clojure]] ])

(require '[io.perun :refer :all]
         '[pandeiro.boot-http :refer [serve]])

causes following error

java.lang.IllegalStateException: trace already refers to: #'boot.util/trace in namespace: boot.user
clojure.lang.ExceptionInfo: trace already refers to: #'boot.util/trace in namespace: boot.user
boot -u
Retrieving maven-metadata.xml from https://repo.clojars.org/ (3k)
#http://boot-clj.com
#Wed Nov 29 16:06:35 EET 2017
BOOT_CLOJURE_NAME=org.clojure/clojure
BOOT_CLOJURE_VERSION=1.9.0-alpha17
BOOT_VERSION=2.7.2

Is there another fix except not to use :all in :refer ?

@arichiardi
Copy link
Collaborator Author

The fix was basically to use an :as instead of referring all the symbols. You can also :rename but it has to be in your file unless the name changes in perun

StatRock added a commit to StatRock/eips that referenced this issue Feb 22, 2018
@devurandom
Copy link

devurandom commented Sep 2, 2018

Could someone please reopen this report and also fix the readme? I also ran into this.

Note that after changing :refer :all to :as perun, the $ boot markdown command becomes $ boot perun/markdown.

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

4 participants