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

Add --deps/root option to install command #70

Open
jeroenvandijk opened this issue Jun 22, 2023 · 1 comment
Open

Add --deps/root option to install command #70

jeroenvandijk opened this issue Jun 22, 2023 · 1 comment

Comments

@jeroenvandijk
Copy link
Contributor

I have a project that has a nested project structure with a bbin entry.

Locally I can install this project with

bbin install . --local/root projects/poly-bb

or directly from the dir with

bbin install . 

From Github I have not been able to install it without cloning the repo locally.

Ideally one would be able to install this bbin binary with

bbin install io.github.jeroenvandijk/polylith --git/sha "6527bf4b4d6165c8ba7d8f678866e662d34b1c19" --deps/root "projects/poly-bb" --as poly-bb-dev1

I've tried adding a bb.edn file in the root of the project without success (it installs but then gets an error):

bbin install io.github.jeroenvandijk/polylith --git/sha "8c1632cf58d257ad574781192872133c0706941a" --as poly-bb-dev2
Error here
----- Error --------------------------------------------------------------------
Type:     java.lang.Exception
Message:  Could not find namespace: polylith-bb.core.
Location: <expr>:1:10

----- Context ------------------------------------------------------------------
1: (ns user (:require [polylith-bb.core])) (apply polylith-bb.core/-main *command-line-args*)
            ^--- Could not find namespace: polylith-bb.core.

----- Stack trace --------------------------------------------------------------
user - <expr>:1:10

Another option would be to change the deps.edn file in the root. This would however conflicts with the project structure of Polylith itself.

@rads
Copy link
Collaborator

rads commented Jun 25, 2023

I'm definitely interested in adding this. I think we should also make this configurable in the :bbin/bin config so you can use this to organize your code without the user noticing anything externally.

@rads rads changed the title Add --deps/root option to install command Add --deps/root option to install command Jun 25, 2023
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