Skip to content

Add dotcl-bin to the implementation check - #41

Merged
svetlyak40wt merged 1 commit into
40ants:masterfrom
snmsts:dotcl
Oct 13, 2026
Merged

svetlyak40wt merged 1 commit into
40ants:masterfrom
snmsts:dotcl

Conversation

@snmsts

@snmsts snmsts commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Roswell v26.10.117 and later can install dotcl
(a Common Lisp implementation on .NET) as dotcl-bin. With this action,
everything up to the last step already works; only the final check in
test.ros fails because dotcl-bin is not in its table. This PR adds the entry.

Usage:

- uses: 40ants/setup-lisp@v4
  with:
    roswell-version: v26.10.118

and LISP: dotcl-bin in the environment. dotcl needs .NET 10, which is
preinstalled on GitHub's ubuntu-latest runners.

I left the default roswell-version alone. Two things changed in
v26.10.118 that may matter if you decide to bump it later:

  • scripts/install-for-ci.sh of a release tag now installs that same release
    (before, a tag's script pointed at the previous one).
  • On Linux the script downloads the prebuilt binary for any architecture
    (x86_64, arm64, riscv64, loongarch64) instead of only x86_64, and falls back
    to building from source when there is no binary.

Tested on ubuntu-latest with roswell-version: v26.10.118 in
https://github.com/dotcl/xmls/actions/runs/38234272709 — dotcl-bin,
sbcl-bin, ccl-bin/1.12.2, ecl, cmu-bin/21e, clisp/2.49, clisp-head
and abcl-bin/1.9.2 all pass.

Not working on macOS yet: there the action installs Roswell from Homebrew
regardless of roswell-version, and the current macos-latest image still
resolves the formula to 26.02.116, which does not know dotcl-bin (the
macos-latest / dotcl-bin job in the run above fails for this reason).
It should start working once the runner image picks up a newer formula.

I did not touch the test matrix or the implementation table in the README,
since they look generated.

@svetlyak40wt

Copy link
Copy Markdown
Member

Thank you!

@svetlyak40wt
svetlyak40wt merged commit 1a5e7c0 into 40ants:master Oct 13, 2026
@snmsts

snmsts commented Oct 13, 2026

Copy link
Copy Markdown
Contributor Author

Thank you for merging!

Do you plan to make a new release? I want to use this from
40ants/setup-lisp@v4 in another project. I can use the commit hash
for now, so there is no hurry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants