Re: [gentoo-dev] new eclass: golang.eclass for compiling go packages

[William Hubbs]

On Thu, Jun 11, 2015 at 04:39:05PM -0700, Zac Medico wrote:
> On 06/11/2015 11:43 AM, William Hubbs wrote:
> > On Thu, Jun 11, 2015 at 08:58:37AM -0700, Andrew Udvare wrote:
> >>
> >>> On 2015-06-11, at 08:38, William Hubbs  wrote:
> >>>
> >>> this eclass is meant to provide a common src_compile function for
> >>> packages written in the Go programming language.
> >>>
> >>> Let me know what you think.
> >>>
> >>
> >> I am wondering about bug 503324 and the issue of needing to create a 
> >> GOROOT with everything except the package to be compiled. Is your way 
> >> solving this issue?
> > 
> > I looked at an example in the tree (specifically go-fuse) and the
> > src_compile I wrote is the same as the one there.
> > 
> > In the testing I've done here, GOPATH must be set or "go get" doesn't
> > work. The newest code for packages will always be stored in the first
> > directory listed in GOPATH, not in GOROOT.
> > 
> > Given how GOPATH works in relation to GOROOT, I don't know yet why the
> > ebuilds in the tree are copying goroot; I'm not quite sure what they are
> > working around yet.
> 
> For example, dev-go/go-net needs this workaround. Otherwise, if
> dev-go/go-net is already installed, it triggers a sandbox violation as
> follows:
> 
> >>> Compiling source in
> /var/tmp/portage/dev-go/go-net-1.4.2_p20150604/work/src/golang.org/x/net ...
> WORK=/var/tmp/portage/dev-go/go-net-1.4.2_p20150604/temp/go-build862715022
> golang.org/x/net/html/charset
> mkdir -p $WORK/golang.org/x/net/html/charset/_obj/
> mkdir -p $WORK/golang.org/x/net/html/
> cd /usr/lib/go/src/golang.org/x/net/html/charset
> /usr/lib/go/pkg/tool/linux_amd64/6g -o
> $WORK/golang.org/x/net/html/charset.a -trimpath $WORK -p
> golang.org/x/net/html/charset -complete -D
> _/usr/lib/go/src/golang.org/x/net/html/charset -I $WORK -pack
> ./charset.go ./table.go
> mkdir -p /usr/lib/go/pkg/linux_amd64/golang.org/x/net/html/
> cp $WORK/golang.org/x/net/html/charset.a
> /usr/lib/go/pkg/linux_amd64/golang.org/x/net/html/charset.a
> go install golang.org/x/net/html/charset: open
> /usr/lib/go/pkg/linux_amd64/golang.org/x/net/html/charset.a: permission
> denied

Part of the problem is that golang.org/x/foo repositories try to install
in $GOROOT directly when you try to use "go install"; no other
repositories do this that I'm aware of.

I recommend using "go build" in src_compile to build the things you need
to install, then using manual methods to install the things you want to
install where they need to go.

Also, you can use "go env GOROOT" to get the value of goroot.

Now that i have golang-vcs.eclass in the tree,, I will take a look at
the go-net live ebuild. I will still take a look at reworking this
eclass. When I post it again it will be called golang-build.eclass and
be at the end of this thread.

Thanks,

William


signature.asc
Description: Digital signature

[gentoo-dev] Last rutes: dev-util/ext4_utils, dev-util/libsparse

[Jauhien Piatlicki]

# Jauhien Piatlicki  (16 Jun 2015)
# Upstream does not provide these as separated packages.
# Nobody interested in maintaining them in Gentoo.
# A number of open bugs.
# Masked for removal in 30 days.
dev-util/ext4_utils
dev-util/libsparse



signature.asc
Description: OpenPGP digital signature

Re: [gentoo-dev] new eclass: go-live.eclass for handling go live ebuilds

[William Hubbs]

There have been no more comments on this eclass.

I did, however, find some typos that I fixed. I will attach the patch
with the fixes here then commit the eclass. Without these fixes the
eclass generates "command not found" errors.

The fixes involve s/umask_push/eumask_push/ and s/umask_pop/eumask_pop/.

William

--- a/golang-vcs.eclass	2015-06-16 15:00:31.995528578 -0500
+++ b/golang-vcs.eclass	2015-06-16 14:47:00.484815701 -0500
@@ -72,7 +72,7 @@
 	local distdir=${PORTAGE_ACTUAL_DISTDIR:-${DISTDIR}}
 	: ${EGO_STORE_DIR:=${distdir}/go-src}
 
-	[[ -n ${EVCS_UMASK} ]] && umask_push $EVCS_UMASK
+	[[ -n ${EVCS_UMASK} ]] && eumask_push $EVCS_UMASK
 
 	if [[ ! -d ${EGO_STORE_DIR} ]]; then
 		(
@@ -84,7 +84,7 @@
 	addwrite "${EGO_STORE_DIR}"
 	export GOPATH="${EGO_STORE_DIR}"
 
-	[[ -n ${EVCS_UMASK} ]] && umask_pop
+	[[ -n ${EVCS_UMASK} ]] && eumask_pop
 	mkdir -p "${S}" ||
 		die "${ECLASS}: unable to create ${S}"
 	return 0
@@ -105,7 +105,7 @@
 		return 0
 	fi
 
-	[[ -n ${EVCS_UMASK} ]] && umask_push ${EVCS_UMASK}
+	[[ -n ${EVCS_UMASK} ]] && eumask_push ${EVCS_UMASK}
 
 	set -- go get -d -t -u -v -x "${EGO_PN}"
 	echo "$@"
@@ -117,7 +117,7 @@
 	# I am hoping this will be fixed so "go get -d" is successful if
 	# downloading the top level repository is successful.
 
-	[[ -n ${EVCS_UMASK} ]] && umask_pop
+	[[ -n ${EVCS_UMASK} ]] && eumask_pop
 	export GOPATH="${S}:${EGO_STORE_DIR}"
 	return 0
 }


signature.asc
Description: Digital signature

Re: [gentoo-dev] Packages up for grabs

[Jauhien Piatlicki]

Hi Daniel,

so will you take the maintainership? I will reassign bugs to you then.

On 06/06/2015 09:23 PM, Daniel "zlg" Campbell wrote:
> On 06/06/2015 09:02 AM, Jauhien Piatlicki wrote:
>> Hi,
> 
>> the list of packages that I'm no longer interested in:
> 
>> media-sound/apulse -- needs version bump dev-util/android-ndk --
>> needs version bump
> 
>> I will remove myself from maintainers of these packages and, if
>> there is no other maintainers reassign bugs to maintainer-needed.
> 
>> dev-util/ext4_utils dev-util/libsparse
> 
>> Adding of these two packages to the tree were, probably, not very
>> good idea, so if nobody wants to maintain them, it makes sense to
>> remove them.
> 
>> -- Jauhien
> 
> 
> I can take apulse since I use Skype, which it was originally written
> for use with.
> 



signature.asc
Description: OpenPGP digital signature