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

lsix: Add version 1.9.1 #6090

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

lsix: Add version 1.9.1 #6090

wants to merge 1 commit into from

Conversation

sitiom
Copy link
Contributor

@sitiom sitiom commented Aug 10, 2024

Like "ls", but for images. Shows thumbnails in terminal using sixel graphics.

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

lsix

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Copy link
Contributor Author

@sitiom sitiom left a comment

Choose a reason for hiding this comment

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

Issue: The lsix.cmd shim created by Scoop adds a 2>nul redirect which causes no output to display:

@rem C:\Users\rycae\scoop\apps\lsix\current\lsix
@bash "$(wslpath -u 'C:\Users\rycae\scoop\apps\lsix\current\lsix')"  %* 2>nul
@if %errorlevel% neq 0 (
  @bash "$(cygpath -u 'C:\Users\rycae\scoop\apps\lsix\current\lsix')"  %* 2>nul
)

Screenshot using Powershell under Windows Terminal Canary:
image

Removing the redirect makes it work otherwise:
image

Opened issue upstream:

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

Successfully merging this pull request may close these issues.

1 participant