Repository navigation
The Terminal's .deb installs amcli, with a man page and completion - #1662
Merged
Merged
Conversation
The package's launcher, angourimath-terminal, ran the Terminal without its arguments, so the one-shot commands of #1654 could not be reached from the .deb, and the package had no amcli command at all. The launcher now has a shebang and passes "$@" on, and amcli is a link to it. The package also installs amcli.1 under /usr/share/man/man1 (#1658) and a bash completion script for the commands under /usr/share/bash-completion/completions (#1659). The script packs under umask 022, so that the package's modes do not depend on who packs it: under a stricter umask, dpkg-deb refused the control directory, and the installed files would have been unreadable to other users. Closes #1658, closes #1659, part of #1660. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura
#1661 merged, so info is one of amcli's commands now. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura
Rafael-SOWNet
force-pushed
the
the-deb-installs-amcli
branch
from
October 1, 2026 23:47
3489d0c to
885bf4f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1658, closes #1659, part of #1660.
The launcher dropped its arguments. The Terminal's
.debinstallsangourimath-terminal, which rancd …-data && ./AngouriMath.Terminal, with no"$@"and no shebang. So the one-shot commands of #1654 couldn't be reached from the package, and the package had noamclicommand at all. Now:#!/bin/sh, thencd … && exec ./AngouriMath.Terminal "$@";amcliis a link to it;amcli.1under/usr/share/man/man1(manentry #1658) and a bash completion script for the commands under/usr/share/bash-completion/completions(bash-autocompletion script #1659).Packing under umask 022. Under a stricter umask,
dpkg-debrefused the control directory, and the installed files would have been unreadable to other users. The script now sets the umask itself, so the package's modes don't depend on who packs it.Checked, by publishing for the three Linux runtimes and packing all three
.debs:dpkg-deb -clists the launcher at 755,amcli -> angourimath-terminal, the man page and the completion script, both 644.simp "sin(x)^2 + cos(x)^2"with1, andecho "1 + x^2" | … diff xwith2 * x.man -lrenders the page, and the completion offers the commands foramcliandangourimath-terminal.The workflow's
pull_requestpath filter now covers the man page and the completion script, so this PR packs on CI too.Since #1661 merged, the man page and the completion list include
info, and the branch is rebased onto it.For #1660: Debian is covered by this. Nix and the AUR are packages kept outside this repository, and I've proposed moving them to Future on that issue.
🤖 Generated with Claude Code
https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura