Skip to content
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

Drop end‐of‐life JDK and JavaFX versions 16 to 20 #331514

Merged
merged 4 commits into from
Aug 1, 2024

Conversation

emilazy
Copy link
Member

@emilazy emilazy commented Aug 1, 2024

Description of changes

This is a prerequisite to my FFmpeg 4 work, but also just a good idea in general. Per https://endoflife.date/oracle-jdk and https://endoflife.date/amazon-corretto, these versions have been unsupported for between 10½ and 22½ months and it’ll be 3 months more by the time the next NixOS release comes out. They were all marked as insecure anyway, and some weren’t even exposed at the top level, so nobody should miss them.

I’ve also removed the end‐of‐life Temurin and Semeru packages for these versions, per https://endoflife.date/eclipse-temurin and https://endoflife.date/ibm-semeru-runtime. These are less of a maintenance burden, though, and don’t block my FFmpeg work, so if people really want to keep them around despite them being marked as insecure I can drop that commit.

Closes: #170825

Result of nixpkgs-review run on x86_64-linux 1

1 package built:
  • mediathekview

Result of nixpkgs-review run on aarch64-linux 1

1 package built:
  • mediathekview

Result of nixpkgs-review run on aarch64-darwin 1

1 package built:
  • mediathekview

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

Not “drop”; these files are literally not referenced anywhere in
the tree, as far as I can tell.
These have all been end‐of‐life for more than 10 months.
Seems to start up fine for me with the default JRE, and the package
couldn’t be easily used before this anyway as version 20 has reached
end of life.

It does complain about what a big mistake we’re making by not passing
Shenandoah GC flags to the JVM on start‐up, but I doubt that has
anything to do with the JRE bump, and it lets you click through.
@emilazy
Copy link
Member Author

emilazy commented Aug 1, 2024

@Aleksanaa @inclyc Any way to tell nixf-tidy “no, we want to keep this unused definition, it’ll be useful later”?

@Aleksanaa
Copy link
Member

See #331085. You just comment it out or ignore it for now, it's fine

These have all been end‐of‐life for more than 10 months.
@emilazy
Copy link
Member Author

emilazy commented Aug 1, 2024

Commenting it out works for me; pushed.

@thiagokokada thiagokokada merged commit 4f93a66 into NixOS:master Aug 1, 2024
28 checks passed
@emilazy emilazy mentioned this pull request Aug 21, 2024
13 tasks
@emilazy emilazy deleted the push-yyzkwpzzzult branch August 26, 2024 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Remove openjdk 12, 13, 14, 15, 16
3 participants