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

Mariadb: Fix URL for source tarball. #27076

Merged
merged 1 commit into from
Jul 3, 2017
Merged

Conversation

sh01
Copy link
Contributor

@sh01 sh01 commented Jul 3, 2017

mariadb.org appears to have changed their URL schemes, and the tarball URL used
by this derivation no longer works, which makes this unbuildable from scratch.

This change updates that URL to a mariadb.org location that will still serve
this tarball.

Hash is unchanged.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • 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.

mariadb.org appears to have changed their URL schemes, and the tarball URL used
by this derivation no longer works, which makes this unbuildable from scratch.

This change updates that URL to a mariadb.org location that will still serve
this tarball.

Hash is unchanged.
@@ -18,8 +18,9 @@ common = rec { # attributes common to both builds
version = "10.1.21";

src = fetchurl {
url = "https://downloads.mariadb.org/interstitial/mariadb-${version}/source/mariadb-${version}.tar.gz";
url = "https://downloads.mariadb.org/f/mariadb-${version}/source/mariadb-${version}.tar.gz/from/http%3A//ftp.hosteurope.de/mirror/archive.mariadb.org/?serve";
Copy link
Member

Choose a reason for hiding this comment

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

The new URL looks like requesting a specific mirror, which isn't a nice way...

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, apparently half of their mirrors don't have our version anymore :-/

vcunat added a commit that referenced this pull request Jul 3, 2017
Not even all their mirrors serve our version, unfortunately.

(cherry picked from commit cc83f35)
@vcunat vcunat merged commit cb9fe8c into NixOS:master Jul 3, 2017
vcunat added a commit that referenced this pull request Jul 3, 2017
Not even all their mirrors serve our version, unfortunately.
@sh01 sh01 deleted the shpr_fixurls_mariadb branch July 3, 2017 18:55
adrianpk added a commit to adrianpk/nixpkgs that referenced this pull request May 31, 2024
Not even all their mirrors serve our version, unfortunately.

(cherry picked from commit cc83f35)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants