Re: please review starplot, pixel-saga-st, cairo-dingbat-ttf

Thomas Klausner <[email protected]> Thu, 27 Feb 2014 15:52:44 +0100
Newsgroups gmane.os.netbsd.devel.pkgsrc.wip.review
Message-ID <[email protected]>
On Fri, Feb 14, 2014 at 02:30:42PM -0600, Tim Larson wrote:
> Another call for review and import by someone with access to do so. (And thanks to Niclas and Aleksey for prior feedback on starplot.) Anybody out there?
> 
> Have had no comments on the font packages. Since those are just data installs, they should be straightforward, but any recommendations for improvements are welcome.

cairo-dingbat is missing a ${DESTDIR}:

=> Generating pre-install file lists
=> Creating installation directories
/usr/bin/install -d -o pbulk -g builder -m 755 /scratch/wip/cairo-dingbat-ttf/work/.destdir/usr/pkg/lib/X11/fonts/TTF
/usr/bin/install -c -o pbulk -g builder -m 644 /scratch/wip/cairo-dingbat-ttf/work/cairo/Cairo*.ttf /scratch/wip/cairo-dingbat-ttf/work/.destdir/usr/pkg/lib/X11/fonts/TTF
/usr/bin/install -d -o pbulk -g builder -m 755 /usr/pkg/share/doc/cairo-dingbat-ttf
install: /usr/pkg/share/doc: mkdir: Permission denied
*** Error code 1


I've imported pixel-sagas-st as pixel-sagas-startrek, after removing
Okuda because I couldn't fetch the distfile.

I've also imported starplot after putting the MESSAGE file in the
standard format.

Thanks,
 Thomas


------------------------------------------------------------------------------
Flow-based real-time traffic analytics software. Cisco certified tool.
Monitor traffic, SLAs, QoS, Medianet, WAAS etc. with NetFlow Analyzer
Customize your own dashboards, set traffic alerts and generate reports.
Network behavioral analysis & security monitoring. All-in-one tool.
http://pubads.g.doubleclick.net/gampad/clk?id=126839071&iu=/4140/ostg.clktrk