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
eclipse: add Eclipse IDE for Java Developers 4.7 and 4.8 #48395
Conversation
|
||
### Eclipse Java | ||
|
||
eclipse-java-47 = buildEclipse { |
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.
Since this is a new package and there is no concern about accommodating people preferring to stay on an older version I think we can remove the 4.7 version here.
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.
Ok
else throw "Unsupported system: ${stdenv.system}"; | ||
}; | ||
|
||
eclipse-java-48 = buildEclipse { |
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.
Should add an eclipse-java = eclipse-java-48;
entry for people that always want the latest version.
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.
Actually, isn't the 2018-09 release called 4.9.0? If so then the package and attribute names should be updated to match.
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.
Don't know how i missed that. I am gonna fix that and also gonna make an eclipse-java entry for latest version.
|
||
eclipse-java-48 = buildEclipse { | ||
name = "eclipse-java-4.8.0"; | ||
description = "Eclipse IDE for Java Developers, Oxygen release"; |
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.
Can remove the ", Oxygen release" part.
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 forgot to change it to Photon release. But it's ok if i just don't say anything about the release?
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.
Yeah, I think that should be fine. Alternatively, if you want, then put
description = "Eclipse IDE for Java Developers 2018-09"
similar to eclipse-platform-49 and eclipse-sdk-49.
Thanks for the contribution! Looks good overall, I just added a few minor comments. |
Changed entry eclipse-java-48 to eclipse-java-49 to match the files represented inside the entry. Added eclipse-java entry pointing to latest version (eclipse-java-49)
@GrahamcOfBorg build eclipses.eclipse-java |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: eclipses.eclipse-java Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: eclipses.eclipse-java Partial log (click to expand)
|
Squashed and rebased to master in 6fe0f07. Thanks! |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)