| 17 Sep 2025 |
 Tomodachi94 (they/them) | Side note: had no idea that Swift was in the dependency graph for Ant | 01:29:58 |
 msgilligan | The wrapper scripts use the following environment variables (if set):
JAVACMD—full path of the Java executable. Use this to invoke a different JVM than JAVA_HOME/bin/java(.exe).
ANT_OPTS—command-line arguments that should be passed to the JVM. For example, you can define system properties or set the maximum Java heap size here.
ANT_ARGS—Ant command-line arguments. For example, set ANT_ARGS to point to a different logger, include a listener, and to include the -find flag.
Note: If you include -find in ANT_ARGS, you should include the name of the build file to find, even if the file is called build.xml.
| 01:31:31 |
| msgilligan | Looks like it defaults to JAVA_HOME unless you set JAVACMD | 01:32:37 |
| Tomodachi94 (they/them) | Hmm, maybe just setting JAVACMD not JAVA_HOME will work? I'll try it | 01:33:14 |
| msgilligan | Maybe the ant Nix package should work similar to Gradle where it runs with and builds with a default JDK but both can be overridden? (And maybe the defaults should be on the older side?) | 01:35:46 |
| Tomodachi94 (they/them) | Changing it to jdk8 worked out for fixing javacc, haven't tried others | 01:37:25 |
| Tomodachi94 (they/them) | Seems to be working for others when I use jdk8 by default | 01:40:43 |
| Tomodachi94 (they/them) | Other ant PRs of mine that could use some love and are less weird than this one:
- https://github.com/NixOS/nixpkgs/pull/357162
- https://github.com/NixOS/nixpkgs/pull/357162 (fixing the merge conflicts)
| 01:53:29 |
| msgilligan | The second bulleted item links to the same PR as the first. | 02:26:30 |
| msgilligan | I asked for one minor change to 357162. | 02:26:54 |
| msgilligan | And once 443476 one is merged, we can remove Oracle GraalVM 24:
https://github.com/NixOS/nixpkgs/pull/443634 | 03:11:23 |
| Tomodachi94 (they/them) | Oops, my bad. Here's the right one: https://github.com/NixOS/nixpkgs/pull/359137 | 03:41:10 |
| Tomodachi94 (they/them) | (And a bunch of good reasons for doing this: https://github.com/NixOS/nixpkgs/issues/357758) | 03:41:55 |
| Tomodachi94 (they/them) | Also add that the original ant-contrib has had no updates since... 200[4-9] iirc? | 03:43:24 |
| Tomodachi94 (they/them) | (There are some active forks that we could package if anything in-tree ends up needing it) | 15:21:15 |
| Tomodachi94 (they/them) | * | 16:43:09 |
| msgilligan | There's probably some Java interoperability component that is built with Ant. Ideally these things would be in separate derivations with Swift and Ant as inputs, right? | 18:27:57 |
| msgilligan | graalvm-oracle_25 is now the default graalvm-oracle. PR to drop graalvm-oracle_24 is ready-for-review:
https://github.com/NixOS/nixpkgs/pull/443634 | 18:30:02 |
| msgilligan | emily is always quick to review/merge the PRs that drop JDKs 😁 | 18:30:43 |
 emily | my weakness… | 18:41:42 |
| emily | wow beaten to it… | 18:41:58 |
| msgilligan | I view it as a strength! | 18:42:43 |
| msgilligan | I'm hoping that getting the new versions merged earlier will help us get the old versions removed faster. | 18:59:46 |
| msgilligan | I fixed the nixpkgs-vet error about new top-level package using pkgs.callPackage in the zulu25 PR: https://github.com/NixOS/nixpkgs/pull/443517/commits/f65e6c648d89794fc801d95280355dcca779f990 | 19:54:01 |
| msgilligan | This is my first time refactoring usage of callPackage so I'm not sure I did it right, but it looks reasonably clean and seems to work, so please take a look and let me know if I got it right. | 19:56:19 |
| emily | you can inherit ({ zulu8 = callPackage …; … }) zulu8 …; | 19:56:35 |
| emily | (yes it's silly) | 19:56:37 |
| msgilligan | in the new default.nix? That seems more verbose, is there an advantage to doing it that way? | 19:58:47 |
| emily | no, in all-packages.nix | 20:00:53 |
| emily | it's just the officially-sanctioned hack for when there is an existing multi-version package outside of by-name | 20:01:04 |
