Laurent Cheylus wrote (2023-11-19 11:37 CET):
> Hi Stefan,
> 
> > I just found this on ports@ after I started to create a port on my
> > own.
> > Attached is an updated version (2.6.1) which also uses DIST_TUPLE. I
> > also broke RUN_DEPENDS into one dependency per line. It's just easier
> > to read.
> > 
> > Without thumbnails, ytfzf works fine. I tried the imv, mpv and chafa
> > thumbnail
> > generation. They don't work because fzf is being killed by pledge:
> > 
> > $ ytfzf -t -T {chafa,mpv,imv}
> > fzf[85237]: pledge "wpath", syscall 5
> > 
> > This happens, because fzf is trying to open the prefiew image in /tmp
> > 
> >   46440 fzf      NAMI  "/tmp/fzf-preview-1776267355"
> >   46440 fzf      PLDG  open, "wpath", errno 1 Operation not permitted
> > 
> > As you're also maintaining fzf, maybe you want to look at this?
> 
> I just sent on ports@ an update for sysutils/fzf v0.44.1 and I added tmppath 
> pledge permission to allow temporary files in /tmp.
> 
> With this fix, your update for multimedia/ytfzf v2.6.1 works as expected. 
> Build and tests OK on current/amd64.
> 
> Thanks for your update and tests with fzf.

Excellent. I've committed the fzf update.

For ytfzf, OK for import.

Reply via email to