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
ed: 1.13 -> 1.14.1 #21855
ed: 1.13 -> 1.14.1 #21855
Conversation
@lsix, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vcunat, @matthiasbeyer and @edolstra to be potential reviewers. |
"http://fossies.org/linux/privat/${name}.tar.bz2" | ||
]; | ||
sha256 = "1iym2fsamxr886l3sz8lqzgf00bip5cr0aly8jp04f89kf5mvl0j"; | ||
url = "http://fossies.org/linux/privat/${name}.tar.gz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is the mirror removed again? #21757
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have remove it because the version we are looking for (1.14.1) is not present in the fedora repo. Should we wait for it to be present in this repo before we proceed with this MR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is available, but they upgraded the hash algorithm to sha512:
http://pkgs.fedoraproject.org/repo/extras/ed/ed-1.14.1.tar.bz2/sha512/84396fe4e4f0bf0b591037277ff8679a08b2883207628aaa387644ad83ca5fbdaa74a581f33310e28222d2fea32a0b8ba37e579597cc7d6145df6eb956ea75db/ed-1.14.1.tar.bz2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
md5 is easy to find out now, so we can commit it, and hopefully Fedora will mirror it soon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great. I checked with the md5 hash and had a 404 reply, but since the sha256 version is there, I’ll use it.
See http://lists.gnu.org/archive/html/info-gnu/2017-01/msg00004.html for full release announcement.
@Mic92 I’ve updated my commit to include the fedora repo in the |
This PR is against the
staging
branch.See http://lists.gnu.org/archive/html/info-gnu/2017-01/msg00004.html
for full release announcement.
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)