Peter Hessler writes:
> I've wanted a local cache of the RFCs for a long time, so I quickly
> wrote up a simple port to keep them local, and a trivial pager helper
> script.

I like this idea.

PKGNAME defaults to ${DISTNAME}, so you can remove that line.

Might as well use https consistently for HOMEPAGE/MASTER_SITES/MASTER_SITES0.

The commented MASTER_SITES line should be removed (or the ftp server
should be listed after the https server in MASTER_SITES, but I don't see
much point in that since they're the same source).

rfc-index.txt and bcp-index.txt seem like unstable files that will
become unfetchable (change hashes) often. They already list RFCs that
won't exist in the package. Not sure it makes sense to keep these around.

The mirror has RFCs1001-1500.tar.gz and RFCs7001-7500.tar.gz, but these
aren't included in the port.

You can use sed -i instead of perl -i -pe.

When I try installing:

/usr/ports/pobj/rfc-0.0.7000/bin/install -c  -m 644 
/usr/ports/pobj/rfc-0.0.7000/rfc*.txt 
/usr/ports/pobj/rfc-0.0.7000/fake-amd64/usr/local/share/doc/rfc/
/bin/sh: /usr/ports/pobj/rfc-0.0.7000/bin/install: Argument list too long

Quick hack to fix it: replace
        ${INSTALL_DATA} ${WRKDIST}/rfc*.txt ${INSTALL_LOC}/
with
.for i in 1 2 3 4 5 6 7 8 9
        ${INSTALL_DATA} ${WRKDIST}/rfc$i*.txt ${INSTALL_LOC}/
.endfor

-- 
Anthony J. Bentley

Reply via email to