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

bb dev fails with unable to clone git@github.com:nextjournal/clerk.git error #40

Closed
ikappaki opened this issue Dec 17, 2021 · 1 comment

Comments

@ikappaki
Copy link
Contributor

ikappaki commented Dec 17, 2021

$ bb dev
yarn install v1.22.10
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.38s.
Cloning: git@github.com:nextjournal/clerk.git
Error building classpath. Unable to clone C:\Users\chaos\.gitlibs\_repos\ssh\github.com\nextjournal\clerk
FATAL ERROR: No supported authentication methods available (server sent: publickey)
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

To reproduce.

  1. (Make sure you deprive yourself of git access to github.com:nextjournal/clerk.git if you happen to be a maintainer)
  2. From a shell, run bb dev

You get the above error.

This is because the deps.edn :sci alias is being called, which has a io.github.nextjournal/viewers dependency on the 1b1374db20b56d14a0f32835c1ce5d9254c23527 git tree

:aliases {:sci {:extra-deps  {applied-science/js-interop  {:mvn/version "0.3.0"}
                               borkdude/sci {:mvn/version "0.2.6"}
                               reagent/reagent {:mvn/version "1.1.0"}
                               io.github.nextjournal/viewers {:git/sha "1b1374db20b56d14a0f32835c1ce5d9254c23527"}
                               metosin/reitit-frontend     {:mvn/version "0.5.15"}}}

whose modules/devdocs/deps.edn is setup with a private git repo https://github.com/nextjournal/viewers/blob/914824bce0240ee57261cfc4011ef7d2aba058a5/modules/devdocs/deps.edn#L9:

{:path ["src"]
 :deps {re-frame/re-frame {:git/url "https://github.com/nextjournal/freerange"
                           :git/sha "8cf68c30722a4c6f8f948a134c900d7a656ecad4"}
        io.github.nextjournal/devcards {:local/root "../devcards"}
        io.github.nextjournal/viewer {:local/root "../viewer"}
        io.github.nextjournal/cljs-extensions {:local/root "../cljs-extensions"}
        io.github.nextjournal/markdown {:local/root "../markdown"}

        com.nextjournal/clerk {:git/url "git@github.com:nextjournal/clerk.git"
                               :git/sha "d2cacf6d3d27768f34b9092c100be9083c6dd1fc"}

        ;; we use shadow-resource directly
        thheller/shadow-cljs {:mvn/version "2.15.12"}
        metosin/reitit-frontend   {:mvn/version "0.5.15"}
        com.lambdaisland/deja-fu {:mvn/version "0.3.33"}
        com.lambdaisland/shellutils {:mvn/version "0.0.10"}
        org.flatland/ordered {:mvn/version "1.15.10"}}}

This has been upgraded since with nextjournal/viewers#31, so any viewers update in clerk's deps.edn from that commit onwards will pick up the updated coords from modules/devdocs/deps.edn:

        io.github.nextjournal/clerk {:git/sha "ebc8f998c761f1b0e2c9fc4750ae87d8f1766faa"}

(i.e. setting the io.github.nextjournal to the latest viewerssha indeps.edn` will fix it:

:sci {:extra-deps  {;;...
                           io.github.nextjournal/viewers {:git/sha "4590d1cf992781924b25f57a3fd54f41abbb9f1e"}
                           ;;...}}

)

I haven't filed a PR because I think is up to the maintainers to choose which viewers version to update clerk to.

Thanks,

@mk
Copy link
Member

mk commented Dec 20, 2021

Thanks, I've fixed this for now by bumping the viewers. Long-term we want to drop the devdocs module completely and replace it with Clerk's static build functionality.

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

2 participants