unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: go: Update to 1.8.1
@ 2017-04-17 12:16 Petter
  2017-04-17 12:52 ` Tobias Geerinckx-Rice
  2017-04-17 12:54 ` Sergei Trofimovich
  0 siblings, 2 replies; 9+ messages in thread
From: Petter @ 2017-04-17 12:16 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 866 bytes --]

Hi,

With this patch I'm not sure how to properly handle Copyright. I've 
taken a guess and prepared a
patch anyway.

Here's the situation.

Updating to 1.8 was not trivial. A fix has been posted in a comment by 
Sergei Trofimovich on
github[1]. It's a proper Guix patch, updating Go to 1.8, with the fix 
and the trivial update stuff,
but as far as I can see not actually submitted to Guix.

[1] https://github.com/golang/go/issues/19132#issuecomment-285897612

After this Go 1.8.1 has been released. The attached patch has the 
trivial update stuff for 1.8.1
and the fix by Sergei. I added Sergei to the Copyright section, but I'm 
not sure if this is
correct, and if this is the way to do it. Also, I didn't find his e-mail 
address, so I just added
his github page, in ()...

A little messy, please let me know if I should do something differently.

Best,
Petter

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-go-Update-to-1.8.1.patch --]
[-- Type: text/x-diff; name=0001-gnu-go-Update-to-1.8.1.patch, Size: 2692 bytes --]

From 9b24372fc95ac32e774bbaa1c5219b771fcd1cda Mon Sep 17 00:00:00 2001
From: Petter <petter@mykolab.ch>
Date: Mon, 17 Apr 2017 13:36:16 +0200
Subject: [PATCH] gnu: go: Update to 1.8.1.

* gnu/packages/golang.scm (go): Update to 1.8.1.
---
 gnu/packages/golang.scm | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 00630ce06..1864f780a 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -3,8 +3,9 @@
 ;;; Copyright © 2016 Matthew Jordan <matthewjordandevops@yandex.com>
 ;;; Copyright © 2016 Andy Wingo <wingo@igalia.com>
 ;;; Copyright © 2016 Ludovic Courtès <ludo@gnu.org>
-;;; Copyright © 2016 Petter <petter@mykolab.ch>
+;;; Copyright © 2016, 2017 Petter <petter@mykolab.ch>
 ;;; Copyright © 2016, 2017 Leo Famulari <leo@famulari.name>
+;;; Copyright © 2017 Sergei Trofimovich (https://github.com/trofi)
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -197,11 +198,11 @@ garbage collection, various safety features and in the style of communicating
 sequential processes (CSP) concurrent programming features added.")
     (license license:bsd-3)))
 
-(define-public go-1.7
+(define-public go-1.8
   (package
     (inherit go-1.4)
     (name "go")
-    (version "1.7.5")
+    (version "1.8.1")
     (source
      (origin
        (method url-fetch)
@@ -209,7 +210,7 @@ sequential processes (CSP) concurrent programming features added.")
                            name version ".src.tar.gz"))
        (sha256
         (base32
-         "058q57zmi23rflingzhy1b87yl69mb62ql2psfxqr7q7l89lb0sf"))))
+         "0mqf8ydxdx1pwmrs8p8wl5y1qrplzxmxzgb6vkghy4l67z0g9nik"))))
     (arguments
      (substitute-keyword-arguments (package-arguments go-1.4)
        ((#:phases phases)
@@ -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))
                  #t)))
            (add-before 'build 'set-bootstrap-variables
              (lambda* (#:key outputs inputs #:allow-other-keys)
@@ -364,4 +365,4 @@ sequential processes (CSP) concurrent programming features added.")
      `(("go" ,go-1.4)
        ,@(package-native-inputs go-1.4)))))
 
-(define-public go go-1.7)
+(define-public go go-1.8)
-- 
2.11.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 12:16 [PATCH] gnu: go: Update to 1.8.1 Petter
@ 2017-04-17 12:52 ` Tobias Geerinckx-Rice
  2017-04-17 12:54 ` Sergei Trofimovich
  1 sibling, 0 replies; 9+ messages in thread
From: Tobias Geerinckx-Rice @ 2017-04-17 12:52 UTC (permalink / raw)
  To: petter, guix-devel


[-- Attachment #1.1: Type: text/plain, Size: 1315 bytes --]

Petter,

There's a copyright header in Guix already, with an e-mail address, so I
doubt Sergei will mind:

  $ grep -r Sergei *
  gnu/packages/re2c.scm:;;; Copyright © 2017 Sergei Trofimovich <...>

While I suspect Sergei's already subscribed to this list, I've CC'd that
adress just in case. Hence the complete quote below.

Kind regards,

T G-R

On 17/04/17 14:16, Petter wrote:
> Hi,
> 
> With this patch I'm not sure how to properly handle Copyright. I've 
> taken a guess and prepared a patch anyway.
> 
> Here's the situation.
> 
> Updating to 1.8 was not trivial. A fix has been posted in a comment
> by Sergei Trofimovich on github[1]. It's a proper Guix patch,
> updating Go to 1.8, with the fix and the trivial update stuff, but as
> far as I can see not actually submitted to Guix.
> 
> [1] https://github.com/golang/go/issues/19132#issuecomment-285897612
> 
> After this Go 1.8.1 has been released. The attached patch has the 
> trivial update stuff for 1.8.1 and the fix by Sergei. I added Sergei
> to the Copyright section, but I'm not sure if this is correct, and if
> this is the way to do it. Also, I didn't find his e-mail address, so
> I just added his github page, in ()...
> 
> A little messy, please let me know if I should do something
> differently.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 504 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 12:16 [PATCH] gnu: go: Update to 1.8.1 Petter
  2017-04-17 12:52 ` Tobias Geerinckx-Rice
@ 2017-04-17 12:54 ` Sergei Trofimovich
  2017-04-17 13:37   ` Petter
  1 sibling, 1 reply; 9+ messages in thread
From: Sergei Trofimovich @ 2017-04-17 12:54 UTC (permalink / raw)
  To: Petter; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1929 bytes --]

On Mon, 17 Apr 2017 14:16:06 +0200
Petter <petter@mykolab.ch> wrote:

> Hi,
> 
> With this patch I'm not sure how to properly handle Copyright. I've 
> taken a guess and prepared a
> patch anyway.
> 
> Here's the situation.
> 
> Updating to 1.8 was not trivial. A fix has been posted in a comment by 
> Sergei Trofimovich on
> github[1]. It's a proper Guix patch, updating Go to 1.8, with the fix 
> and the trivial update stuff,
> but as far as I can see not actually submitted to Guix.
> 
> [1] https://github.com/golang/go/issues/19132#issuecomment-285897612
> 
> After this Go 1.8.1 has been released. The attached patch has the 
> trivial update stuff for 1.8.1
> and the fix by Sergei. I added Sergei to the Copyright section, but I'm 
> not sure if this is
> correct, and if this is the way to do it. Also, I didn't find his e-mail 
> address, so I just added
> his github page, in ()...
> 
> A little messy, please let me know if I should do something differently.

I consider the fix itself trivial thus I'm perfectly fine with not being
attributed at all. Apologies for making it hard to track me down.
I've added one of emails to github's front page.

[ While the fix is trivial debugging was a bit more fun :) ]

If you still like to add attribution (or have other reasons) feel free
to add "Sergei Trofimovich <slyfox@inbox.ru>"

I didn't get to submitting something that works because some unrelated
go-1.8.0 tests failed for me.

go-1.8.1 fails in a similar way, but if it works for you
it's very probable just my environment (x86_64-linux):

guix-master $ ./pre-inst-env guix build go-1.8

  ##### ../misc/cgo/test
  runtime/cgo: pthread_create failed: Resource temporarily unavailable
  scatter = 0x55ffc0
  hello from C
  sqrt is: 0
  SIGABRT: abort
  PC=0x7ffff71232c4 m=12 sigcode=18446744073709551610

Thanks for finishing go update!

-- 

  Sergei

[-- Attachment #2: Цифровая подпись OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 12:54 ` Sergei Trofimovich
@ 2017-04-17 13:37   ` Petter
  2017-04-17 17:18     ` Leo Famulari
  2017-04-22 17:39     ` Sergei Trofimovich
  0 siblings, 2 replies; 9+ messages in thread
From: Petter @ 2017-04-17 13:37 UTC (permalink / raw)
  To: Sergei Trofimovich; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 2494 bytes --]

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 :)

Best,
Petter

On 2017-04-17 14:54, Sergei Trofimovich wrote:
> On Mon, 17 Apr 2017 14:16:06 +0200
> Petter <petter@mykolab.ch> wrote:
> 
>> Hi,
>> 
>> With this patch I'm not sure how to properly handle Copyright. I've
>> taken a guess and prepared a
>> patch anyway.
>> 
>> Here's the situation.
>> 
>> Updating to 1.8 was not trivial. A fix has been posted in a comment by
>> Sergei Trofimovich on
>> github[1]. It's a proper Guix patch, updating Go to 1.8, with the fix
>> and the trivial update stuff,
>> but as far as I can see not actually submitted to Guix.
>> 
>> [1] https://github.com/golang/go/issues/19132#issuecomment-285897612
>> 
>> After this Go 1.8.1 has been released. The attached patch has the
>> trivial update stuff for 1.8.1
>> and the fix by Sergei. I added Sergei to the Copyright section, but 
>> I'm
>> not sure if this is
>> correct, and if this is the way to do it. Also, I didn't find his 
>> e-mail
>> address, so I just added
>> his github page, in ()...
>> 
>> A little messy, please let me know if I should do something 
>> differently.
> 
> I consider the fix itself trivial thus I'm perfectly fine with not 
> being
> attributed at all. Apologies for making it hard to track me down.
> I've added one of emails to github's front page.
> 
> [ While the fix is trivial debugging was a bit more fun :) ]
> 
> If you still like to add attribution (or have other reasons) feel free
> to add "Sergei Trofimovich <slyfox@inbox.ru>"
> 
> I didn't get to submitting something that works because some unrelated
> go-1.8.0 tests failed for me.
> 
> go-1.8.1 fails in a similar way, but if it works for you
> it's very probable just my environment (x86_64-linux):
> 
> guix-master $ ./pre-inst-env guix build go-1.8
> 
>   ##### ../misc/cgo/test
>   runtime/cgo: pthread_create failed: Resource temporarily unavailable
>   scatter = 0x55ffc0
>   hello from C
>   sqrt is: 0
>   SIGABRT: abort
>   PC=0x7ffff71232c4 m=12 sigcode=18446744073709551610
> 
> Thanks for finishing go update!

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-go-Update-to-1.8.1.patch --]
[-- Type: text/x-diff; name=0001-gnu-go-Update-to-1.8.1.patch, Size: 2683 bytes --]

From ae8434c72d75f68fa25571bee72084a623661a57 Mon Sep 17 00:00:00 2001
From: Petter <petter@mykolab.ch>
Date: Mon, 17 Apr 2017 15:14:18 +0200
Subject: [PATCH] gnu: go: Update to 1.8.1.

* gnu/packages/golang.scm (go): Update to 1.8.1.
---
 gnu/packages/golang.scm | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 00630ce06..90a809eb2 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -3,8 +3,9 @@
 ;;; Copyright © 2016 Matthew Jordan <matthewjordandevops@yandex.com>
 ;;; Copyright © 2016 Andy Wingo <wingo@igalia.com>
 ;;; Copyright © 2016 Ludovic Courtès <ludo@gnu.org>
-;;; Copyright © 2016 Petter <petter@mykolab.ch>
+;;; Copyright © 2016, 2017 Petter <petter@mykolab.ch>
 ;;; Copyright © 2016, 2017 Leo Famulari <leo@famulari.name>
+;;; Copyright © 2017 Sergei Trofimovich <slyfox@inbox.ru>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -197,11 +198,11 @@ garbage collection, various safety features and in the style of communicating
 sequential processes (CSP) concurrent programming features added.")
     (license license:bsd-3)))
 
-(define-public go-1.7
+(define-public go-1.8
   (package
     (inherit go-1.4)
     (name "go")
-    (version "1.7.5")
+    (version "1.8.1")
     (source
      (origin
        (method url-fetch)
@@ -209,7 +210,7 @@ sequential processes (CSP) concurrent programming features added.")
                            name version ".src.tar.gz"))
        (sha256
         (base32
-         "058q57zmi23rflingzhy1b87yl69mb62ql2psfxqr7q7l89lb0sf"))))
+         "0mqf8ydxdx1pwmrs8p8wl5y1qrplzxmxzgb6vkghy4l67z0g9nik"))))
     (arguments
      (substitute-keyword-arguments (package-arguments go-1.4)
        ((#:phases phases)
@@ -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))
                  #t)))
            (add-before 'build 'set-bootstrap-variables
              (lambda* (#:key outputs inputs #:allow-other-keys)
@@ -364,4 +365,4 @@ sequential processes (CSP) concurrent programming features added.")
      `(("go" ,go-1.4)
        ,@(package-native-inputs go-1.4)))))
 
-(define-public go go-1.7)
+(define-public go go-1.8)
-- 
2.11.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 13:37   ` Petter
@ 2017-04-17 17:18     ` Leo Famulari
  2017-04-17 17:55       ` Petter
  2017-04-22 17:39     ` Sergei Trofimovich
  1 sibling, 1 reply; 9+ messages in thread
From: Leo Famulari @ 2017-04-17 17:18 UTC (permalink / raw)
  To: Petter; +Cc: guix-devel


[-- Attachment #1.1: Type: text/plain, Size: 1400 bytes --]

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.

[-- Attachment #1.2: 0001-gnu-go-Update-to-1.8.1.patch --]
[-- Type: text/plain, Size: 2839 bytes --]

From c64da0b22a602302d9262b7e76ce7a95bee97f95 Mon Sep 17 00:00:00 2001
From: Petter <petter@mykolab.ch>
Date: Mon, 17 Apr 2017 15:14:18 +0200
Subject: [PATCH] gnu: go: Update to 1.8.1.

* gnu/packages/golang.scm (go-1.7): Replace with ...
(go-1.8): New variable.
[arguments]: Modify substitutions in 'prebuild' phase.
(go): Update to go-1.8.
---
 gnu/packages/golang.scm | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 00630ce06..f84622bae 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -3,8 +3,9 @@
 ;;; Copyright © 2016 Matthew Jordan <matthewjordandevops@yandex.com>
 ;;; Copyright © 2016 Andy Wingo <wingo@igalia.com>
 ;;; Copyright © 2016 Ludovic Courtès <ludo@gnu.org>
-;;; Copyright © 2016 Petter <petter@mykolab.ch>
+;;; Copyright © 2016, 2017 Petter <petter@mykolab.ch>
 ;;; Copyright © 2016, 2017 Leo Famulari <leo@famulari.name>
+;;; Copyright © 2017 Sergei Trofimovich <slyfox@inbox.ru>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -197,11 +198,11 @@ garbage collection, various safety features and in the style of communicating
 sequential processes (CSP) concurrent programming features added.")
     (license license:bsd-3)))
 
-(define-public go-1.7
+(define-public go-1.8
   (package
     (inherit go-1.4)
     (name "go")
-    (version "1.7.5")
+    (version "1.8.1")
     (source
      (origin
        (method url-fetch)
@@ -209,7 +210,7 @@ sequential processes (CSP) concurrent programming features added.")
                            name version ".src.tar.gz"))
        (sha256
         (base32
-         "058q57zmi23rflingzhy1b87yl69mb62ql2psfxqr7q7l89lb0sf"))))
+         "0mqf8ydxdx1pwmrs8p8wl5y1qrplzxmxzgb6vkghy4l67z0g9nik"))))
     (arguments
      (substitute-keyword-arguments (package-arguments go-1.4)
        ((#:phases phases)
@@ -301,7 +302,9 @@ sequential processes (CSP) concurrent programming features added.")
                  (substitute* "time/zoneinfo_unix.go"
                    (("/usr/share/zoneinfo/") tzdata-path))
                  (substitute* (find-files "cmd" "asm.c")
-                   (("/lib/ld-linux.*\\.so\\.[0-9]") loader))
+                   (("/lib(64)?/ld-linux.*\\.so\\.[0-9]") loader))
+                 (substitute* (find-files "cmd" "\\.go")
+                   (("/lib(64)?/ld-linux.*\\.so\\.[0-9]") loader))
                  #t)))
            (add-before 'build 'set-bootstrap-variables
              (lambda* (#:key outputs inputs #:allow-other-keys)
@@ -364,4 +367,4 @@ sequential processes (CSP) concurrent programming features added.")
      `(("go" ,go-1.4)
        ,@(package-native-inputs go-1.4)))))
 
-(define-public go go-1.7)
+(define-public go go-1.8)
-- 
2.12.2


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 17:18     ` Leo Famulari
@ 2017-04-17 17:55       ` Petter
  2017-04-17 18:18         ` Leo Famulari
  0 siblings, 1 reply; 9+ messages in thread
From: Petter @ 2017-04-17 17:55 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Hi Leo,

Good point! As the build succeeded I didn't think any of it. This could 
be important for other architectures than what Sergei and I are using.

However,
> find /tmp/go/src/cmd/ -name '*asm\.c*'
doesn't return any file. Looks like they've removed these files. I'm 
pretty sure they're working to "free" their source of C code, so I 
wouldn't expect these files to reappear in a later release either.

With this information I think it's safe to remove the asm.c 
substitution.

Best,
Petter

On 2017-04-17 19:18, Leo Famulari wrote:
> 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.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 17:55       ` Petter
@ 2017-04-17 18:18         ` Leo Famulari
  2017-04-17 18:28           ` Petter
  0 siblings, 1 reply; 9+ messages in thread
From: Leo Famulari @ 2017-04-17 18:18 UTC (permalink / raw)
  To: Petter; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 723 bytes --]

On Mon, Apr 17, 2017 at 07:55:35PM +0200, Petter wrote:
> Hi Leo,
> 
> Good point! As the build succeeded I didn't think any of it. This could be
> important for other architectures than what Sergei and I are using.
> 
> However,
> > find /tmp/go/src/cmd/ -name '*asm\.c*'
> doesn't return any file. Looks like they've removed these files. I'm pretty
> sure they're working to "free" their source of C code, so I wouldn't expect
> these files to reappear in a later release either.

Indeed, I must have been looking at an old version of the Go source
tree.

> With this information I think it's safe to remove the asm.c substitution.

Yup! I pushed the update as c04ef86e8114225f5e5859f379582ba6f69aacfb.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 18:18         ` Leo Famulari
@ 2017-04-17 18:28           ` Petter
  0 siblings, 0 replies; 9+ messages in thread
From: Petter @ 2017-04-17 18:28 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Thanks Leo!

On 2017-04-17 20:18, Leo Famulari wrote:
> On Mon, Apr 17, 2017 at 07:55:35PM +0200, Petter wrote:
>> Hi Leo,
>> 
>> Good point! As the build succeeded I didn't think any of it. This 
>> could be
>> important for other architectures than what Sergei and I are using.
>> 
>> However,
>> > find /tmp/go/src/cmd/ -name '*asm\.c*'
>> doesn't return any file. Looks like they've removed these files. I'm 
>> pretty
>> sure they're working to "free" their source of C code, so I wouldn't 
>> expect
>> these files to reappear in a later release either.
> 
> Indeed, I must have been looking at an old version of the Go source
> tree.
> 
>> With this information I think it's safe to remove the asm.c 
>> substitution.
> 
> Yup! I pushed the update as c04ef86e8114225f5e5859f379582ba6f69aacfb.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] gnu: go: Update to 1.8.1
  2017-04-17 13:37   ` Petter
  2017-04-17 17:18     ` Leo Famulari
@ 2017-04-22 17:39     ` Sergei Trofimovich
  1 sibling, 0 replies; 9+ messages in thread
From: Sergei Trofimovich @ 2017-04-22 17:39 UTC (permalink / raw)
  To: Petter; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 538 bytes --]

On Mon, 17 Apr 2017 15:37:49 +0200
Petter <petter@mykolab.ch> wrote:

> > > ##### ../misc/cgo/test
> > >  runtime/cgo: pthread_create failed: Resource temporarily unavailable
...
> > $ /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!

Finally managed to fix go-1.8.1 for me as well \o/.

The culprit was a low thread limit for guix-daemon container:
    http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26611

-- 

  Sergei

[-- Attachment #2: Цифровая подпись OpenPGP --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2017-04-22 17:39 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-17 12:16 [PATCH] gnu: go: Update to 1.8.1 Petter
2017-04-17 12:52 ` Tobias Geerinckx-Rice
2017-04-17 12:54 ` Sergei Trofimovich
2017-04-17 13:37   ` Petter
2017-04-17 17:18     ` Leo Famulari
2017-04-17 17:55       ` Petter
2017-04-17 18:18         ` Leo Famulari
2017-04-17 18:28           ` Petter
2017-04-22 17:39     ` Sergei Trofimovich

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).