On 2023/05/04 12:05, Laurent Cheylus wrote:
> On 05/04/2023 11:29, Stuart Henderson wrote:
> > On 2023/05/04 09:52, Laurent Cheylus wrote:
> > > proposal to import sysutils/lsd (version 0.23.1)
> > 
> > Looks reasonable
> 
> Thanks
> 
> > post-install:
> >         sed -i '1,8d' ${WRKSRC}/doc/lsd.md
> >         lowdown -s -tman -m section=1 -mtitle=lsd -mvolume='' \
> >                 ${WRKSRC}/doc/lsd.md > ${PREFIX}/man/man1/lsd.1
> > 
> > I suggest not doing that sed in place, but write to a different file
> > instead. That way, if post-install is interrupted for any reason,
> > it won't chop off extra lines.
> 
> OK, I modified my Makefile not to modify doc/lsd.md in place with sed
> (temporary output file in WRKBUILD directory).

this is OK sthen@ to import. (I don't think I'll be using it though,
I don't find that unicode "missing character" boxes enhance my ls
experience ;)

Reply via email to