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

check-clojure-tools does not work #159

Closed
introom opened this issue May 12, 2022 · 6 comments
Closed

check-clojure-tools does not work #159

introom opened this issue May 12, 2022 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@introom
Copy link

introom commented May 12, 2022

check-clojure-tools does not work tools where the installation path is ~/.config/clojure/tools/. ~/.config is $XDG_CONFIG_HOME

The full command that I found this issue is:
clojure -Tantq outdated :check-clojure-tools true :upgrade true

liquidz added a commit that referenced this issue May 12, 2022
cf. #159
Changed to use `clojure.tools.deps.alpha/user-deps-path`
so it should be the same path with tools.alpha
@liquidz
Copy link
Owner

liquidz commented May 12, 2022

@introom Thanks for your reporting!
I fixed the problem in bugfix/tool-dir branch.
Could you try?

# install bugfix/tool-dir branch
clojure -Ttools install com.github.liquidz/antq '{:git/url "https://github.com/liquidz/antq" :git/sha "c23871f018c20eff7655f78f2160abdb7cbea510"}' :as antq

# rerun antq as a tool
clojure -Tantq outdated :check-clojure-tools true :upgrade true

@liquidz liquidz added the bug Something isn't working label May 12, 2022
@liquidz liquidz self-assigned this May 12, 2022
@introom
Copy link
Author

introom commented May 13, 2022

I've this in my deps.edn config.

           :antq {:extra-deps {com.github.liquidz/antq {:git/url "https://github.com/liquidz/antq"
                                                        :sha "c23871f018c20eff7655f78f2160abdb7cbea510"}}
                  :main-opts ["-m" "antq.core"]}

I run the command under some directory that contains a deps.edn file (otherwise antq will complain no project found).

 clojure -M:antq :check-clojure-tools true :upgrade true

The output is:

All dependencies are up-to-date.

This file ~/.config/clojure/tools/new.edn is unchanged .

image

@liquidz
Copy link
Owner

liquidz commented May 13, 2022

@introom You should run clojure -M:antq --check-clojure-tools --upgrade when you use -M

@introom
Copy link
Author

introom commented May 13, 2022 via email

@liquidz
Copy link
Owner

liquidz commented May 13, 2022

Thanks for your confirmation! I'll cut a new release this weekend :)

@liquidz
Copy link
Owner

liquidz commented May 13, 2022

@introom Just released v1.6.2 #162
Thank you so much!

@liquidz liquidz closed this as completed May 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants