On Thu May 04, 2023 at 11:31:24AM +0100, Stuart Henderson wrote:
> 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 ;)
> 

Imported, thanks Laurent

Reply via email to