Created attachment 216132 [details]
* portlint: OK (looks fine).
* testport: OK (poudriere: 12.1-RELEASE-p5, amd64).
MFH: No (Feature only release)
Thanks for the patch. I have a question. Does this port need bash/zsh to work? I couldn't make it work in a jail with csh:
root@12_1i386-default:~ # broot
Broot should be launched using a shell function (see https://github.com/Canop/broot for explanations).
Can I install it now? [Y n]
Writing br shell function in /root/.local/share/broot/launcher/bash/1.
Creating link from /root/.config/broot/launcher/bash/br to /root/.local/share/broot/launcher/bash/1.
I found no sourcing file for the bash/zsh family.
If you're using bash or zsh, then installation isn't complete:
the br function initialization script won't be sourced unless you source it yourself.
(In reply to Fernando Apesteguía from comment #1)
It's not a hard dependency in order to make broot work, what's installed is a set of shell aliases that allow you to change the directory you're currently in by pressing alt + enter. At this current point in time broot doesn't come with any csh aliases that grant this behavior, so explicitly marking bash/zsh/foo as a runtime dependency I wasn't in favor of (which also presumes either of those are set as the users shell). Whilst there's no csh alias you can still use the application, just without that particular feature. For further readup see: https://dystroy.org/broot/install/#installation-completion-the-br-shell-function - it's rather more for convenience above all. :)
(In reply to Lewis Cook from comment #2)
Understood. Thanks for the clarification!
A commit references this bug:
Date: Mon Jul 6 15:54:16 UTC 2020
New revision: 541349
misc/broot: Update to 0.18.3
Submitted by: email@example.com (maintainer)