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
ccache: 3.4.1 -> 3.7.10 #91589
ccache: 3.4.1 -> 3.7.10 #91589
Conversation
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.
Thank you for contributing to nixpkgs! I have added one requested change.
Thanks for the feedback, I’ll update the PR.
…--
For a successful technology, reality must take precedence over public relations, for nature cannot be fooled. (R. P. Feynman)
Am 27.06.2020 um 20:56 schrieb Daniël de Kok ***@***.***>:
@danieldk requested changes on this pull request.
Thank you for contributing to nixpkgs! I have added one requested change.
In pkgs/development/tools/misc/ccache/default.nix:
> src = fetchurl {
- sha256 = "1pppi4jbkkj641cdynmc35jaj40jjicw7gj75ran5qs5886jcblc";
- url = "mirror://samba/ccache/${pname}-${version}.tar.xz";
+ sha256 = "0a7dajdpnh14nmd10ji897rzqn6847lr36hdbig2rv7sxbc6d58c";
+ url = "https://github.com/ccache/${pname}/releases/download/v${version}/${pname}-${version}.tar.xz";
};
Since the source is on GitHub now, we can use fetchFromGitHub:
src = fetchFromGitHub {
owner = pname;
repo = pname;
rev = "v${version}";
sha256 = "0v6pvj50y18fxh2f6cx6xyg545xcpxfmrcg9bzbc6lc5acrvvjm7";
};
fetchFromGitHub should also be added as an argument to the derivation at the top. fetchurl can be removed, since it is then not used anymore.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
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.
Thanks for documenting the patches. One more small suggested change.
@@ -1,29 +1,23 @@ | |||
{ stdenv, fetchurl, perl, zlib, makeWrapper }: | |||
{ stdenv, fetchFromGitHub, asciidoc-full, gperf, perl, autoconf, zlib, makeWrapper }: |
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.
{ stdenv, fetchFromGitHub, asciidoc-full, gperf, perl, autoconf, zlib, makeWrapper }: | |
{ stdenv, fetchFromGitHub, asciidoc-full, gperf, perl, autoreconfHook, zlib, makeWrapper }: |
The fix-debug-prefix-map-suite patch became obsolete with upstream commit: ccache/ccache@82010e0 The skip-fs-dependent-test patch with PR: ccache/ccache#444
Result of 3 packages built:- baresip - ccache - ccacheStdenv (webbrowser was too large a dependency to test.) I also updated the commit message a bit to have links to the changes that deprecate the patches. Will merge when ofborg is happy. |
Motivation for this change
We're using Nix - provided ccache and were bitten by
This is the first release fixing this problem. The patches for disabling blocksize-dependent tests were dropped (the tests run without problems at least on ext4 and zfs).
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)