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

fetchFromGitLab: Use API archive download endpoint. #39467

Merged
merged 1 commit into from May 1, 2018

Conversation

typetetris
Copy link
Contributor

According to

https://gitlab.com/gitlab-org/gitlab-ce/issues/45586#note_69828451

this URL is something with guaranteed stability.

Motivation for this change

Trying to get the "officially" correct archive download URL for gitlab.com

Things done

As this doesn't change any hashes, it doesn't trigger any rebuilds and is hard to test.

I manually tried it for procps-ng and checked that the downloaded archive doesn't differ from
a checked out clone of the specified repository.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@typetetris typetetris mentioned this pull request Apr 25, 2018
8 tasks
Copy link
Member

@matthewbauer matthewbauer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

%2F is just / right?

@typetetris
Copy link
Contributor Author

Yes and no. It is necessary to write it as %2F for gitlab.com's api. The difference is, written as %2F it doesn't count as path separator of the URL but as a character of the path component.

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

Successfully merging this pull request may close these issues.

None yet

3 participants