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

Remove unnecessary submodules #300

Open
bmyerz opened this issue Oct 2, 2015 · 3 comments
Open

Remove unnecessary submodules #300

bmyerz opened this issue Oct 2, 2015 · 3 comments

Comments

@bmyerz
Copy link
Member

bmyerz commented Oct 2, 2015

@senderista and I discussed that

  • six and dateutil may be remnants of fixing the requests dependency
@bmyerz bmyerz added the refactor label Oct 2, 2015
@senderista senderista self-assigned this Oct 29, 2015
@senderista
Copy link
Contributor

We should remove requests as well, in fact all third-party submodules should be converted into proper install dependencies. I will have to revisit the requests dependency (I converted it to a submodule because I couldn't find a release version corresponding to the commit that fixed my SSL issue).

@jortiz16
Copy link
Contributor

jortiz16 commented Aug 3, 2016

Currently, there's an issue with six -- that repository no longer exists. Seems to be ok if we replace with https://github.com/gsutil-mirrors/six for now but who knows how long that will be around for.

Any temporary fix for this would be great. I would help, but I don't know how to update submodules and I'm not sure what functionality we need from these dependencies.

@senderista
Copy link
Contributor

d47ed0f temporarily fixed the six breakage by changing the remote to https://github.com/gsutil-mirrors/six.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants