Skip to content

Add tmux feature#106

Open
Copilot wants to merge 4 commits intomainfrom
copilot/add-tmux-feature
Open

Add tmux feature#106
Copilot wants to merge 4 commits intomainfrom
copilot/add-tmux-feature

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

Adds a tmux devcontainer feature. Since tmux only ships source tarballs on GitHub releases (no pre-built binaries), the install script builds from source using the repo's standard patterns.

Changes

  • src/tmux/devcontainer-feature.json — Feature metadata; version option defaults to latest, accepts tmux's X.Y[a] format (e.g. 3.5a, 3.4)
  • src/tmux/install.sh — Builds tmux from the GitHub release source tarball:
    • Installs build deps: build-essential, autoconf, automake, pkg-config, libevent-dev, libncurses-dev, bison
    • Adapts github_list_releases / utils_check_version regex from X.Y.Z to ^[0-9]+\.[0-9]+[a-z]?$ to match tmux's versioning scheme
    • Resolves latest/pinned release tag via GitHub API, downloads source tarball, builds with autogen.sh + ./configure --prefix=/usr/local + make install
  • test/tmux/test.sh — Asserts tmux -V | grep 'tmux'
  • .github/workflows/test.yaml — Adds tmux to both test-autogenerated and test-scenarios matrix lists

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/tmux/tmux/releases
    • Triggering command: /usr/bin/curl curl -s REDACTED conntrack --ctstate INVALID,NEW -j DROP (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Add tmux feature</issue_title>
<issue_description>Files:

src/tmux/devcontainer-feature.json
src/tmux/install.sh (make sure to set executable bit)
test/tmux/test.sh
.github/workflows/test.yaml

Releases:

https://github.com/tmux/tmux/releases</issue_description>

<agent_instructions>install from github releases with known patterns for install.sh</agent_instructions>

Comments on the Issue (you are @copilot in this section)


📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copilot AI changed the title [WIP] Add tmux feature to devcontainer Add tmux feature Mar 19, 2026
Copilot AI requested a review from sebst March 19, 2026 09:30
@sebst sebst marked this pull request as ready for review March 19, 2026 09:32
Copy link
Member

@sebst sebst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests fail

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 19, 2026

tests fail

Fixed in b8f9144. The test-autogenerated job was failing with bash: autogen.sh: No such file or directory because tmux release tarballs include a pre-generated configure script and don't ship autogen.sh. Removed the autogen.sh step along with the now-unneeded autoconf/automake/bison build deps — the install now just runs ./configure --prefix=/usr/local && make install directly.

Copilot AI requested a review from sebst March 19, 2026 10:16
Copy link
Member

@sebst sebst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests still failing

Co-authored-by: sebst <592313+sebst@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 19, 2026

tests still failing

Fixed in bf86955. The new failure was configure: error: "yacc not found" — tmux's ./configure requires yacc, which is provided by bison. The previous fix incorrectly removed bison along with autoconf/automake. Added bison back to the build deps.

Copilot AI requested a review from sebst March 19, 2026 11:29
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.

Add tmux feature

2 participants