| 2 Jul 2025 |
 msgilligan | But that isn't the right place to talk about testing JDK updates. | 03:23:49 |
| msgilligan | I wrote a script (in Java!) to pull down the Zulu versions and hashes for updating a specified JDK version, but it’s not quite an auto-updater yet | 04:31:27 |
 Infinidoge 🏳️⚧️ | In reply to @msgilligan:matrix.org
What is the situation for automatic testing of JDKs on Nix? I've been looking through the documentation here and the source code of the various packages and it has been a slow learning process.
It would be nice if there was some general documentation on the various JDK packages in Nixpkgs as well as documentation specifically on contributing to the JDK modules such as how to run updaters (where they exist), how to run tests, how to add tests, etc.
Does anything like that exist? If not, where should it go so I can make an attempt to start one or more documents?
I haven't thought about it until now, but it's surprising that there doesn't seem to be any tests for the JDK in particular outside of generic builds/doesn't build | 04:38:28 |
| msgilligan | There are some really basic tests in correct to | 04:41:01 |
| msgilligan | * There are some really basic tests in corretto | 04:41:11 |
| Infinidoge 🏳️⚧️ | Oh huh, so there are, those should be ported to all of the other JDKs as a sanity check | 04:42:26 |
| msgilligan | Yeah, I was thinking it would make sense to have one set of tests that can run against all the different JDKs | 04:43:50 |
| Infinidoge 🏳️⚧️ | Something in pkgs/test at least | 04:44:09 |
| msgilligan | I made this draft PR earlier today: https://github.com/NixOS/nixpkgs/pull/421637 | 04:52:23 |
| Infinidoge 🏳️⚧️ | Put a review. installCheckPhase vs passthru tests I'm not entirely sure, but I think things like checking version should be installCheckPhase, and checking compilation should be passthru tests. | 05:07:24 |
| Infinidoge 🏳️⚧️ | (Above and beyond the checkPhase they should probably already have) | 05:07:48 |
| msgilligan | Yeah, I started writing that before I even knew about passthru. All the different options for tests are pretty confusing for a beginner. | 05:26:41 |
| Infinidoge 🏳️⚧️ | Nix in a nutshell, really | 05:27:02 |
| msgilligan | Well, it’s always easier to add a test when there are existing tests to add to. But I do think a paragraph or two about where the tests are or will/should be a would be helpful | 05:32:09 |
| msgilligan | Maybe someday we can run this:
https://adoptium.net/aqavit/ | 06:28:16 |
| msgilligan | I just realized this suggestion is wrong. The Languages and frameworks section is under Build Helpers and there is no JDK Build Helper. | 14:29:20 |
| 3 Jul 2025 |
| msgilligan | I'm getting schooled by @dtomvan (and GitHub checks) on testing on (above) PR #421637 😁 | 00:05:28 |
| 4 Jul 2025 |
| msgilligan | Merged: https://github.com/NixOS/nixpkgs/pull/421637 | 15:29:28 |
| msgilligan | * Merged: https://github.com/NixOS/nixpkgs/pull/421637
Thanks everyone for their help! | 15:30:40 |
| msgilligan | and I rebased https://github.com/NixOS/nixpkgs/pull/421362 so it will now get (minimally) tested. | 16:00:56 |
| 10 Jul 2025 |
|  @aerunakai:matrix.org joined the room. | 05:43:42 |
| 11 Jul 2025 |
| @aerunakai:matrix.org removed their profile picture. | 16:40:28 |
| @aerunakai:matrix.org removed their display name aerunakai. | 16:40:34 |
| @aerunakai:matrix.org left the room. | 16:40:41 |
| 12 Jul 2025 |
|  plan9better joined the room. | 11:05:56 |
| 13 Jul 2025 |
| msgilligan | Does anyone on this list have the ability to merge PR #421362 (above) ? It's a straightforward version bump and has 2 approvals. | 23:58:08 |
|  @softmoonworld:matrix.org joined the room. | 23:58:56 |
| 14 Jul 2025 |
| @softmoonworld:matrix.org | has anyone had any luck with using nix-ld within the context of using a flake as a java dev env? I could use some clarity since I'm new to nixOS and flakes | 00:10:44 |
| msgilligan | * Does anyone on this list have the ability to merge PR #421362 (above) ? It's a straightforward version bump and has 2 approvals. Update: it's been merged! | 15:54:14 |
| @softmoonworld:matrix.org | i think i don't need to use a flake, i'm going to focus on working with nix-shell and ensuring LD_LIBRARY_PATH is path.
| 15:58:35 |
