iriscouch / build-couchdb

Batteries-included CouchDB build system
227 stars 36 forks source link

Replacing git:// scheme with https:// #64

Open zepag opened 12 years ago

zepag commented 12 years ago

Hello,

I know it's not an important requirement, though this allows people in a restricted network configuration (allowing only access to http/https for instance) to checkout recursively everything needed to build couch using your great wrapper.

Thanks in advance.

clayzermk1 commented 11 years ago

I agree https is the way to go.

As a work-around for anyone having trouble with the git protocol:

git clone https://github.com/iriscouch/build-couchdb
cd build-couchdb
vi .gitmodules
#change all "git:" to "https:", save and quit
git submodule init
git submodule update

This worked for me.

zepag commented 11 years ago

Yes this can definitely be solved with a checkout and a sed, though it would be better to work out of the box ;)

gmaggess commented 11 years ago

Tried your suggestion but still can't access https://git.sv.gnu.org/autoconf/info/refs. Any hints?

jhs commented 11 years ago

I am willing to change any protocols as long as they are officially supported by their hosts.

On the other hand, I sort of feel like choosing the correct network protocol is out of scope for this project. If you can't clone from an (any!) official Git repository, then you have your own problems.

However, I would not enforce this belief of mine until we had another feature I have always wanted, which is all-in-one downloadable .zip or .tar.gz files. I do not want to restrict people's ability to get CouchDB, I just want to reduce the scope of this project so it is manageable.

gmaggess commented 11 years ago

Hi Jason,

I found a workaround for that matter here: http://www.emilsit.net/blog/archives/how-to-use-the-git-protocol-through-a-http-connect-proxy/

It works like a charm.

Cheers,

zepag commented 11 years ago

Jason, In the precise case, I propose replacing protocols for github.com repositories with their https counterpart, which I think is fully supported. Am I missing something?