NetBSD Problem Report #52081

From ryo_on@yk.rim.or.jp  Thu Mar 16 06:23:57 2017
Return-Path: <ryo_on@yk.rim.or.jp>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id AD2C17A1C0
	for <gnats-bugs@gnats.NetBSD.org>; Thu, 16 Mar 2017 06:23:57 +0000 (UTC)
Message-Id: <3vkGhK1QLTz1XLkfW@mail.SiriusCloud.jp>
Date: Thu, 16 Mar 2017 14:08:13 +0900
From: ryoon@NetBSD.org
Reply-To: ryoon@NetBSD.org
To: gnats-bugs@NetBSD.org
Subject: Build failure of pkgsrc/pkgtools/pkglint with lang/go 1.8
X-Send-Pr-Version: 3.95

>Number:         52081
>Category:       pkg
>Synopsis:       Build failure of pkgsrc/pkgtools/pkglint with lang/go 1.8
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    bsiegert
>State:          feedback
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Mar 16 06:25:00 +0000 2017
>Closed-Date:    
>Last-Modified:  Mon Mar 20 23:05:01 +0000 2017
>Originator:     Ryo ONODERA
>Release:        NetBSD 7.99.66
>Organization:

>Environment:


System: NetBSD brownie 7.99.66 NetBSD 7.99.66 (DTRACE7) #4: Wed Mar 15 16:13:02 JST 2017 ryoon@brownie:/usr/world/7.99/amd64/obj/sys/arch/amd64/compile/DTRACE7 amd64
Architecture: x86_64
Machine: amd64
>Description:
Go 1.8 from pkgsrc/lang/go removed 'go tool yacc' and pkgsrc/pkgtools/pkglint
uses it during build.

===> Building for pkglint-5.4.17
go tool: no such tool "yacc"
/usr/tmp/pkgsrc/pkgtools/pkglint/work/src/netbsd.org/pkglint/shtypes.go:7: running "go": exit status 2
go tool: no such tool "yacc"
/usr/tmp/pkgsrc/pkgtools/pkglint/work/src/netbsd.org/pkglint/licenses/licenses.go:54: running "go": exit status 2
*** Error code 1

Stop.
make[1]: stopped in /usr/pkgsrc/pkgtools/pkglint
*** Error code 1

Stop.
make: stopped in /usr/pkgsrc/pkgtools/pkglint

>How-To-Repeat:

>Fix:

Use x/tools/cmd/goyacc?

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->bsiegert
Responsible-Changed-By: bsiegert@NetBSD.org
Responsible-Changed-When: Thu, 16 Mar 2017 18:45:55 +0000
Responsible-Changed-Why:
I did the update. Sorry for the breakage.


From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/net/go-net
Date: Thu, 16 Mar 2017 19:14:19 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Thu Mar 16 19:14:19 UTC 2017

 Modified Files:
 	pkgsrc/net/go-net: Makefile PLIST buildlink3.mk distinfo

 Log Message:
 Update go-net to 20170308. Part of PR pkg/52081 (a dependency of go-tools).

 No changelog from upstream other than "git log".


 To generate a diff of this commit:
 cvs rdiff -u -r1.12 -r1.13 pkgsrc/net/go-net/Makefile
 cvs rdiff -u -r1.4 -r1.5 pkgsrc/net/go-net/PLIST \
     pkgsrc/net/go-net/buildlink3.mk
 cvs rdiff -u -r1.5 -r1.6 pkgsrc/net/go-net/distinfo

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/security/go-crypto
Date: Thu, 16 Mar 2017 19:27:36 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Thu Mar 16 19:27:36 UTC 2017

 Modified Files:
 	pkgsrc/security/go-crypto: Makefile PLIST buildlink3.mk distinfo

 Log Message:
 Update go-crypto to 20170317, part of PR pkg/52081.

 No changelog from upstream but there is a new acme package for Let's
 Encrypt certificates.


 To generate a diff of this commit:
 cvs rdiff -u -r1.6 -r1.7 pkgsrc/security/go-crypto/Makefile
 cvs rdiff -u -r1.1 -r1.2 pkgsrc/security/go-crypto/PLIST \
     pkgsrc/security/go-crypto/buildlink3.mk \
     pkgsrc/security/go-crypto/distinfo

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/devel/go-tools
Date: Thu, 16 Mar 2017 19:33:50 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Thu Mar 16 19:33:50 UTC 2017

 Modified Files:
 	pkgsrc/devel/go-tools: Makefile PLIST buildlink3.mk distinfo

 Log Message:
 Update go-tools to 1.8 release as part of PR pkg/52081.

 No changelog from upstream. There is now a "goyacc" tool to replace
 "go tool yacc" from the main repo.


 To generate a diff of this commit:
 cvs rdiff -u -r1.11 -r1.12 pkgsrc/devel/go-tools/Makefile
 cvs rdiff -u -r1.2 -r1.3 pkgsrc/devel/go-tools/PLIST
 cvs rdiff -u -r1.1 -r1.2 pkgsrc/devel/go-tools/buildlink3.mk
 cvs rdiff -u -r1.3 -r1.4 pkgsrc/devel/go-tools/distinfo

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/pkgtools/pkglint
Date: Thu, 16 Mar 2017 20:03:22 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Thu Mar 16 20:03:22 UTC 2017

 Modified Files:
 	pkgsrc/pkgtools/pkglint: Makefile
 	pkgsrc/pkgtools/pkglint/files: shtypes.go
 	pkgsrc/pkgtools/pkglint/files/licenses: licenses.go

 Log Message:
 pkglint 5.4.18.

 Changes:
  - go tool yacc -> goyacc to fix building with Go 1.8 (PR pkg/52081).


 To generate a diff of this commit:
 cvs rdiff -u -r1.510 -r1.511 pkgsrc/pkgtools/pkglint/Makefile
 cvs rdiff -u -r1.4 -r1.5 pkgsrc/pkgtools/pkglint/files/shtypes.go
 cvs rdiff -u -r1.1 -r1.2 pkgsrc/pkgtools/pkglint/files/licenses/licenses.go

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->feedback
State-Changed-By: bsiegert@NetBSD.org
State-Changed-When: Thu, 16 Mar 2017 20:10:17 +0000
State-Changed-Why:
Please try version 5.4.18.


