unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: tbb: Update to 2017_20160722.
@ 2016-09-09  9:51 ng0
  2016-09-09 14:57 ` Eric Bavier
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2016-09-09  9:51 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/tbb.scm (tbb): Update to 2017_20160722.
---
 gnu/packages/tbb.scm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
index 0e9db72..22ce068 100644
--- a/gnu/packages/tbb.scm
+++ b/gnu/packages/tbb.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,16 +28,16 @@
 (define-public tbb
   (package
     (name "tbb")
-    (version "4.3.2")
+    (version "2017_20160722")
     (source (origin
               (method url-fetch)
               (uri (string-append
                     "https://www.threadingbuildingblocks.org/sites/default"
                     "/files/software_releases/source/"
-                    "tbb43_20141204oss_src.tgz"))
+                    "tbb" version "oss_src.tgz"))
               (sha256
                (base32
-                "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
+                "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
               (modules '((guix build utils)))
               (snippet
                '(substitute* "build/common.inc"
-- 
2.10.0

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-09  9:51 [PATCH] gnu: tbb: Update to 2017_20160722 ng0
@ 2016-09-09 14:57 ` Eric Bavier
  2016-09-09 15:10   ` ng0
  0 siblings, 1 reply; 8+ messages in thread
From: Eric Bavier @ 2016-09-09 14:57 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel, Guix-devel

On 2016-09-09 04:51, ng0 wrote:
> * gnu/packages/tbb.scm (tbb): Update to 2017_20160722.
> ---
>  gnu/packages/tbb.scm | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
> index 0e9db72..22ce068 100644
> --- a/gnu/packages/tbb.scm
> +++ b/gnu/packages/tbb.scm
> @@ -1,5 +1,6 @@
>  ;;; GNU Guix --- Functional package management for GNU
>  ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
> +;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -27,16 +28,16 @@
>  (define-public tbb
>    (package
>      (name "tbb")
> -    (version "4.3.2")
> +    (version "2017_20160722")

I think we should go with "2017" as the version; it seems to be what's 
advertised.  This is their first release with the year-as-version, so we 
don't know how they'd like to denote patch versions.

>      (source (origin
>                (method url-fetch)
>                (uri (string-append
>                      
> "https://www.threadingbuildingblocks.org/sites/default"
>                      "/files/software_releases/source/"
> -                    "tbb43_20141204oss_src.tgz"))
> +                    "tbb" version "oss_src.tgz"))
>                (sha256
>                 (base32
> -                
> "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
> +                
> "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
>                (modules '((guix build utils)))
>                (snippet
>                 '(substitute* "build/common.inc"

The 2017 version also brings with it a change in license from GPLv2 with 
libstdc++ runtime exception to the ASL2.0.  Could you include that in 
this patch too?

Thanks,
-- 
`~Eric

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-09 14:57 ` Eric Bavier
@ 2016-09-09 15:10   ` ng0
  2016-09-09 15:42     ` Eric Bavier
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2016-09-09 15:10 UTC (permalink / raw)
  To: Eric Bavier; +Cc: guix-devel

Eric Bavier <ericbavier@openmailbox.org> writes:

> On 2016-09-09 04:51, ng0 wrote:
>> * gnu/packages/tbb.scm (tbb): Update to 2017_20160722.
>> ---
>>  gnu/packages/tbb.scm | 7 ++++---
>>  1 file changed, 4 insertions(+), 3 deletions(-)
>> 
>> diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
>> index 0e9db72..22ce068 100644
>> --- a/gnu/packages/tbb.scm
>> +++ b/gnu/packages/tbb.scm
>> @@ -1,5 +1,6 @@
>>  ;;; GNU Guix --- Functional package management for GNU
>>  ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
>> +;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
>>  ;;;
>>  ;;; This file is part of GNU Guix.
>>  ;;;
>> @@ -27,16 +28,16 @@
>>  (define-public tbb
>>    (package
>>      (name "tbb")
>> -    (version "4.3.2")
>> +    (version "2017_20160722")
>
> I think we should go with "2017" as the version; it seems to be what's 
> advertised.  This is their first release with the year-as-version, so we 
> don't know how they'd like to denote patch versions.

Okay

>>      (source (origin
>>                (method url-fetch)
>>                (uri (string-append
>>                      
>> "https://www.threadingbuildingblocks.org/sites/default"
>>                      "/files/software_releases/source/"
>> -                    "tbb43_20141204oss_src.tgz"))
>> +                    "tbb" version "oss_src.tgz"))
>>                (sha256
>>                 (base32
>> -                
>> "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
>> +                
>> "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
>>                (modules '((guix build utils)))
>>                (snippet
>>                 '(substitute* "build/common.inc"
>
> The 2017 version also brings with it a change in license from GPLv2 with 
> libstdc++ runtime exception to the ASL2.0.  Could you include that in 
> this patch too?
>
> Thanks,
> -- 
> `~Eric
>

