-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
jikes: removing #70666
jikes: removing #70666
Conversation
@cke You need to remove the callPackage in |
And by an alias I mean something like https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/aliases.nix#L309. And we'll remove it in the next nixos release. |
Thanks @worldofpeace ! I've updated the PR. |
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.
LGTM, will backport to the beta release. Can you squash your commits into one?
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.
nix-review
looks right
$ nix-env -f /home/jon/.cache/nix-review/pr-70666/nixpkgs -qaP --xml --out-path --show-trace --meta
Nothing changed
https://github.com/NixOS/nixpkgs/pull/70666
No packages were successfully build, skip nix-shell
$ git worktree prune
EDIT: please squash commits into one :)
I squashed the commits into one |
commits should be written in the "imperative mood" https://git.kernel.org/pub/scm/git/git.git/tree/Documentation/SubmittingPatches?id=HEAD#n133 but it's close enough so I'll merge :)
|
Motivation for this change
Jikes was an alternative Java compiler. The last release was in 2005 and it hasn't been touched in years. The last Java version it (partially) supports is Java 5. There are no packages depending on it.
Homepage: https://sourceforge.net/projects/jikes/
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)