all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: mesa: Update to 13.0.1.
@ 2016-11-30 12:06 Marius Bakke
  2016-12-01 14:09 ` Ludovic Courtès
  0 siblings, 1 reply; 9+ messages in thread
From: Marius Bakke @ 2016-11-30 12:06 UTC (permalink / raw)
  To: guix-devel; +Cc: Marius Bakke

* gnu/packages/gl.scm (mesa): Update to 13.0.1.
[native-inputs]: Move 'mesa-wayland-egl-symbols-check-mips.patch' to ...
[source]: ... here.
[arguments]: Don't apply patch.
[inputs]: Remove eudev.
---
 gnu/packages/gl.scm | 28 +++++-----------------------
 1 file changed, 5 insertions(+), 23 deletions(-)

diff --git a/gnu/packages/gl.scm b/gnu/packages/gl.scm
index 50b474c..850dfe1 100644
--- a/gnu/packages/gl.scm
+++ b/gnu/packages/gl.scm
@@ -196,7 +196,7 @@ also known as DXTn or DXTC) for Mesa.")
 (define-public mesa
   (package
     (name "mesa")
-    (version "12.0.1")
+    (version "13.0.1")
     (source
       (origin
         (method url-fetch)
@@ -204,7 +204,9 @@ also known as DXTn or DXTC) for Mesa.")
                             version "/mesa-" version ".tar.xz"))
         (sha256
          (base32
-          "12b3i59xdn2in2hchrkgh4fwij8zhznibx976l3pdj3qkyvlzcms"))))
+          "0cd7axwihwsay0i9fvcw14cldbxyvf8b8rd5sh53plvppyr2z5ki"))
+        (patches
+         (search-patches "mesa-wayland-egl-symbols-check-mips.patch"))))
     (build-system gnu-build-system)
     (propagated-inputs
       `(("glproto" ,glproto)
@@ -227,20 +229,10 @@ also known as DXTn or DXTC) for Mesa.")
         ("makedepend" ,makedepend)
         ("presentproto" ,presentproto)
         ("s2tc" ,s2tc)
-        ("udev" ,eudev)
         ("wayland" ,wayland)))
     (native-inputs
       `(("pkg-config" ,pkg-config)
-        ("python" ,python-2)
-
-         ;; XXX To prevent a large number of rebuilds on other systems,
-         ;; apply the following patch on MIPS systems only.  In the next
-         ;; core-updates cycle, this patch could be applied on all platforms.
-        ,@(if (string-prefix? "mips" (or (%current-target-system)
-                                         (%current-system)))
-              `(("mips-patch"
-                 ,(search-patch "mesa-wayland-egl-symbols-check-mips.patch")))
-              '())))
+        ("python" ,python-2)))
     (arguments
      `(#:configure-flags
        '(;; drop r300 from default gallium drivers, as it requires llvm
@@ -267,16 +259,6 @@ also known as DXTn or DXTC) for Mesa.")
               '("--with-dri-drivers=nouveau,r200,radeon,swrast"))))
        #:phases
        (modify-phases %standard-phases
-         ;; Add an 'apply-mips-patch' phase conditionally (see above.)
-         ,@(if (string-prefix? "mips" (or (%current-target-system)
-                                          (%current-system)))
-               `((add-after 'unpack 'apply-mips-patch
-                   (lambda* (#:key inputs #:allow-other-keys)
-                     (let ((patch (assoc-ref inputs "mips-patch")))
-                       (zero? (system* "patch" "-p1" "--force"
-                                       "--input" patch))))))
-               '())
-
          (add-after
            'unpack 'patch-create_test_cases
            (lambda _
-- 
2.10.2

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-11-30 12:06 [PATCH] gnu: mesa: Update to 13.0.1 Marius Bakke
@ 2016-12-01 14:09 ` Ludovic Courtès
  2016-12-01 17:14   ` Marius Bakke
  2016-12-01 18:39   ` [PATCH] gnu: mesa: Update to 13.0.1 Mark H Weaver
  0 siblings, 2 replies; 9+ messages in thread
From: Ludovic Courtès @ 2016-12-01 14:09 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> skribis:

> * gnu/packages/gl.scm (mesa): Update to 13.0.1.
> [native-inputs]: Move 'mesa-wayland-egl-symbols-check-mips.patch' to ...
> [source]: ... here.
> [arguments]: Don't apply patch.
> [inputs]: Remove eudev.

eudev was added in this commit:

  commit ce4d7abf02627e990551c3bf0b67abfdf6725952
  Author: Mark H Weaver <mhw@netris.org>
  Date:   Tue Dec 16 17:22:47 2014 -0500

      gnu: mesa: Update to 10.4.0.

      * gnu/packages/gl.scm (mesa): Update to 10.4.0.  Use current libdrm.  Add
        eudev, dri3proto, presentproto, and libxshmfence to inputs.  Add gnu-gettext
        to native-inputs.  Add --enable-xa configure flag.  Add new phases
        'add-missing-m4-files' and 'patch-create_test_cases'.

Are you sure it’s no longer necessary?  Mark?

The parts about apply the MIPS patch is good to me.  Perhaps you can
already push it as a separate patch to ‘staging’.

Thanks,
Ludo’.

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-12-01 14:09 ` Ludovic Courtès
@ 2016-12-01 17:14   ` Marius Bakke
  2016-12-01 23:02     ` Ludovic Courtès
  2016-12-01 18:39   ` [PATCH] gnu: mesa: Update to 13.0.1 Mark H Weaver
  1 sibling, 1 reply; 9+ messages in thread
From: Marius Bakke @ 2016-12-01 17:14 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

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

Ludovic Courtès <ludo@gnu.org> writes:

> Marius Bakke <mbakke@fastmail.com> skribis:
>
>> * gnu/packages/gl.scm (mesa): Update to 13.0.1.
>> [native-inputs]: Move 'mesa-wayland-egl-symbols-check-mips.patch' to ...
>> [source]: ... here.
>> [arguments]: Don't apply patch.
>> [inputs]: Remove eudev.
>
> eudev was added in this commit:
>
>   commit ce4d7abf02627e990551c3bf0b67abfdf6725952
>   Author: Mark H Weaver <mhw@netris.org>
>   Date:   Tue Dec 16 17:22:47 2014 -0500
>
>       gnu: mesa: Update to 10.4.0.
>
>       * gnu/packages/gl.scm (mesa): Update to 10.4.0.  Use current libdrm.  Add
>         eudev, dri3proto, presentproto, and libxshmfence to inputs.  Add gnu-gettext
>         to native-inputs.  Add --enable-xa configure flag.  Add new phases
>         'add-missing-m4-files' and 'patch-create_test_cases'.
>
> Are you sure it’s no longer necessary?  Mark?

I removed eudev after reading the release notes for 13.0.0:

http://mesa3d.org/relnotes/13.0.0.html

(very bottom of page)

> The parts about apply the MIPS patch is good to me.  Perhaps you can
> already push it as a separate patch to ‘staging’.

I can submit an update for 12.0.4 if the major version bump is too risky
right now, or bump to 13.0.2 which was released three days ago. WDYT?

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

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-12-01 14:09 ` Ludovic Courtès
  2016-12-01 17:14   ` Marius Bakke
@ 2016-12-01 18:39   ` Mark H Weaver
  1 sibling, 0 replies; 9+ messages in thread
From: Mark H Weaver @ 2016-12-01 18:39 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> writes:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Marius Bakke <mbakke@fastmail.com> skribis:
>>
>>> * gnu/packages/gl.scm (mesa): Update to 13.0.1.
>>> [native-inputs]: Move 'mesa-wayland-egl-symbols-check-mips.patch' to ...
>>> [source]: ... here.
>>> [arguments]: Don't apply patch.
>>> [inputs]: Remove eudev.
>>
>> eudev was added in this commit:
>>
>>   commit ce4d7abf02627e990551c3bf0b67abfdf6725952
>>   Author: Mark H Weaver <mhw@netris.org>
>>   Date:   Tue Dec 16 17:22:47 2014 -0500
>>
>>       gnu: mesa: Update to 10.4.0.
>>
>>       * gnu/packages/gl.scm (mesa): Update to 10.4.0.  Use current libdrm.  Add
>>         eudev, dri3proto, presentproto, and libxshmfence to inputs.  Add gnu-gettext
>>         to native-inputs.  Add --enable-xa configure flag.  Add new phases
>>         'add-missing-m4-files' and 'patch-create_test_cases'.
>>
>> Are you sure it’s no longer necessary?  Mark?
>
> I removed eudev after reading the release notes for 13.0.0:
>
> http://mesa3d.org/relnotes/13.0.0.html
>
> (very bottom of page)

Indeed, the release notes seem to indicate that 'eudev' can be removed
from 'inputs'.  Looks okay to me.

    Thanks!
      Mark

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-12-01 17:14   ` Marius Bakke
@ 2016-12-01 23:02     ` Ludovic Courtès
  2016-12-02 11:26       ` Marius Bakke
  0 siblings, 1 reply; 9+ messages in thread
From: Ludovic Courtès @ 2016-12-01 23:02 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Marius Bakke <mbakke@fastmail.com> skribis:
>>
>>> * gnu/packages/gl.scm (mesa): Update to 13.0.1.
>>> [native-inputs]: Move 'mesa-wayland-egl-symbols-check-mips.patch' to ...
>>> [source]: ... here.
>>> [arguments]: Don't apply patch.
>>> [inputs]: Remove eudev.
>>
>> eudev was added in this commit:
>>
>>   commit ce4d7abf02627e990551c3bf0b67abfdf6725952
>>   Author: Mark H Weaver <mhw@netris.org>
>>   Date:   Tue Dec 16 17:22:47 2014 -0500
>>
>>       gnu: mesa: Update to 10.4.0.
>>
>>       * gnu/packages/gl.scm (mesa): Update to 10.4.0.  Use current libdrm.  Add
>>         eudev, dri3proto, presentproto, and libxshmfence to inputs.  Add gnu-gettext
>>         to native-inputs.  Add --enable-xa configure flag.  Add new phases
>>         'add-missing-m4-files' and 'patch-create_test_cases'.
>>
>> Are you sure it’s no longer necessary?  Mark?
>
> I removed eudev after reading the release notes for 13.0.0:
>
> http://mesa3d.org/relnotes/13.0.0.html
>
> (very bottom of page)

OK, thanks for confirming.

>> The parts about apply the MIPS patch is good to me.  Perhaps you can
>> already push it as a separate patch to ‘staging’.
>
> I can submit an update for 12.0.4 if the major version bump is too risky
> right now, or bump to 13.0.2 which was released three days ago. WDYT?

Go for 13.0.2 if you think it’s safe enough (the API doesn’t really
change I suppose, it’s “just OpenGL” no?), but better check a couple of
key packages reported by ‘guix package -l mesa’.

Thanks!

Ludo’.

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-12-01 23:02     ` Ludovic Courtès
@ 2016-12-02 11:26       ` Marius Bakke
  2016-12-04 10:17         ` Ricardo Wurmus
  0 siblings, 1 reply; 9+ messages in thread
From: Marius Bakke @ 2016-12-02 11:26 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

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

Ludovic Courtès <ludo@gnu.org> writes:

>>> The parts about apply the MIPS patch is good to me.  Perhaps you can
>>> already push it as a separate patch to ‘staging’.
>>
>> I can submit an update for 12.0.4 if the major version bump is too risky
>> right now, or bump to 13.0.2 which was released three days ago. WDYT?
>
> Go for 13.0.2 if you think it’s safe enough (the API doesn’t really
> change I suppose, it’s “just OpenGL” no?), but better check a couple of
> key packages reported by ‘guix package -l mesa’.

I tried 'mpv' and 'vlc' on a few media files without problems. Mesa 13
brings OpenGL 4.4 support depending on the driver, I've only tested an
older Intel card.. Pushed to staging anyway!

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

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

* Re: [PATCH] gnu: mesa: Update to 13.0.1.
  2016-12-02 11:26       ` Marius Bakke
@ 2016-12-04 10:17         ` Ricardo Wurmus
  2016-12-04 18:05           ` Staging (was [PATCH] gnu: mesa: Update to 13.0.1.) Leo Famulari
  0 siblings, 1 reply; 9+ messages in thread
From: Ricardo Wurmus @ 2016-12-04 10:17 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel


Marius Bakke <mbakke@fastmail.com> writes:

> Pushed to staging anyway!

Just a note: the original plan was to freeze staging on November 30 and
merge on December 7 (Wednesday).  I think it’s time to actually freeze
staging now.

-- 
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
http://elephly.net

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

* Staging (was [PATCH] gnu: mesa: Update to 13.0.1.)
  2016-12-04 10:17         ` Ricardo Wurmus
@ 2016-12-04 18:05           ` Leo Famulari
  2016-12-04 20:52             ` Ludovic Courtès
  0 siblings, 1 reply; 9+ messages in thread
From: Leo Famulari @ 2016-12-04 18:05 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

On Sun, Dec 04, 2016 at 11:17:12AM +0100, Ricardo Wurmus wrote:
> 
> Marius Bakke <mbakke@fastmail.com> writes:
> 
> > Pushed to staging anyway!
> 
> Just a note: the original plan was to freeze staging on November 30 and
> merge on December 7 (Wednesday).  I think it’s time to actually freeze
> staging now.

Is there a reason to wait? Or should we start evaluating staging on
Hydra today?

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

* Re: Staging (was [PATCH] gnu: mesa: Update to 13.0.1.)
  2016-12-04 18:05           ` Staging (was [PATCH] gnu: mesa: Update to 13.0.1.) Leo Famulari
@ 2016-12-04 20:52             ` Ludovic Courtès
  0 siblings, 0 replies; 9+ messages in thread
From: Ludovic Courtès @ 2016-12-04 20:52 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> skribis:

> On Sun, Dec 04, 2016 at 11:17:12AM +0100, Ricardo Wurmus wrote:
>> 
>> Marius Bakke <mbakke@fastmail.com> writes:
>> 
>> > Pushed to staging anyway!
>> 
>> Just a note: the original plan was to freeze staging on November 30 and
>> merge on December 7 (Wednesday).  I think it’s time to actually freeze
>> staging now.
>
> Is there a reason to wait? Or should we start evaluating staging on
> Hydra today?

No there’s no reason to wait indeed!  (I emailed you later today on this
topic.  :-))

Ludo’.

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

end of thread, other threads:[~2016-12-04 20:53 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-30 12:06 [PATCH] gnu: mesa: Update to 13.0.1 Marius Bakke
2016-12-01 14:09 ` Ludovic Courtès
2016-12-01 17:14   ` Marius Bakke
2016-12-01 23:02     ` Ludovic Courtès
2016-12-02 11:26       ` Marius Bakke
2016-12-04 10:17         ` Ricardo Wurmus
2016-12-04 18:05           ` Staging (was [PATCH] gnu: mesa: Update to 13.0.1.) Leo Famulari
2016-12-04 20:52             ` Ludovic Courtès
2016-12-01 18:39   ` [PATCH] gnu: mesa: Update to 13.0.1 Mark H Weaver

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.