Skip to content

Clone recursively #1852

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

Merged
merged 1 commit into from
Feb 29, 2016
Merged

Clone recursively #1852

merged 1 commit into from
Feb 29, 2016

Conversation

robrix
Copy link
Contributor

@robrix robrix commented Feb 28, 2016

git packages using submodules may need to be cloned recursively.

  • Allows us to pass arguments to the clone command on a case-by-case basis.
  • Adds the --recursive option to git clone.
  • hg clone will already clone subrepos.

I have not added a configuration option to control this, because it doesn’t seem particularly useful in the context of repos which are Haskell packages.

Fixes #1764.

readInNull (parent dirTmp) commandName menv
[ "clone"
readInNull (parent dirTmp) commandName menv $
"clone" : cloneArgs ++ [
, T.unpack url
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo here; don't want that leading , (this caused the builds to fail)

@robrix
Copy link
Contributor Author

robrix commented Feb 28, 2016

Whoops—my bad!

@robrix
Copy link
Contributor Author

robrix commented Feb 28, 2016

Oof. My bad again. Excuse me while I correct this.

@robrix
Copy link
Contributor Author

robrix commented Feb 28, 2016

There we go, green builds.

@mgsloan
Copy link
Contributor

mgsloan commented Feb 29, 2016

Thanks! Can you please squash this into one commit?

Add the arguments to the list.

Note recursive clones in the changelog.

Remove leading comma.

Parenthesize.

Format across lines.
@robrix
Copy link
Contributor Author

robrix commented Feb 29, 2016

Consider it squashed.

@mgsloan
Copy link
Contributor

mgsloan commented Feb 29, 2016

LGTM, Thanks!!

mgsloan added a commit that referenced this pull request Feb 29, 2016
@mgsloan mgsloan merged commit a720236 into commercialhaskell:master Feb 29, 2016
@robrix robrix deleted the clone-recursively branch February 29, 2016 23:56
@robrix
Copy link
Contributor Author

robrix commented Mar 1, 2016

🙇 Thank you!

@acfoltzer
Copy link
Contributor

Thanks!

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

Successfully merging this pull request may close these issues.

4 participants