Re: [PATCH] gnu: go: Update to 1.8.1

[Leo Famulari]

On Mon, Apr 17, 2017 at 03:37:49PM +0200, Petter wrote:
> Hi Sergei,
> 
> I definitely think your fix deserves recognition. Without it I would
> certainly have been unable to build 1.8(.1)? this month. While editing the
> recipe is trivial mechanical wise, figuring out what to change was not :)
> 
> Sorry to hear about your issues; I'm on x86_64 and Linux as well though...
> 
> > $ /gnu/store/b4gflqj64yvksq7959r6m22mf9lzdy69-go-1.8.1/bin/go version
> > go version go1.8.1 linux/amd64
> 
> Hope you'll be able to work this out!
> 
> Thanks for fixing Go 1.8 :)

Yes, thanks to both of you for picking this up :)

> @@ -300,8 +301,8 @@ sequential processes (CSP) concurrent programming 
> features added.")
>                     (("/etc/services") (string-append net-base 
> "/etc/services")))
>                   (substitute* "time/zoneinfo_unix.go"
>                     (("/usr/share/zoneinfo/") tzdata-path))
> -                 (substitute* (find-files "cmd" "asm.c")
> -                   (("/lib/ld-linux.*\\.so\\.[0-9]") loader))
> +                 (substitute* (find-files "cmd" "\\.go")
> +                   (("/lib(64)?/ld-linux.*\\.so\\.[0-9]") loader))

The substitution that is removed here would still patch some files.
Shouldn't we adjust the old substitution to also match '/lib64', as well
as add the new substitution?

I've attached a patch for that.

0001-gnu-go-Update-to-1.8.1.patch
Description: Text document

signature.asc
Description: PGP signature