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

Change libgit2 to a git-external #11435

Merged
merged 1 commit into from
May 27, 2015
Merged

Change libgit2 to a git-external #11435

merged 1 commit into from
May 27, 2015

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented May 25, 2015

note that the tarball checksums changed because of switching from /archive/
to /tarball/ url for downloading, which changes the leading folder name
inside the tarball

note that the tarball checksums changed because of switching from /archive/
to /tarball/ url for downloading, which changes the leading folder name
inside the tarball
@tkelman tkelman added the building Build system, or building Julia or its dependencies label May 25, 2015
@tkelman
Copy link
Contributor Author

tkelman commented May 27, 2015

@vtjnash does this look right?

@vtjnash
Copy link
Member

vtjnash commented May 27, 2015

Lgtm

tkelman added a commit that referenced this pull request May 27, 2015
@tkelman tkelman merged commit 4a685fa into master May 27, 2015
@tkelman tkelman deleted the tk/libgit2-external branch May 27, 2015 06:25
@ScottPJones
Copy link
Contributor

I wonder if this has just broken my ability to build julia?
I'm getting this:

fatal: unable to connect to github.com:
github.com[0: 192.30.252.131]: errno=Connection refused

make[1]: *** [libgit2] Error 128
make: *** [julia-deps] Error 2

@tkelman
Copy link
Contributor Author

tkelman commented May 27, 2015

That just sounds like your connection might be flaky?

@ScottPJones
Copy link
Contributor

No, I don't think so, git is working perfectly from the command line... (using https).

@tkelman
Copy link
Contributor Author

tkelman commented May 27, 2015

What happens if you do, say, make -C deps distclean-libuv && make -C deps get-libuv ?

@ScottPJones
Copy link
Contributor

rm -rf libuv libuv.tar.gz
git clone -q --depth=50 --branch julia-uv0.11.26 git://github.com/JuliaLang/libuv.git libuv
fatal: unable to connect to github.com:
github.com[0: 192.30.252.130]: errno=Connection refused

make: *** [libuv] Error 128

Why is it trying to use the git:: protocol instead of https?

@ScottPJones
Copy link
Contributor

(works fine with https::)

@tkelman
Copy link
Contributor Author

tkelman commented May 27, 2015

Because @vtjnash wrote #10743 that way? git config --global url."https://".insteadOf git:// should fix it

@ScottPJones
Copy link
Contributor

It seems to be ignoring my setting... it already is set to https://

[remote "JuliaLang"]
    url = https://github.com/JuliaLang/julia.git
    fetch = +refs/heads/*:refs/remotes/JuliaLang/*

@tkelman
Copy link
Contributor Author

tkelman commented May 27, 2015

That's just the remote for this repo, libuv and libgit2 and a few other deps are cloned separately now - you can try building with NO_GIT = 1 to download them as tarballs instead

@pao
Copy link
Member

pao commented May 27, 2015

If the machine is in a no-git-protocol zone, using the global-config rewrite (as @tkelman listed) is the preferred option.

@ScottPJones
Copy link
Contributor

The issue was, that there were no config entries, global or local, that had git://, so the rewrite wouldn't help...

@pao
Copy link
Member

pao commented May 27, 2015

Yes, it will. The X.insteadOf config option changes the URL on the fly, including ones entered at the command line (as when the make system clones libgit2, for instance.)

@ScottPJones
Copy link
Contributor

Ah!!! Thanks so much! I totally misunderstood @tkelman 's very good advice... I saw the "insteadOf", and thought he meant that I should manual change to https:// intead of git://... Must be the 2 hours of sleep!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants