xyproto / orbiton

Configuration-free text editor and IDE limited to VT100. Suitable for writing git commit messages, editing Markdown, config files, source code, man pages and for quick edit-format-compile cycles when programming. Has syntax highlighting, jump-to-error, rainbow parentheses, macros, llm tab completion, cut/paste portals and a simple gdb front-end.
https://roboticoverlords.org/orbiton
BSD 3-Clause "New" or "Revised" License
501 stars 15 forks source link

MANPAGER=o no longer works #14

Closed kseistrup closed 1 year ago

kseistrup commented 1 year ago

When I set MANPAGER=o, all I get when attempting to view a manpage is o saying “Loaded empty file: - (read only)”.

It looks like o is not reading from standard input, and when invoked as o - (some utilities regard - as a synonym for standard input), o will just create a file named -.

So I am wondering: Does MANPAGER=o really work?

Is this possibly related to #7?


xyproto commented 1 year ago

Thanks for reporting! I'll check.

xyproto commented 1 year ago

It works as it should here, using Alacritty and ZSH, on Arch Linux.

I tried MANPAGER=o man ls.

Which terminal emulator are you using?

kseistrup commented 1 year ago

Which terminal emulator are you using?

I have tried in terminator, sakura, xterm, lilyterm, and alacritty (!) × the shells fish, bash, and osh.

I have compiled o from the sources (from the orbiton package on AUR (pkgver=2.60.4-1)), but I get the same result with the pre-compiled version for x86_64 that I can download from the release section here. No difference.

My pc is running with the en_DK.UTF-8 locale. Switching to C does not help.

I can see that the man command is spawning o, but o claims that the file is empty. However, setting MANPAGER=cat tells me that the input is non-empty (as expected).

Running ps | o will show me the same “Loaded empty file: -” message at the bottom of o's “window”, and from my point of view it looks like o cannot or does not read anything from standard input.

(I tried the pre-compiled binary for aarch64 on an RPi just now and see the same thing (not reading standard input).)

xyproto commented 1 year ago

Thanks for all the testing! I'll try reproducing the issue.

xyproto commented 1 year ago

I can reproduce the issue now. 👍

xyproto commented 1 year ago

It works again here now. Please test. :)

kseistrup commented 1 year ago

Thanks, it works now! :+1: