On 2015/03/24 15:13, Kent R. Spillner wrote:
> On Tue, Mar 24, 2015 at 02:34:30PM -0500, Kent R. Spillner wrote:
> > Attached is an updated version based on feedback from czarkoff@:
> >
> > - Enable tests
> > - Prefer ${...} over $(...)
> >
> > Ok?
>
> Once more, this time with a slightly improved do-test target.
>
> Tests pass and works fine on amd64 with the www/websocketd port I also
> sent.
Your do-install is amd64 only, and your PLIST uses MACHINE_ARCH.
We probably want something like this:
do-install:
${INSTALL_DATA_DIR} ${PREFIX}/go/${GO_PKG}/golang.org/x/net
${INSTALL_DATA} -p \
${WRKDIST}/${GO_PKG}/golang.org/x/net/websocket.a \
${PREFIX}/go/${GO_PKG}/golang.org/x/net/
${INSTALL_DATA_DIR} ${PREFIX}/go/src/golang.org/x/net/websocket
${INSTALL_DATA} -p \
${WRKDIST}/src/golang.org/x/net/websocket/websocket.go \
${PREFIX}/go/src/golang.org/x/net/websocket/
GO_PKG = pkg/openbsd_${MACHINE_ARCH:S/i386/386/}
SUBST_VARS += GO_PKG
(plus regen PLIST).
