All three variants build a derivation called ammonite-3.0.9, so the
description is what tells them apart in nix search and nix profile list.
Assisted-by: Claude Code (Claude Opus 5)
ammonite.AmmoniteMain is class-file 55, so an older jre failed the install
check with UnsupportedClassVersionError buried in a build log rather than
saying why.
Assisted-by: Claude Code (Claude Opus 5)
The repl pinned its own jvm but handed subprocesses whatever the shell
had. makeWrapper is not usable here because the launcher passes itself as
the classpath with -cp "$0", so the exports go ahead of the exec instead,
and the result is checked rather than assumed.
Assisted-by: Claude Code (Claude Opus 5)
The sed behind it looks for ammonite.Main', but the 3.x launcher runs
ammonite.AmmoniteMain, so it has matched nothing for some time and the
flag was never added: the default build contains no --no-remote-logging.
Upstream removed the feature as well, and answers the flag with "Option
--no-remote-logging is deprecated (remote logging has been removed)".
So the argument only ever chose between two identical launchers, except
that passing it false also skipped the shebang.
The built output is unchanged. This does drop the argument from the
.override interface, so `ammonite.override { disableRemoteLogging = false; }`
now fails to evaluate rather than quietly doing nothing; nothing in the
tree passes it.
Assisted-by: Claude Code (Claude Opus 5)
Upstream ships a .bat/.sh polyglot with no interpreter directive, and the
sed adding one sat inside the disableRemoteLogging block, so building with
that argument false produced a launcher execve cannot start: shells retry
it under /bin/sh, but anything calling execve directly gets ENOEXEC.
The default build is byte-identical; only the --no-remote-logging flag
stays conditional.
Assisted-by: Claude Code (Claude Opus 5)
The launcher carries an appended jar, so sourceProvenance is
binaryBytecode rather than the fromSource default.
Assisted-by: Claude Code (Claude Opus 5)
It refreshed ammonite_2_12 and ammonite_2_13 only, so a bump left
ammonite_3_3 - which is what the plain ammonite attribute points at - on
the new version with its old hash. The variants are now one table the
script is generated from, so none can be missed.
It also drove git ls-remote off git@github.com:..., which needs an ssh key
and so could never have run for r-ryantm.
The rest is the shape the other Scala packages use: shellcheck at build
time, a downgrade guard, prose on stderr, package.nix restored if a
refresh fails partway, and the commit feature declared.
Assisted-by: Claude Code (Claude Opus 5)
The launcher picks $JAVA_HOME over its own default when set, and only the
default was being patched, so `override { jre = ... }` had no effect in any
shell that exports JAVA_HOME - which is most of them. Ammonite 3 needs
Java 11, so on an older JAVA_HOME it crashed in the launcher instead.
sed rather than substituteInPlace because the launcher carries an appended
jar, and null bytes make substituteInPlace refuse the file. Since sed is
silent when it matches nothing, which is how this went unnoticed, the
result is checked.
Assisted-by: Claude Code (Claude Opus 5)
This commit was created by a combination of scripts and tools:
- an ast-grep script to prefix things in meta with `lib.`,
- a modified nixf-diagnose / nixf combination to remove unused `with
lib;`, and
- regular nixfmt.
Co-authored-by: Wolfgang Walther <walther@technowledgy.de>
After final improvements to the official formatter implementation,
this commit now performs the first treewide reformat of Nix files using it.
This is part of the implementation of RFC 166.
Only "inactive" files are reformatted, meaning only files that
aren't being touched by any PR with activity in the past 2 months.
This is to avoid conflicts for PRs that might soon be merged.
Later we can do a full treewide reformat to get the rest,
which should not cause as many conflicts.
A CI check has already been running for some time to ensure that new and
already-formatted files are formatted, so the files being reformatted here
should also stay formatted.
This commit was automatically created and can be verified using
nix-build a08b3a4d19.tar.gz \
--argstr baseRev b32a094368
result/bin/apply-formatting $NIXPKGS_PATH
Upstream doesn't include a shebang line but recommends adding one at
installation time. NixOS users noticed that the command wouldn't exec
when using certain shells.