From: Leonardo Taccari <leot@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/52081 (Build failure of pkgsrc/pkgtools/pkglint with lang/go 1.8)
Date: Fri, 17 Mar 2017 11:47:39 +0100

 Hello Benny!

 bsiegert@NetBSD.org writes:
 > Synopsis: Build failure of pkgsrc/pkgtools/pkglint with lang/go 1.8
 > =

 > State-Changed-From-To: open->feedback
 > State-Changed-By: bsiegert@NetBSD.org
 > State-Changed-When: Thu, 16 Mar 2017 20:10:17 +0000
 > State-Changed-Why:
 > Please try version 5.4.18.

 As pointed out by David via pkgsrc-users@ pkgtools/pkglint now fails due a
 circular dependency due security/go-crypto and net/go-net:

  <http://mail-index.netbsd.org/pkgsrc-users/2017/03/16/msg024530.html>

 FTR here the relevant error messages (during the building pkgtools/pkglint=
 ):

 [...]
 =3D=3D=3D> Installing dependencies for pkglint-5.4.18
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-tools>=3D1.8: NOT found
 =3D> Verifying reinstall for ../../devel/go-tools
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for tools-2946dd1.tar.gz
 =3D> Checksum RMD160 OK for tools-2946dd1.tar.gz
 =3D> Checksum SHA512 OK for tools-2946dd1.tar.gz
 =3D=3D=3D> Installing dependencies for go-tools-1.8
 =3D> Tool dependency bash-[0-9]*: found bash-4.4.012
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency google-api-go-client>=3D20150507: NOT found
 =3D> Verifying reinstall for ../../devel/google-api-go-client
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for google-api-go-client-a69f0f19d2.tar.gz
 =3D> Checksum RMD160 OK for google-api-go-client-a69f0f19d2.tar.gz
 =3D> Checksum SHA512 OK for google-api-go-client-a69f0f19d2.tar.gz
 =3D=3D=3D> Installing dependencies for google-api-go-client-20160909nb2
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency go-genproto>=3D20160816: NOT found
 =3D> Verifying reinstall for ../../misc/go-genproto
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for go-genproto-20160816-44808ed.tar.gz
 =3D> Checksum RMD160 OK for go-genproto-20160816-44808ed.tar.gz
 =3D> Checksum SHA512 OK for go-genproto-20160816-44808ed.tar.gz
 =3D=3D=3D> Installing dependencies for go-genproto-20160816nb3
 =3D> Tool dependency bsdtar-[0-9]*: found bsdtar-3.3.1
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency go-protobuf>=3D20160829: found go-protobuf-20160829n=
 b3
 =3D> Build dependency go-grpc>=3D1.0.1: NOT found
 =3D> Verifying reinstall for ../../net/go-grpc
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for go-grpc-1.0.1.tar.gz
 =3D> Checksum RMD160 OK for go-grpc-1.0.1.tar.gz
 =3D> Checksum SHA512 OK for go-grpc-1.0.1.tar.gz
 =3D=3D=3D> Installing dependencies for go-grpc-1.0.1nb3
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency go-glog>=3D20160125: found go-glog-20160125nb2
 =3D> Build dependency go-protobuf>=3D20160829: found go-protobuf-20160829n=
 b3
 =3D> Build dependency go-net>=3D20170308: NOT found
 =3D> Verifying reinstall for ../../net/go-net
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for go-net-20170308-a6577fa.tar.gz
 =3D> Checksum RMD160 OK for go-net-20170308-a6577fa.tar.gz
 =3D> Checksum SHA512 OK for go-net-20170308-a6577fa.tar.gz
 =3D=3D=3D> Installing dependencies for go-net-20170308
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency go-text>=3D20150310: found go-text-20151217nb6
 =3D> Build dependency go-crypto>=3D0.0.20170317: NOT found
 =3D> Verifying reinstall for ../../security/go-crypto
 =3D> Bootstrap dependency digest>=3D20010302: found digest-20160304
 =3D> Checksum SHA1 OK for go-crypto-0.0.20170317-728b753.tar.gz
 =3D> Checksum RMD160 OK for go-crypto-0.0.20170317-728b753.tar.gz
 =3D> Checksum SHA512 OK for go-crypto-0.0.20170317-728b753.tar.gz
 =3D=3D=3D> Installing dependencies for go-crypto-0.0.20170317
 =3D> Tool dependency checkperms>=3D1.1: found checkperms-1.11nb1
 =3D> Build dependency go-1.8*: found go-1.8
 =3D> Build dependency go-text>=3D20150310: found go-text-20151217nb6
 =3D> Build dependency cwrappers>=3D20150314: found cwrappers-20170112
 =3D=3D=3D> Overriding tools for go-crypto-0.0.20170317
 =3D=3D=3D> Extracting for go-crypto-0.0.20170317
 tar: Global extended headers posix ustar archive. Extracting as plain file=
 s. Following files might be in the wrong directory or have wrong attribute=
 s.
 =3D=3D=3D> Patching for go-crypto-0.0.20170317
 =3D=3D=3D> Creating toolchain wrappers for go-crypto-0.0.20170317
 =3D=3D=3D> Configuring for go-crypto-0.0.20170317
 =3D> Checking for portability problems in extracted files
 =3D=3D=3D> Building for go-crypto-0.0.20170317
 /tmp/pkgsrc/security/go-crypto/work/src/golang.org/x/crypto/acme/acme.go:4=
 0:2: cannot find package "golang.org/x/net/context" in any of:
         /usr/pkg/go/src/golang.org/x/net/context (from $GOROOT)
         /tmp/pkgsrc/security/go-crypto/work/src/golang.org/x/net/context (=
 from $GOPATH)
         /tmp/pkgsrc/security/go-crypto/work/.buildlink/gopkg/src/golang.or=
 g/x/net/context
 /tmp/pkgsrc/security/go-crypto/work/src/golang.org/x/crypto/acme/acme.go:4=
 1:2: cannot find package "golang.org/x/net/context/ctxhttp" in any of:
         /usr/pkg/go/src/golang.org/x/net/context/ctxhttp (from $GOROOT)
         /tmp/pkgsrc/security/go-crypto/work/src/golang.org/x/net/context/c=
 txhttp (from $GOPATH)
         /tmp/pkgsrc/security/go-crypto/work/.buildlink/gopkg/src/golang.or=
 g/x/net/context/ctxhttp
 *** Error code 1
 [...]

