Thanks for the review, comments below:

On Mon, Feb 20, 2017 at 07:52:16PM +0000, Stuart Henderson wrote:
> On 2017/02/20 19:37, Anton Lindqvist wrote:
> > Update to newest version 0.8.2.
> > 
> > On Sat, Feb 18, 2017 at 09:05:24AM +0100, Anton Lindqvist wrote:
> > > The yank(1) utility reads input from stdin and displays a selection
> > > interface that allows a field to be selected and copied to the
> > > clipboard.
> > > 
> > > I'm the creator and maintainer of yank(1).
> > > 
> > > https://github.com/mptre/yank
> 
> - HOMEPAGE isn't needed, that's the default with GH_* anyway.

Fixed

> - the @conflict is done on the package name, so it should be EMBOSS-*
> not emboss-*.

Fixed

> - we're trying to keep track of things in ports using pledge, so please
> add a "# uses pledge()" comment near the WANTLIB

Thanks, didn't know. Maybe this convention should mentioned in
infrastructure/templates/Makefile.template.

> rest is ok ports-wise, though I'm not a fan of hard-coded terminal
> escape sequences.

Agree, I will migrate to curses at some point in the future.

Attachment: yank.tar.gz
Description: application/tar-gz

Reply via email to