Re: netmask package at wip

["Sijmen J. Mulder" <ik%sjmulder.nl@localhost>]

Hi Fran,

Francisco J. Tsao Santín <tsao%disroot.org@localhost>:
> I've just added the netmask [1] package to the wip repository. It is my
> first package, so I would be very grateful if someone reviews it :-)

Cool! Some small notes.

1. From `pkglint -Wall`:

  NOTE: Makefile:7: Trailing whitespace.
  WARN: Makefile:22--23: Please switch to "set -e" mode before using a
   semicolon (after "cd ${WRKSRC}") to separate commands.

(I'd write `cd ${WRKSRC} && ./autogen` instead.)

2. The first section of the Makefile can be simplified:

  DISTNAME=       netmask-2.4.4
  CATEGORIES=     sysutils
  MASTER_SITES=   ${MASTER_SITE_GITHUB:=tlby/}
  GITHUB_TAG=     v${PKGVERSION_NOREV}

pkgsrc will correctly extract the package name and version number from
DISTNAME. By using PKGVERSION_NOREV you can avoid repeating the version
number.

You can then also remove the WRKSRC line, it's inferred.

3. Some small nits:

 - Makefile: Whitespace before and after pre-configure:.
 - DESCR: 'This is a handy tool for...' can just be 'Tool for...', same
    with COMMENT.
 - DESCR: I'd remove all paragraphs but the first.

Sijmen