Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Repository url can be https or git #3601

Closed
wants to merge 2 commits into from
Closed

Repository url can be https or git #3601

wants to merge 2 commits into from

Conversation

wildart
Copy link
Member

@wildart wildart commented Oct 1, 2015

No description provided.

@jiahao
Copy link
Member

jiahao commented Oct 1, 2015

I forget why we choose git over https, but this change will cause Travis to fail since every currently registered package uses git now and your change tests the assert on every package.

@tkelman
Copy link
Contributor

tkelman commented Oct 1, 2015

I think we will have to make this change but do it all at once. And once we pull the trigger on it, Pkg.generate run from 0.3 or 0.4 would always fail metadata travis. Not sure how to best manage this.

@ivarne
Copy link
Member

ivarne commented Oct 1, 2015

We chose git over https in #402, because both git and https had firewall issues, and the insteadOf configuration in git makes it possible to change on a per user basis.

@wildart
Copy link
Member Author

wildart commented Oct 1, 2015

Let's disable protocol check.

@tkelman
Copy link
Contributor

tkelman commented Oct 1, 2015

No, but maybe loosen it to accept either https or git so packages registered by 0.5 don't fail?

@ViralBShah
Copy link
Member

I like having fallbacks and trying different protocols before completely giving up.

@wildart wildart changed the title Use https instead of git Repository url can be https or git Oct 8, 2015
@hayd
Copy link
Member

hayd commented Oct 8, 2015

Does the protocol have to match the METADATA? For github projects the git/https URLs can be inferred from the project url (or inferred from one another). The protocol should surely be a user-setting rather than a project one... and the default should be https.

@wildart
Copy link
Member Author

wildart commented Oct 8, 2015

So, do we allow both protocols?

@wildart
Copy link
Member Author

wildart commented Oct 8, 2015

A package URL is normalized by Pkg into https anyway. Ah, not really.

@wildart
Copy link
Member Author

wildart commented Oct 8, 2015

The protocol should surely be a user-setting rather than a project one... and the default should be https.

New libgit2-backed package manager will support only https protocol. So, for any package in METADATA, we can convert URL to use https protocol in Pkg. By default, https will be used when a new package is generated in Julia PDK package. Let's allow both protocols for now.

@hayd
Copy link
Member

hayd commented Oct 8, 2015

The clone/remote url used for packages is currently git:// even in 0.5... Maybe I'm misunderstanding you.

IMO Even if git:// is the METADATA url for the package, https should be used (user-setting).

@wildart
Copy link
Member Author

wildart commented Oct 8, 2015

There is a small caveat. In Pkg 0.5, we disable ssh protocol in libgit2 for cross-platform reasons. Because, we host packages on github, we are allowed to use https or git protocols. But github version of git protocol is actually git+ssh (git itself does not support any authentication mechanism). There is no problem in using pure git protocol, but if repository is on github, we cannot use git protocol anymore because of ssh dependency. So, either we allow both protocols and deal with different protocols by overwriting URL in Pkg or enforce only https protocol for registered packages. Or do both and merge this PR.

@tkelman
Copy link
Contributor

tkelman commented Oct 13, 2015

superceded by #3752

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

Successfully merging this pull request may close these issues.

6 participants