This is what I used in the mislead "Add tbb" patch I've sent
yesterday. It is now completely ASL2.0, or is it GPL2+ASL2.0? I've read
it yesterday as ASL only.
-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-09 15:10   ` ng0
@ 2016-09-09 15:42     ` Eric Bavier
  2016-09-09 16:13       ` ng0
  0 siblings, 1 reply; 8+ messages in thread
From: Eric Bavier @ 2016-09-09 15:42 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

On 2016-09-09 10:10, ng0 wrote:
> Eric Bavier <ericbavier@openmailbox.org> writes:
> 
>> The 2017 version also brings with it a change in license from GPLv2 
>> with
>> libstdc++ runtime exception to the ASL2.0.  Could you include that in
>> this patch too?
>> 
>> Thanks,
>> --
>> `~Eric
>> 
> 
> This is what I used in the mislead "Add tbb" patch I've sent
> yesterday. It is now completely ASL2.0, or is it GPL2+ASL2.0? I've read
> it yesterday as ASL only.

ASL only AFAICT.  The www.threadbuildingblocks.org/licensing page says 
GPLv2, but that may just be outdated.  The tarball is ASL, which is what 
we have to go with.

-- 
`~Eric

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-09 15:42     ` Eric Bavier
@ 2016-09-09 16:13       ` ng0
  2016-09-10 20:48         ` Efraim Flashner
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2016-09-09 16:13 UTC (permalink / raw)
  To: Eric Bavier; +Cc: guix-devel


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

Eric Bavier <ericbavier@openmailbox.org> writes:

> On 2016-09-09 10:10, ng0 wrote:
>> Eric Bavier <ericbavier@openmailbox.org> writes:
>> 
>>> The 2017 version also brings with it a change in license from GPLv2 
>>> with
>>> libstdc++ runtime exception to the ASL2.0.  Could you include that in
>>> this patch too?
>>> 
>>> Thanks,
>>> --
>>> `~Eric
>>> 
>> 
>> This is what I used in the mislead "Add tbb" patch I've sent
>> yesterday. It is now completely ASL2.0, or is it GPL2+ASL2.0? I've read
>> it yesterday as ASL only.
>
> ASL only AFAICT.  The www.threadbuildingblocks.org/licensing page says 
> GPLv2, but that may just be outdated.  The tarball is ASL, which is what 
> we have to go with.
>
> -- 
> `~Eric
>

Ok.


[-- Attachment #1.2: 0001-gnu-tbb-Update-to-2017.patch --]
[-- Type: text/x-patch, Size: 2006 bytes --]

From a3708f40b760009c5128bd048d5cba1a63462909 Mon Sep 17 00:00:00 2001
From: ng0 <ng0@we.make.ritual.n0.is>
Date: Thu, 8 Sep 2016 23:04:52 +0000
Subject: [PATCH] gnu: tbb: Update to 2017.

* gnu/packages/tbb.scm (tbb): Update to 2017.
---
 gnu/packages/tbb.scm | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
index 0e9db72..c19391a 100644
--- a/gnu/packages/tbb.scm
+++ b/gnu/packages/tbb.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,16 +28,16 @@
 (define-public tbb
   (package
     (name "tbb")
-    (version "4.3.2")
+    (version "2017")
     (source (origin
               (method url-fetch)
               (uri (string-append
                     "https://www.threadingbuildingblocks.org/sites/default"
                     "/files/software_releases/source/"
-                    "tbb43_20141204oss_src.tgz"))
+                    "tbb" version "_20160722oss_src.tgz"))
               (sha256
                (base32
-                "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
+                "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
               (modules '((guix build utils)))
               (snippet
                '(substitute* "build/common.inc"
@@ -85,6 +86,4 @@ the low-level threading details necessary for optimal multi-core performance.
 It uses common C++ templates and coding style to eliminate tedious threading
 implementation work.  It provides parallel loop constructs, asynchronous
 tasks, synchronization primitives, atomic operations, and more.")
-    ;; GPLv2 with run-time exception:
-    ;; <https://www.threadingbuildingblocks.org/licensing#runtime-exception>
-    (license gpl2)))
+    (license asl2.0)))
-- 
2.10.0


[-- Attachment #1.3: Type: text/plain, Size: 74 bytes --]


-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

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

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-09 16:13       ` ng0
@ 2016-09-10 20:48         ` Efraim Flashner
  2016-09-11  7:40           ` ng0
  0 siblings, 1 reply; 8+ messages in thread
From: Efraim Flashner @ 2016-09-10 20:48 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

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

On Fri, Sep 09, 2016 at 04:13:54PM +0000, ng0 wrote:
> Eric Bavier <ericbavier@openmailbox.org> writes:
> 
> > On 2016-09-09 10:10, ng0 wrote:
> >> Eric Bavier <ericbavier@openmailbox.org> writes:
> >> 
> >>> The 2017 version also brings with it a change in license from GPLv2 
> >>> with
> >>> libstdc++ runtime exception to the ASL2.0.  Could you include that in
> >>> this patch too?
> >>> 
> >>> Thanks,
> >>> --
> >>> `~Eric
> >>> 
> >> 
> >> This is what I used in the mislead "Add tbb" patch I've sent
> >> yesterday. It is now completely ASL2.0, or is it GPL2+ASL2.0? I've read
> >> it yesterday as ASL only.
> >
> > ASL only AFAICT.  The www.threadbuildingblocks.org/licensing page says 
> > GPLv2, but that may just be outdated.  The tarball is ASL, which is what 
> > we have to go with.
> >
> > -- 
> > `~Eric
> >
> 
> Ok.
> 

> From a3708f40b760009c5128bd048d5cba1a63462909 Mon Sep 17 00:00:00 2001
> From: ng0 <ng0@we.make.ritual.n0.is>
> Date: Thu, 8 Sep 2016 23:04:52 +0000
> Subject: [PATCH] gnu: tbb: Update to 2017.
> 
> * gnu/packages/tbb.scm (tbb): Update to 2017.

don't forget to mention the license change here.

> ---
>  gnu/packages/tbb.scm | 11 +++++------
>  1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
> index 0e9db72..c19391a 100644
> --- a/gnu/packages/tbb.scm
> +++ b/gnu/packages/tbb.scm
> @@ -1,5 +1,6 @@
>  ;;; GNU Guix --- Functional package management for GNU
>  ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
> +;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -27,16 +28,16 @@
>  (define-public tbb
>    (package
>      (name "tbb")
> -    (version "4.3.2")
> +    (version "2017")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append
>                      "https://www.threadingbuildingblocks.org/sites/default"
>                      "/files/software_releases/source/"
> -                    "tbb43_20141204oss_src.tgz"))
> +                    "tbb" version "_20160722oss_src.tgz"))
>                (sha256
>                 (base32
> -                "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
> +                "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
>                (modules '((guix build utils)))
>                (snippet
>                 '(substitute* "build/common.inc"
> @@ -85,6 +86,4 @@ the low-level threading details necessary for optimal multi-core performance.
>  It uses common C++ templates and coding style to eliminate tedious threading
>  implementation work.  It provides parallel loop constructs, asynchronous
>  tasks, synchronization primitives, atomic operations, and more.")
> -    ;; GPLv2 with run-time exception:
> -    ;; <https://www.threadingbuildingblocks.org/licensing#runtime-exception>
> -    (license gpl2)))
> +    (license asl2.0)))
> -- 
> 2.10.0
> 

> 
> -- 
> ng0
> For non-prism friendly talk find me on http://www.psyced.org




-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-10 20:48         ` Efraim Flashner
@ 2016-09-11  7:40           ` ng0
  2016-09-12  8:41             ` Efraim Flashner
  0 siblings, 1 reply; 8+ messages in thread
From: ng0 @ 2016-09-11  7:40 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: guix-devel

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

Efraim Flashner <efraim@flashner.co.il> writes:

> [ Unknown signature status ]
> On Fri, Sep 09, 2016 at 04:13:54PM +0000, ng0 wrote:
>> Eric Bavier <ericbavier@openmailbox.org> writes:
>> 
>> > On 2016-09-09 10:10, ng0 wrote:
>> >> Eric Bavier <ericbavier@openmailbox.org> writes:
>> >> 
>> >>> The 2017 version also brings with it a change in license from GPLv2 
>> >>> with
>> >>> libstdc++ runtime exception to the ASL2.0.  Could you include that in
>> >>> this patch too?
>> >>> 
>> >>> Thanks,
>> >>> --
>> >>> `~Eric
>> >>> 
>> >> 
>> >> This is what I used in the mislead "Add tbb" patch I've sent
>> >> yesterday. It is now completely ASL2.0, or is it GPL2+ASL2.0? I've read
>> >> it yesterday as ASL only.
>> >
>> > ASL only AFAICT.  The www.threadbuildingblocks.org/licensing page says 
>> > GPLv2, but that may just be outdated.  The tarball is ASL, which is what 
>> > we have to go with.
>> >
>> > -- 
>> > `~Eric
>> >
>> 
>> Ok.
>> 
>
>> From a3708f40b760009c5128bd048d5cba1a63462909 Mon Sep 17 00:00:00 2001
>> From: ng0 <ng0@we.make.ritual.n0.is>
>> Date: Thu, 8 Sep 2016 23:04:52 +0000
>> Subject: [PATCH] gnu: tbb: Update to 2017.
>> 
>> * gnu/packages/tbb.scm (tbb): Update to 2017.
>
> don't forget to mention the license change here.

Oh, yes. Thanks.

[…]



[-- Attachment #2: 0001-gnu-tbb-Update-to-2017.patch --]
[-- Type: text/x-patch, Size: 1983 bytes --]

From a4533698ea5915a56cb7c415481d9ed22d43ff29 Mon Sep 17 00:00:00 2001
From: ng0 <ng0@we.make.ritual.n0.is>
Date: Thu, 8 Sep 2016 23:04:52 +0000
Subject: [PATCH] gnu: tbb: Update to 2017.

* gnu/packages/tbb.scm (tbb): Update to 2017.
[license]: Change to ASL2.0.
---
 gnu/packages/tbb.scm | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
index 0e9db72..c19391a 100644
--- a/gnu/packages/tbb.scm
+++ b/gnu/packages/tbb.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015 Ricardo Wurmus <rekado@elephly.net>
+;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,16 +28,16 @@
 (define-public tbb
   (package
     (name "tbb")
-    (version "4.3.2")
+    (version "2017")
     (source (origin
               (method url-fetch)
               (uri (string-append
                     "https://www.threadingbuildingblocks.org/sites/default"
                     "/files/software_releases/source/"
-                    "tbb43_20141204oss_src.tgz"))
+                    "tbb" version "_20160722oss_src.tgz"))
               (sha256
                (base32
-                "0jsczl99jfgj47kj7c4sd4fk7v3rbaiax1ng9ypykz1hh0lrrsws"))
+                "038rmv3s8si51bjrzwyv8ldqw742fjjdfayi8pmjaq5zw32b8pzx"))
               (modules '((guix build utils)))
               (snippet
                '(substitute* "build/common.inc"
@@ -85,6 +86,4 @@ the low-level threading details necessary for optimal multi-core performance.
 It uses common C++ templates and coding style to eliminate tedious threading
 implementation work.  It provides parallel loop constructs, asynchronous
 tasks, synchronization primitives, atomic operations, and more.")
-    ;; GPLv2 with run-time exception:
-    ;; <https://www.threadingbuildingblocks.org/licensing#runtime-exception>
-    (license gpl2)))
+    (license asl2.0)))
-- 
2.10.0


[-- Attachment #3: Type: text/plain, Size: 71 bytes --]



-- 
ng0
For non-prism friendly talk find me on http://www.psyced.org

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

* Re: [PATCH] gnu: tbb: Update to 2017_20160722.
  2016-09-11  7:40           ` ng0
@ 2016-09-12  8:41             ` Efraim Flashner
  0 siblings, 0 replies; 8+ messages in thread
From: Efraim Flashner @ 2016-09-12  8:41 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

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

On Sun, Sep 11, 2016 at 07:40:03AM +0000, ng0 wrote:
> Efraim Flashner <efraim@flashner.co.il> writes:
> 
> >
> > don't forget to mention the license change here.
> 
> Oh, yes. Thanks.
> 
> […]
> 
> 

Patch pushed!


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

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

end of thread, other threads:[~2016-09-12  8:41 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-09  9:51 [PATCH] gnu: tbb: Update to 2017_20160722 ng0
2016-09-09 14:57 ` Eric Bavier
2016-09-09 15:10   ` ng0
2016-09-09 15:42     ` Eric Bavier
2016-09-09 16:13       ` ng0
2016-09-10 20:48         ` Efraim Flashner
2016-09-11  7:40           ` ng0
2016-09-12  8:41             ` Efraim Flashner

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