acdw
/
bollux
1
1
Fork 1
Code Issues 5 Pull Requests Projects Releases Wiki Activity

less key bindings not honored in bollux 0.4.1 on slackware64 15.0 #13

New Issue
Open
opened 2022-07-21 15:38:46 +00:00 by bencollver · 3 comments
bencollver commented 2022-07-21 15:38:46 +00:00
Copy Link

I installed bollox 0.4.1 on slackware64 15.0.

bash-5.1$ less --version
less 590 (POSIX regular expressions)

When i run "bollux gemini://tilde.pink/~bencollver/" it loads the page as expected. However, it shows the following message on startup.

NOTE: lesskey is deprecated.
      It is no longer necessary to run lesskey,
      when using less version 582 and later.

When i press "=" it shows the short help message, which i can dismiss by pressing RETURN.

When i press "o" to open a link, nothing happens and the screen remains the same.

When i press "[" i get an error: No matching bracket (press RETURN)

When i press "]" i get an error: No bracket in bottom line (press RETURN)

I installed bollox 0.4.1 on slackware64 15.0. ``` bash-5.1$ less --version less 590 (POSIX regular expressions) ``` When i run "bollux gemini://tilde.pink/~bencollver/" it loads the page as expected. However, it shows the following message on startup. ``` NOTE: lesskey is deprecated. It is no longer necessary to run lesskey, when using less version 582 and later. ``` When i press "=" it shows the short help message, which i can dismiss by pressing RETURN. When i press "o" to open a link, nothing happens and the screen remains the same. When i press "[" i get an error: No matching bracket (press RETURN) When i press "]" i get an error: No bracket in bottom line (press RETURN)
acdw referenced this issue from a commit 2022-08-10 04:40:46 +00:00
Rewrite mklesskey to fix #13
acdw closed this issue 2022-08-10 04:40:46 +00:00
acdw commented 2022-08-10 04:41:14 +00:00
Owner
Copy Link

Thanks for the update, ben! I'm not able to test it on my machine, but hopefully it'll work now!

Thanks for the update, ben! I'm not able to test it on my machine, but hopefully it'll work now!
bencollver commented 2022-09-01 03:41:59 +00:00
Author
Copy Link

I pulled the latest code and now it gives a different error:

Cannot use lesskey file "/home/ben/.local/share/bollux/lesskey"

As an experiment, i made the following change to bollux:

OLD: mklesskey && less_cmd+=(-k "$BOLLUX_LESSKEY")
NEW: mklesskey && less_cmd+=(--lesskey-src="$BOLLUX_LESSKEY")

This resolved the problem for me, but it won't work with older versions of less.

New versions need to use --lesskey-src=

Old versions need to use -k

I pulled the latest code and now it gives a different error: Cannot use lesskey file "/home/ben/.local/share/bollux/lesskey" As an experiment, i made the following change to bollux: ``` OLD: mklesskey && less_cmd+=(-k "$BOLLUX_LESSKEY") NEW: mklesskey && less_cmd+=(--lesskey-src="$BOLLUX_LESSKEY") ``` This resolved the problem for me, but it won't work with older versions of less. New versions need to use --lesskey-src= Old versions need to use -k
acdw commented 2022-09-01 17:11:55 +00:00
Owner
Copy Link

Oh my god. Thanks for testing and figuring this out, @bencollver ! I'll try and fix this ASAP.

Oh my god. Thanks for testing and figuring this out, @bencollver ! I'll try and fix this ASAP.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
bookmarks
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: acdw/bollux#13
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?