-
Notifications
You must be signed in to change notification settings - Fork 19
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 submodule-add
command
#127
Comments
@erikd i've also not been able to find docs for how i can depend on other repos... been trying to dig that up since you mentioned it |
The way we currently do it is with git:
and it should just all work. |
Spoke to @jystic about this at lunch. He's not (yet) convinced this is a good idea. The Pros:
The Cons:
|
It took me 10 minutes to figure out that git add is how I update what the
submodule ref. And that in contrast git submodule update resets that child
repo.
I am not an outlier in my git weakness, and I'm probably more competent
than it's reasonable to expect ;)
…On Thu, Mar 9, 2017 at 8:54 PM Erik de Castro Lopo ***@***.***> wrote:
Spoke to @jystic <https://github.com/jystic> about this at lunch. He's
not (yet) convinced this is a good idea.
The Pros:
- The submodule command will be listed in the mafia help output making
it easily discoverable.
- We can add both submodule add and submodule rm, the second of which
is a glaring omission in the git submodule command.
The Cons:
- Ties mafia to git (but no more so that is already the case).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#127 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAAQwhYEmdPMO0KMrQc0BUN-UbQCjvKdks5rkK1IgaJpZM4JFwpv>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When adding package
whatever
atlib/whatever
it would then checkwhatever
's dependencies and recursively add them as well.The text was updated successfully, but these errors were encountered: