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
gradle: 5.6.4 -> 6.5.1 #92103
gradle: 5.6.4 -> 6.5.1 #92103
Conversation
@ludovicc thanks for opening your first pr to nixpkgs :) |
nativeVersion = "0.22"; | ||
|
||
src = fetchurl { | ||
url = "http://services.gradle.org/distributions/${name}-bin.zip"; |
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 get a 403:
builder for '/nix/store/v34wfmafk54chdkascgsycwmsdnw1ypc-gradle-6.5.1-bin.zip.drv' failed with exit code 1; last 7 log lines:
trying http://services.gradle.org/distributions/gradle-6.5.1-bin.zip
% Total % Received % Xferd Average Speed Time Time Time Current
Dload Upload Total Spent Left Speed
0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:-- 0
curl: (22) The requested URL returned error: 403 Forbidden
error: cannot download gradle-6.5.1-bin.zip from any mirror
@jonringer Thanks for the welcome and the review. It looks like you had some kind of transient error while downloading the zip, http://services.gradle.org/distributions/gradle-6.5.1-bin.zip works both from browser and using nix-review on this pr. |
I use https for all urls as suggested. |
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.
regression in jd-gui
package.
also, please squash fixup commits to adhere to CONTRIBUTING.md
https://github.com/NixOS/nixpkgs/pull/92103
1 package failed to build:
jd-gui
4 packages built:
frostwire gradle jadx mxisd
Co-authored-by: Daniël de Kok <me@github.danieldk.eu> Use https for all urls fix version for native library pin gradle for jd-gui
jd-gui should work better, I pinned gradle to version 5 as used by this package. |
@@ -1,4 +1,4 @@ | |||
{ stdenv, fetchFromGitHub, jre, jdk, gradle, makeDesktopItem, perl, writeText, runtimeShell }: | |||
{ stdenv, fetchFromGitHub, jre, jdk, gradle_5, makeDesktopItem, perl, writeText, runtimeShell }: |
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.
the gradle_5 pinning should probably be done in another commit
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.
jd-gui: pin gradle to 5.X
gradle: 5.6.4 -> 6.5.1
@jonringer I have moved the gradle_5 pinning to #92292, this pr is now dependant on #92292 |
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.
https://github.com/NixOS/nixpkgs/pull/92103
4 packages built:
frostwire gradle jadx mxisd
Motivation for this change
Upgrade Gradle to latest version 6.5.1
Add gradle_6 to the top level packages.
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)