On Tue, Aug 04, 2015 at 12:17:50PM -0400, Mike Gilbert wrote:
> The documentation says you are extracting to ${S}, but the function
> actually extracts to ${WORKDIR}/${PN}.
 
 s/PN/P/
> I would get rid of the useless "destdir" variable and replace all
> usages with "${S}". Or update the docs.
 
 I got rid of the f and destdir variables and cleaned up the docs; here
 is the latest; let me know what you think.

 William

# Copyright 1999-2015 Gentoo Foundation
# Distributed under the terms of the GNU General Public License v2
# $Header: $

# @ECLASS: golang-vcs-snapshot.eclass
# @MAINTAINER:
# William Hubbs <[email protected]>
# @BLURB: support eclass for unpacking VCS snapshot tarballs for
# software written in the Go programming language
# @DESCRIPTION:
# This eclass provides a convenience src_unpack() which unpacks the
# first tarball mentioned in SRC_URI to its appropriate location in
# ${WORKDIR}/${P}, treating ${WORKDIR}/${P} as a go workspace.
#
# The location where the tarball is extracted is defined as
# ${WORKDIR}/${P}/src/${EGO_PN}.
#
# The typical use case is VCS snapshots coming from github, bitbucket
# and similar services.
#
# Please note that this eclass currently handles only tarballs
# (.tar.gz), but support for more formats may be added in the future.
#
# @EXAMPLE:
#
# @CODE
# EGO_PN=github.com/user/package
# inherit golang-vcs-snapshot
#
# SRC_URI="http://github.com/example/${PN}/tarball/v${PV} -> ${P}.tar.gz"
# @CODE
#
# The above example will extract the tarball to
# ${WORKDIR}/${P}/src/github.com/user/package

inherit golang-base

case ${EAPI:-0} in
        5) ;;
        *) die "${ECLASS} API in EAPI ${EAPI} not yet established."
esac

EXPORT_FUNCTIONS src_unpack

# @FUNCTION: golang-vcs-snapshot_src_unpack
# @DESCRIPTION:
# Extract the first archive from ${A} to the appropriate location for GOPATH.
golang-vcs-snapshot_src_unpack() {
        local x
        ego_pn_check
        set -- ${A}
        x="$1"
                mkdir -p "${WORKDIR}/${P}/src/${EGO_PN%/*}"
        tar -C "${WORKDIR}/${P}/src/${EGO_PN%/*}" -x --strip-components 1 \
                -f "${DISTDIR}/${x}" || die
}

Attachment: signature.asc
Description: Digital signature

Reply via email to