Skip to content

Conversation

@robrix
Copy link

@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.

[ "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
Author

robrix commented Feb 28, 2016

Whoops—my bad!

@robrix
Copy link
Author

robrix commented Feb 28, 2016

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

@robrix
Copy link
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
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
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