State-Changed-From-To: feedback->open
State-Changed-By: leot@NetBSD.org
State-Changed-When: Fri, 17 Mar 2017 11:25:23 +0000
State-Changed-Why:
Now it fails due a circulary dependency involving
security/go-crypto and net/go-net (not specific
to pkgtools/pkglint though, but it also affects it).


From: Leonardo Taccari <leot@NetBSD.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/52081 (Build failure of pkgsrc/pkgtools/pkglint with lang/go 1.8)
Date: Sun, 19 Mar 2017 01:39:23 +0100

 leot@NetBSD.org writes:
 > Now it fails due a circulary dependency involving
 > security/go-crypto and net/go-net (not specific
 > to pkgtools/pkglint though, but it also affects it).

 JFTR pkgtools/pkglint after the following <rillig>'s commits:

  cvs rdiff -u -r1.511 -r1.512 pkgsrc/pkgtools/pkglint/Makefile
  cvs rdiff -u -r0 -r1.1 pkgsrc/pkgtools/pkglint/files/shellyacc.go
  cvs rdiff -u -r0 -r1.1 pkgsrc/pkgtools/pkglint/files/licenses/licensesyacc.go

 ...builds and works again!

From: Benny Siegert <bsiegert@gmail.com>
To: gnats-bugs@NetBSD.org
Cc: Benny Siegert <bsiegert@NetBSD.org>,
 gnats-admin@netbsd.org,
 pkgsrc-bugs@netbsd.org,
 ryoon@NetBSD.org
