Re: [Update] sysutils/skim

On Tue, Aug 11, 2026 at 02:51:07PM +0100, Laurence Tratt wrote:
> On Wed, Aug 05, 2026 at 03:14:02PM +0100, Laurence Tratt wrote:
> 
> > This updates sysutils/skim to the latest version. Semantic versioning makes
> > the version number jump seem huge. This version seems to have fixed an
> > issue I had with the previous default search algorithm. There's also a new
> > `--image` feature which can render images in compatible terminals (I don't
> > need this personally, but have verified that it works).
> 
> If anyone has a chance to look at this, I have continued testing this
> successfully (and it's an improvement on what we have in-tree IMHO).

Committed with util added to WANTLIB (it now uses openpty(3)).

As with most ports, rust ports also need a 'make port-lib-depends-check'
run after every update.

Tests are quite broken right now (18 failures) because SK is set
to ./target/release/sk (seen from from the crate root). This can be
reduced to two failures if you drop SEPARATE_BUILD = Yes.
I think this is acceptable since SEPARATE_BUILD doesn't buy much in
rust ports anyway.

If for some reason you want to keep SEPARATE_BUILD, you'd need to patch
tests/common/mod.rs to adjust SK to use MODCARGO_TARGET_DIR and do a
SUBST_VAR dance.

The remaining two failures are expected since we don't have zellij in
ports (that's not going to be an easy one):

thread 'execute_interactive_child_keeps_receiving_keys_fullscreen' (560971) panicked at tests/common/zellij.rs:356:25:
Please install zellij (>= 0.44) to $PATH: CannotFindBinaryPath