From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id D0926138334 for ; Thu, 26 Sep 2019 19:49:58 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 2D6D6E0A0B; Thu, 26 Sep 2019 19:49:54 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id CD2C2E0844 for ; Thu, 26 Sep 2019 19:49:53 +0000 (UTC) Received: from pomiot (c134-66.icpnet.pl [85.221.134.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: mgorny) by smtp.gentoo.org (Postfix) with ESMTPSA id B146F34B5F6; Thu, 26 Sep 2019 19:49:51 +0000 (UTC) Message-ID: <42d4b7dc97d86cf748c9d18f97e21eb5f68b1dc1.camel@gentoo.org> Subject: Re: [gentoo-dev] [PATCH v2 1/1] go-module.eclass: new eclass for go modules From: =?UTF-8?Q?Micha=C5=82_G=C3=B3rny?= To: gentoo-dev@lists.gentoo.org Cc: William Hubbs Date: Thu, 26 Sep 2019 21:49:47 +0200 In-Reply-To: <20190926002443.5117-2-williamh@gentoo.org> References: <20190926002443.5117-1-williamh@gentoo.org> <20190926002443.5117-2-williamh@gentoo.org> Organization: Gentoo Content-Type: multipart/signed; micalg="pgp-sha512"; protocol="application/pgp-signature"; boundary="=-U1Y9sX6Bx92cr5Cfmd/z" User-Agent: Evolution 3.32.4 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply MIME-Version: 1.0 X-Archives-Salt: a7019cf8-18a2-4b64-8c98-96ef71d76732 X-Archives-Hash: 3024d1ba9733f44e33fb85bbf208c3ce --=-U1Y9sX6Bx92cr5Cfmd/z Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, 2019-09-25 at 19:24 -0500, William Hubbs wrote: > This eclass includes the basic settings and src_unpack/pkg_postinst > functions for Go modules. >=20 > Signed-off-by: William Hubbs > --- > eclass/go-module.eclass | 176 ++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 176 insertions(+) > create mode 100644 eclass/go-module.eclass >=20 > diff --git a/eclass/go-module.eclass b/eclass/go-module.eclass > new file mode 100644 > index 00000000000..f920457f3ab > --- /dev/null > +++ b/eclass/go-module.eclass > @@ -0,0 +1,176 @@ > +# Copyright 2019 Gentoo authors > +# Distributed under the terms of the GNU General Public License v2 > + > +# @ECLASS: go-module.eclass > +# @MAINTAINER: > +# William Hubbs > +# @SUPPORTED_EAPIS: 7 > +# @BLURB: basic eclass for building software written as go modules > +# @DESCRIPTION: > +# This eclass provides basic settings and functions > +# needed by all software written in the go programming language that use= s > +# go modules. > +# > +# You will know the software you are packaging uses modules because > +# it will have files named go.sum and go.mod in its top-level source > +# directory. If it does not have these files, use the golang-* eclasses. > +# > +# If it has these files and a directory named vendor in its top-level > +# source directory, you only need to inherit the eclass since upstream > +# is vendoring the dependencies. > +# > +# If it does not have a vendor directory, you should use the EGO_VENDOR > +# variable and the go-module_vendor_uris function as shown in the > +# example below to handle dependencies. > +# > +# Since Go programs are statically linked, it is important that your ebu= ild's > +# LICENSE=3D setting includes the licenses of all statically linked > +# dependencies. So please make sure it is accurate. > +# > +# @EXAMPLE: > +# > +# @CODE > +# > +# inherit go-module > +# > +# EGO_VENDOR=3D( > +# "github.com/xenolf/lego 6cac0ea7d8b28c889f709ec7fa92e92b82f490dd" > +# "golang.org/x/crypto 453249f01cfeb54c3d549ddb75ff152ca243f9d8 github.c= om/golang/crypto" > +# ) > +# > +# SRC_URI=3D"https://github.com/example/${PN}/archive/v${PV}.tar.gz -> $= {P}.tar.gz > +# $(go-module_vendor_uris)" > +# > +# @CODE > + > +case ${EAPI:-0} in > + 7) ;; > + *) die "${ECLASS} API in EAPI ${EAPI} not yet established." > +esac > + > +if [[ -z ${_GO_MODULE} ]]; then > + > +_GO_MODULE=3D1 > + > +BDEPEND=3D">=3Ddev-lang/go-1.12" > + > +# Force go to build in module mode. > +# In this mode the GOPATH environment variable is ignored. > +# this will become the default in the future. > +export GO111MODULE=3Don > + > +# The following go flags should be used for all builds. > +# -mod=3Dvendor stopps downloading of dependencies from the internet. > +# -v prints the names of packages as they are compiled > +# -x prints commands as they are executed > +export GOFLAGS=3D"-mod=3Dvendor -v -x" > + > +# Do not complain about CFLAGS etc since go projects do not use them. > +QA_FLAGS_IGNORED=3D'.*' > + > +# Go packages should not be stripped with strip(1). > +RESTRICT=3D"strip" > + > +EXPORT_FUNCTIONS src_unpack pkg_postinst > + > +# @ECLASS-VARIABLE: EGO_VENDOR > +# @DESCRIPTION: > +# This variable contains a list of vendored packages. > +# The items of this array are strings that contain the > +# import path and the git commit hash for a vendored package. > +# If the import path does not start with github.com, the third argument > +# can be used to point to a github repository. > + > +# @FUNCTION: go-module_vendor_uris > +# @DESCRIPTION: > +# Convert the information in EGO_VENDOR to a format suitable for > +# SRC_URI. > +# A call to this function should be added to SRC_URI in your ebuild if > +# the upstream package does not include vendored dependencies. > +go-module_vendor_uris() { > + local hash import line repo _ > + for line in "${EGO_VENDOR[@]}"; do > + read -r import hash repo _ <<< "${line}" > + if [[ "$_" ]]; then Nit: you don't need to quote stuff inside double brackets (except for arg to '=3D=3D', if you don't want wildcards to work there). > + eerror "Trailing information in EGO_VENDOR in ${P}.ebuild" > + eerror "${line}" > + die "Invalid EGO_VENDOR format" > + fi > + : "${repo:=3D${import}}" > + echo "https://${repo}/archive/${hash}.tar.gz -> ${repo//\//-}-${hash}.= tar.gz" > + done > +} > + > +# @FUNCTION: go-module_src_unpack > +# @DESCRIPTION: > +# Extract all archives in ${a} which are not nentioned in ${EGO_VENDOR} > +# to their usual locations then extract all archives mentioned in > +# ${EGO_VENDOR} to ${S}/vendor. > +go-module_src_unpack() { > + debug-print-function ${FUNCNAME} "$@" > + local f hash import line repo tarball vendor_tarballs _ > + vendor_tarballs=3D Why not make it an array? > + for line in "${EGO_VENDOR[@]}"; do > + read -r import hash repo _ <<< "${line}" > + if [[ "$_" ]]; then > + eerror "Trailing information in EGO_VENDOR in ${P}.ebuild" > + eerror "${line}" > + die "Invalid EGO_VENDOR format" > + fi > + : "${repo:=3D${import}}" > + tarball=3D${repo//\//-}-${hash}.tar.gz > + vendor_tarballs+=3D" ${tarball}" > + done > + for f in $A; do > + [[ -n "${vendor_tarballs}" ]] && has $f $vendor_tarballs && continue > + unpack $f > + done > + > + [[ -z "${vendor_tarballs}" ]] && return > + ebegin "Vendoring dependencies" > + for line in "${EGO_VENDOR[@]}"; do > + read -r import hash repo _ <<< "${line}" > + : "${repo:=3D${import}}" > + tarball=3D${repo//\//-}-${hash}.tar.gz > + einfo "Vendoring ${import} ${tarball}" > + rm -fr "${S}/vendor/${import}" || die > + mkdir -p "${S}/vendor/${import}" || die > + tar -C "${S}/vendor/${import}" -x --strip-components 1 \ > + -f "${DISTDIR}/${tarball}" || die > + done > + eend You have output between ebegin and eend, so eend will go on the wrong line. So just replace it with einfo, I guess. You may use ebegin/eend for individual tarballs, and I suppose add some indentation to it. > +} > + > +# @FUNCTION: go-module_live_vendor > +# @DESCRIPTION: > +# This function is used in live ebuilds to vendor the dependencies when > +# upstream doesn't vendor them. > +go-module_live_vendor() { > + debug-print-function ${FUNCNAME} "$@" > + > + has live ${PROPERTIES} || > + die "${FUNCNAME} only allowed in live ebuilds" > + [[ "${EBUILD_PHASE}" =3D=3D unpack ]] || > + die "${FUNCNAME} only allowed in src_unpack" > + [[ -d "${S}"/vendor ]] || > + die "${FUNCNAME} only allowed when upstream isn't vendoring" > + > + cd "${S}" || die Didn't I suggest pushd/popd? > + go mod vendor || die > +} > + > +# @FUNCTION: go-module_pkg_postinst > +# @DESCRIPTION: > +# Display a warning about security updates for Go programs. > +go-module_pkg_postinst() { > + debug-print-function ${FUNCNAME} "$@" > + ewarn "${PN} is written in the Go programming language." > + ewarn "Since this language is statically linked, security" > + ewarn "updates will be handled in individual packages and will be" > + ewarn "difficult for us to track as a distribution." > + ewarn "For this reason, please update any go packages asap when new" > + ewarn "versions enter the tree or go stable if you are running the" > + ewarn "stable tree." > +} > + > +fi --=20 Best regards, Micha=C5=82 G=C3=B3rny --=-U1Y9sX6Bx92cr5Cfmd/z Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- iQGTBAABCgB9FiEEx2qEUJQJjSjMiybFY5ra4jKeJA4FAl2NFlxfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEM3 NkE4NDUwOTQwOThEMjhDQzhCMjZDNTYzOUFEQUUyMzI5RTI0MEUACgkQY5ra4jKe JA7+Xwf+Pmi5kPGbmXSLHNFGy2WE7+9JTLWL1PL4SKQ1hEOSHM/ienK5DPR5NxfF T2wTvzg8t3jy9Sm43GyUMBAhnYffd8LUo53JEmOVQwqJcI1Sl0fKd24xsqbij51t Beab1gKybU6RdqmXUOq1RFZBnxCBz+h+foPAbPNw7CSzWb6OQ2hDj/TUp16RX80E 00NBa/ZoLz4FIS37wCUnQVcSTFi4rUQGTg5ium1Lm3A9HQv8BmMmzSBvJC4CXffr PoW9TQshmhNlR2opjYHGfoIqKr9ZMsJbcME5LhhnGzIFjJH8x92o+7m/NmfiCp6y tDOUVPAJikPDDMwWLukvsSevoUYM0Q== =j6fB -----END PGP SIGNATURE----- --=-U1Y9sX6Bx92cr5Cfmd/z--