Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tmux: add --enable-sixel #4419

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

tmux: add --enable-sixel #4419

wants to merge 1 commit into from

Conversation

Kreijstal
Copy link
Contributor

This commit adds the --enable-sixel flag of tmux.

This commit adds the --enable-sixel flag of tmux.
@lazka
Copy link
Member

lazka commented Feb 15, 2024

increase pkgrel.

and why is this needed/wanted?

@lazka
Copy link
Member

lazka commented Feb 17, 2024

to be clear, I don't oppose this, I'd just like to have some context

@Kreijstal
Copy link
Contributor Author

to be clear, I don't oppose this, I'd just like to have some context

Well, it simply adds sixel capabilities to tmux, so while mintty is sixel capable when you are using tmux, it your sixels won't be seen. Which is a shame, but now that I think of it, the libsixel library might be a build-dependency on tmux when this flag is enabled. I'd have to test this.

@Kreijstal Kreijstal marked this pull request as draft February 18, 2024 00:07
@Biswa96
Copy link
Member

Biswa96 commented Feb 18, 2024

but now that I think of it, the libsixel library might be a build-dependency on tmux when this flag is enabled.

According to https://gitlab.archlinux.org/archlinux/packaging/packages/tmux/-/commit/3f25ffee1121a4eb837a292a19d49751a70dd49d

as it turns out that tmux comes with its own implementation, and does not depend in libsixel, at all.

@Biswa96
Copy link
Member

Biswa96 commented Feb 18, 2024

I have tested the CI artifact but sixel is not shown in tmux with and without conpty. This repository may help to test sixel https://codeberg.org/coffee/sixel-experiments e.g. cat text-test.sixel

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.

None yet

3 participants