Subject: Re: pkg/52081 (Build failure of pkgsrc/pkgtools/pkglint with lang/go
 1.8)
Date: Sun, 19 Mar 2017 19:32:48 +0100

 > JFTR pkgtools/pkglint after the following <rillig>'s commits:
 >=20
 >  cvs rdiff -u -r1.511 -r1.512 pkgsrc/pkgtools/pkglint/Makefile
 >  cvs rdiff -u -r0 -r1.1 pkgsrc/pkgtools/pkglint/files/shellyacc.go
 >  cvs rdiff -u -r0 -r1.1 =
 pkgsrc/pkgtools/pkglint/files/licenses/licensesyacc.go
 >=20
 > ...builds and works again!

 Yes, this bug is now about fixing the fallout from my other commits.

 I think that the only way out is to split one of the two package, i.e. =
 either create go-net-context or go-crypto-acme.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/lang/go
Date: Mon, 20 Mar 2017 22:33:21 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Mon Mar 20 22:33:21 UTC 2017

 Modified Files:
 	pkgsrc/lang/go: go-package.mk

 Log Message:
 Do not take the basename of GO_DIST_BASE when it is overridden. Needed for
 fixing PR pkg/52081.


 To generate a diff of this commit:
 cvs rdiff -u -r1.9 -r1.10 pkgsrc/lang/go/go-package.mk

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc/security
Date: Mon, 20 Mar 2017 22:36:49 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Mon Mar 20 22:36:49 UTC 2017

 Modified Files:
 	pkgsrc/security: Makefile
 Added Files:
 	pkgsrc/security/go-crypto-acme: DESCR Makefile PLIST buildlink3.mk

 Log Message:
 New package go-crpyto-acme, a spin-off from go-crypto to get rid of the
 circular dependency from PR pkg/52081.


 To generate a diff of this commit:
 cvs rdiff -u -r1.607 -r1.608 pkgsrc/security/Makefile
 cvs rdiff -u -r0 -r1.1 pkgsrc/security/go-crypto-acme/DESCR \
     pkgsrc/security/go-crypto-acme/Makefile \
     pkgsrc/security/go-crypto-acme/PLIST \
     pkgsrc/security/go-crypto-acme/buildlink3.mk

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->feedback
State-Changed-By: bsiegert@NetBSD.org
State-Changed-When: Mon, 20 Mar 2017 22:55:17 +0000
State-Changed-Why:
Better now?


From: "Benny Siegert" <bsiegert@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52081 CVS commit: pkgsrc
Date: Mon, 20 Mar 2017 22:53:45 +0000

 Module Name:	pkgsrc
 Committed By:	bsiegert
 Date:		Mon Mar 20 22:53:45 UTC 2017

 Modified Files:
 	pkgsrc/devel/go-tools: Makefile buildlink3.mk
 	pkgsrc/net/go-net: Makefile
 	pkgsrc/security/go-crypto: Makefile PLIST buildlink3.mk
 Added Files:
 	pkgsrc/security/go-crypto: Makefile.common

 Log Message:
 Unbreak go-crypto and go-net, they no longer depend on one another.
 Bump go-crypto and go-tools, which now depends on go-crypto-acme.

 Fixes PR pkg/52081 breakage.


 To generate a diff of this commit:
 cvs rdiff -u -r1.12 -r1.13 pkgsrc/devel/go-tools/Makefile
 cvs rdiff -u -r1.2 -r1.3 pkgsrc/devel/go-tools/buildlink3.mk
 cvs rdiff -u -r1.14 -r1.15 pkgsrc/net/go-net/Makefile
 cvs rdiff -u -r1.8 -r1.9 pkgsrc/security/go-crypto/Makefile
 cvs rdiff -u -r0 -r1.1 pkgsrc/security/go-crypto/Makefile.common
 cvs rdiff -u -r1.2 -r1.3 pkgsrc/security/go-crypto/PLIST \
     pkgsrc/security/go-crypto/buildlink3.mk

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2014 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.