* bug#22049: libreoffice compile error
@ 2015-11-29 3:42 rennes
2015-11-29 15:02 ` Ludovic Courtès
0 siblings, 1 reply; 16+ messages in thread
From: rennes @ 2015-11-29 3:42 UTC (permalink / raw)
To: 22049
hi,
i try install libreoffice in guixsd (guix package -i libreoffice) and
return error:
can you help me please.
...
ImathTest: testBoxAlgo.cpp:892: void {anonymous}::boxMatrixTransform():
Assertion `b21 == b2' failed.
/gnu/store/isxqjfaglyfsbcv75y8qbqbph8v28ykr-bash-4.3.39/bin/bash: line
5: 4297 Aborted ${dir}$tst
FAIL: ImathTest
==================
1 of 1 test failed
==================
Makefile:403: recipe for target 'check-TESTS' failed
make[2]: *** [check-TESTS] Error 1
make[2]: Leaving directory
'/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
Makefile:525: recipe for target 'check-am' failed
make[1]: *** [check-am] Error 2
make[1]: Leaving directory
'/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
Makefile:356: recipe for target 'check-recursive' failed
make: *** [check-recursive] Error 1
phase `check' failed after 39.8 seconds
builder for
`/gnu/store/ihwc1rm9ka080x223bf3ajp4d2n1sgnz-ilmbase-2.2.0.drv' failed
with exit code 1
cannot build derivation
`/gnu/store/970sjpar4z0yy99if7p6nm7a4k4nx9zp-vigra-1.10.0.drv': 1
dependencies couldn't be built
cannot build derivation
`/gnu/store/vhbikp9n9kzzjrl9dg2x5aq9xhr38294-libreoffice-5.0.0.2.drv': 1
dependencies couldn't be built
cannot build derivation
`/gnu/store/l7gvchzisln5q0dr4l9px6ikpqp1y9m2-profile.drv': 1
dependencies couldn't be built
guix package: error: build failed: build of
`/gnu/store/l7gvchzisln5q0dr4l9px6ikpqp1y9m2-profile.drv' failed
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-29 3:42 bug#22049: libreoffice compile error rennes
@ 2015-11-29 15:02 ` Ludovic Courtès
2015-11-29 16:50 ` rennes
0 siblings, 1 reply; 16+ messages in thread
From: Ludovic Courtès @ 2015-11-29 15:02 UTC (permalink / raw)
To: rennes; +Cc: 22049
rennes@openmailbox.org skribis:
> i try install libreoffice in guixsd (guix package -i libreoffice) and
> return error:
[...]
> ImathTest: testBoxAlgo.cpp:892: void
> {anonymous}::boxMatrixTransform(): Assertion `b21 == b2' failed.
> /gnu/store/isxqjfaglyfsbcv75y8qbqbph8v28ykr-bash-4.3.39/bin/bash: line
> 5: 4297 Aborted ${dir}$tst
> FAIL: ImathTest
> ==================
> 1 of 1 test failed
> ==================
> Makefile:403: recipe for target 'check-TESTS' failed
> make[2]: *** [check-TESTS] Error 1
> make[2]: Leaving directory
> '/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
What platform is this on? i686, x86_64, armhf, or mips64el?
Thanks for your report!
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-29 15:02 ` Ludovic Courtès
@ 2015-11-29 16:50 ` rennes
2015-11-30 19:56 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: rennes @ 2015-11-29 16:50 UTC (permalink / raw)
To: ludo; +Cc: 22049
On 2015-11-29 09:02, ludo@gnu.org wrote:
> rennes@openmailbox.org skribis:
>
>> i try install libreoffice in guixsd (guix package -i libreoffice) and
>> return error:
>
> [...]
>
>> ImathTest: testBoxAlgo.cpp:892: void
>> {anonymous}::boxMatrixTransform(): Assertion `b21 == b2' failed.
>> /gnu/store/isxqjfaglyfsbcv75y8qbqbph8v28ykr-bash-4.3.39/bin/bash: line
>> 5: 4297 Aborted ${dir}$tst
>> FAIL: ImathTest
>> ==================
>> 1 of 1 test failed
>> ==================
>> Makefile:403: recipe for target 'check-TESTS' failed
>> make[2]: *** [check-TESTS] Error 1
>> make[2]: Leaving directory
>> '/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
>
> What platform is this on? i686, x86_64, armhf, or mips64el?
>
> Thanks for your report!
>
> Ludo’.
Hi Ludo,
i use i686 in virtualbox.
and thanks.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-29 16:50 ` rennes
@ 2015-11-30 19:56 ` Leo Famulari
2015-11-30 20:14 ` Ludovic Courtès
2016-02-28 18:06 ` Ludovic Courtès
0 siblings, 2 replies; 16+ messages in thread
From: Leo Famulari @ 2015-11-30 19:56 UTC (permalink / raw)
To: rennes; +Cc: 22049
On Sun, Nov 29, 2015 at 10:50:22AM -0600, rennes@openmailbox.org wrote:
> On 2015-11-29 09:02, ludo@gnu.org wrote:
> >rennes@openmailbox.org skribis:
> >
> >>i try install libreoffice in guixsd (guix package -i libreoffice) and
> >>return error:
> >
> >[...]
> >
> >>ImathTest: testBoxAlgo.cpp:892: void
> >>{anonymous}::boxMatrixTransform(): Assertion `b21 == b2' failed.
> >>/gnu/store/isxqjfaglyfsbcv75y8qbqbph8v28ykr-bash-4.3.39/bin/bash: line
> >>5: 4297 Aborted ${dir}$tst
> >>FAIL: ImathTest
> >>==================
> >>1 of 1 test failed
> >>==================
> >>Makefile:403: recipe for target 'check-TESTS' failed
> >>make[2]: *** [check-TESTS] Error 1
> >>make[2]: Leaving directory
> >>'/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
> >
> >What platform is this on? i686, x86_64, armhf, or mips64el?
> >
> >Thanks for your report!
> >
> >Ludo’.
>
>
>
> Hi Ludo,
>
> i use i686 in virtualbox.
>
> and thanks.
It looks like the problem is with a libreoffice dependency, ilmbase.
Looking at the build history on hydra.gnu.org, it hasn't built
successfully since at least 2015-02-25:
http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
Here is how to view the most recent build log from the command line:
$ guix build --system=i686-linux --log-file ilmbase
http://hydra.gnu.org/log/dpbr03zii79ff3cbin9c6pqd9fh4sc1c-ilmbase-2.2.0
And here it can be viewed in a browser:
http://hydra.gnu.org/build/767427/nixlog/1
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-30 19:56 ` Leo Famulari
@ 2015-11-30 20:14 ` Ludovic Courtès
2015-12-01 5:58 ` Leo Famulari
2016-02-28 18:06 ` Ludovic Courtès
1 sibling, 1 reply; 16+ messages in thread
From: Ludovic Courtès @ 2015-11-30 20:14 UTC (permalink / raw)
To: Leo Famulari; +Cc: rennes, 22049
Leo Famulari <leo@famulari.name> skribis:
> It looks like the problem is with a libreoffice dependency, ilmbase.
> Looking at the build history on hydra.gnu.org, it hasn't built
> successfully since at least 2015-02-25:
> http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
Right. This sounds very much like a rounding issue, whereby the epsilon
in floating-point number comparisons is to strict for 32-bit machines.
We should report it upstream, but I don’t see any clear way to report
bugs, neither in README, nor on openexr.com. Am I overlooking it?
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-30 20:14 ` Ludovic Courtès
@ 2015-12-01 5:58 ` Leo Famulari
2015-12-03 20:01 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2015-12-01 5:58 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
On Mon, Nov 30, 2015 at 10:14:49PM +0200, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
>
> > It looks like the problem is with a libreoffice dependency, ilmbase.
> > Looking at the build history on hydra.gnu.org, it hasn't built
> > successfully since at least 2015-02-25:
> > http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
>
> Right. This sounds very much like a rounding issue, whereby the epsilon
> in floating-point number comparisons is to strict for 32-bit machines.
>
> We should report it upstream, but I don’t see any clear way to report
> bugs, neither in README, nor on openexr.com. Am I overlooking it?
openexr-devel appears to be the main list:
https://lists.gnu.org/mailman/listinfo/openexr-devel
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-12-01 5:58 ` Leo Famulari
@ 2015-12-03 20:01 ` Leo Famulari
2015-12-04 11:59 ` Ludovic Courtès
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2015-12-03 20:01 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
On Tue, Dec 01, 2015 at 12:58:27AM -0500, Leo Famulari wrote:
> On Mon, Nov 30, 2015 at 10:14:49PM +0200, Ludovic Courtès wrote:
> > Leo Famulari <leo@famulari.name> skribis:
> >
> > > It looks like the problem is with a libreoffice dependency, ilmbase.
> > > Looking at the build history on hydra.gnu.org, it hasn't built
> > > successfully since at least 2015-02-25:
> > > http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
> >
> > Right. This sounds very much like a rounding issue, whereby the epsilon
> > in floating-point number comparisons is to strict for 32-bit machines.
> >
> > We should report it upstream, but I don’t see any clear way to report
> > bugs, neither in README, nor on openexr.com. Am I overlooking it?
>
> openexr-devel appears to be the main list:
> https://lists.gnu.org/mailman/listinfo/openexr-devel
I'm happy to send a bug report to openexr-devel but I don't understand
the issue well enough to write a decent report. Would it be sufficient to
just copy your message, Ludovic? Along with information about the
platform, of course.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-12-03 20:01 ` Leo Famulari
@ 2015-12-04 11:59 ` Ludovic Courtès
0 siblings, 0 replies; 16+ messages in thread
From: Ludovic Courtès @ 2015-12-04 11:59 UTC (permalink / raw)
To: Leo Famulari; +Cc: rennes, 22049
Leo Famulari <leo@famulari.name> skribis:
> On Tue, Dec 01, 2015 at 12:58:27AM -0500, Leo Famulari wrote:
>> On Mon, Nov 30, 2015 at 10:14:49PM +0200, Ludovic Courtès wrote:
>> > Leo Famulari <leo@famulari.name> skribis:
>> >
>> > > It looks like the problem is with a libreoffice dependency, ilmbase.
>> > > Looking at the build history on hydra.gnu.org, it hasn't built
>> > > successfully since at least 2015-02-25:
>> > > http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
>> >
>> > Right. This sounds very much like a rounding issue, whereby the epsilon
>> > in floating-point number comparisons is to strict for 32-bit machines.
>> >
>> > We should report it upstream, but I don’t see any clear way to report
>> > bugs, neither in README, nor on openexr.com. Am I overlooking it?
>>
>> openexr-devel appears to be the main list:
>> https://lists.gnu.org/mailman/listinfo/openexr-devel
>
> I'm happy to send a bug report to openexr-devel but I don't understand
> the issue well enough to write a decent report. Would it be sufficient to
> just copy your message, Ludovic? Along with information about the
> platform, of course.
I would suggest giving them as much information as possible about the
platform, including versions of dependencies, and an excerpt from the
build log (probably with a link to the full log on hydra.gnu.org.)
So, thanks in advance! :-)
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2015-11-30 19:56 ` Leo Famulari
2015-11-30 20:14 ` Ludovic Courtès
@ 2016-02-28 18:06 ` Ludovic Courtès
2016-02-29 8:35 ` Leo Famulari
1 sibling, 1 reply; 16+ messages in thread
From: Ludovic Courtès @ 2016-02-28 18:06 UTC (permalink / raw)
To: Leo Famulari; +Cc: rennes, 22049
Leo Famulari <leo@famulari.name> skribis:
> On Sun, Nov 29, 2015 at 10:50:22AM -0600, rennes@openmailbox.org wrote:
>> On 2015-11-29 09:02, ludo@gnu.org wrote:
>> >rennes@openmailbox.org skribis:
>> >
>> >>i try install libreoffice in guixsd (guix package -i libreoffice) and
>> >>return error:
>> >
>> >[...]
>> >
>> >>ImathTest: testBoxAlgo.cpp:892: void
>> >>{anonymous}::boxMatrixTransform(): Assertion `b21 == b2' failed.
>> >>/gnu/store/isxqjfaglyfsbcv75y8qbqbph8v28ykr-bash-4.3.39/bin/bash: line
>> >>5: 4297 Aborted ${dir}$tst
>> >>FAIL: ImathTest
>> >>==================
>> >>1 of 1 test failed
>> >>==================
>> >>Makefile:403: recipe for target 'check-TESTS' failed
>> >>make[2]: *** [check-TESTS] Error 1
>> >>make[2]: Leaving directory
>> >>'/tmp/nix-build-ilmbase-2.2.0.drv-0/ilmbase-2.2.0/ImathTest'
>> >
>> >What platform is this on? i686, x86_64, armhf, or mips64el?
>> >
>> >Thanks for your report!
>> >
>> >Ludo’.
>>
>>
>>
>> Hi Ludo,
>>
>> i use i686 in virtualbox.
>>
>> and thanks.
>
> It looks like the problem is with a libreoffice dependency, ilmbase.
> Looking at the build history on hydra.gnu.org, it hasn't built
> successfully since at least 2015-02-25:
> http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
The problem appears to be fixed in IlmBase 2.0.1:
https://github.com/openexr/openexr/issues/3
Would you like to try?
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-02-28 18:06 ` Ludovic Courtès
@ 2016-02-29 8:35 ` Leo Famulari
2016-02-29 8:37 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2016-02-29 8:35 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
On Sun, Feb 28, 2016 at 07:06:22PM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
> > It looks like the problem is with a libreoffice dependency, ilmbase.
> > Looking at the build history on hydra.gnu.org, it hasn't built
> > successfully since at least 2015-02-25:
> > http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
>
> The problem appears to be fixed in IlmBase 2.0.1:
>
> https://github.com/openexr/openexr/issues/3
>
> Would you like to try?
We currently package 2.2.0. I did read that bug report, and then kept
reading, until I came to the Debian bug report [0], where Steven
Chamberlain linked to our bug report with OpenEXR and produced two
patches that Debian has merged into their "unstable" branch.
I've attached my (working) attempt at conditionally applying these
patches on i686-* architectures. Please feel free to criticize or modify
this patch :)
I copied the patches from [1].
Thanks for the reminder!
[0]
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
[1]
https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/
[2]
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712#194
Relevant commit in Debian ilmbase packaging repo:
https://anonscm.debian.org/cgit/pkg-phototools/ilmbase.git/commit/?id=ab28bb45cdad8adc32e345b777ab8e692b1d9a9c
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-02-29 8:35 ` Leo Famulari
@ 2016-02-29 8:37 ` Leo Famulari
2016-03-01 13:36 ` Ludovic Courtès
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2016-02-29 8:37 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
[-- Attachment #1: Type: text/plain, Size: 1513 bytes --]
On Mon, Feb 29, 2016 at 03:35:40AM -0500, Leo Famulari wrote:
> On Sun, Feb 28, 2016 at 07:06:22PM +0100, Ludovic Courtès wrote:
> > Leo Famulari <leo@famulari.name> skribis:
> > > It looks like the problem is with a libreoffice dependency, ilmbase.
> > > Looking at the build history on hydra.gnu.org, it hasn't built
> > > successfully since at least 2015-02-25:
> > > http://hydra.gnu.org/job/gnu/master/ilmbase-2.2.0.i686-linux
> >
> > The problem appears to be fixed in IlmBase 2.0.1:
> >
> > https://github.com/openexr/openexr/issues/3
> >
> > Would you like to try?
>
> We currently package 2.2.0. I did read that bug report, and then kept
> reading, until I came to the Debian bug report [0], where Steven
> Chamberlain linked to our bug report with OpenEXR and produced two
> patches that Debian has merged into their "unstable" branch.
>
> I've attached my (working) attempt at conditionally applying these
> patches on i686-* architectures. Please feel free to criticize or modify
> this patch :)
I forgot the attachment. Here it is!
>
> I copied the patches from [1].
>
> Thanks for the reminder!
>
> [0]
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
>
> [1]
> https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/
>
> [2]
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712#194
>
> Relevant commit in Debian ilmbase packaging repo:
> https://anonscm.debian.org/cgit/pkg-phototools/ilmbase.git/commit/?id=ab28bb45cdad8adc32e345b777ab8e692b1d9a9c
>
>
>
[-- Attachment #2: 0001-gnu-ilmbase-Add-patches-to-fix-build-on-i686.patch --]
[-- Type: text/x-diff, Size: 9052 bytes --]
From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
From: Leo Famulari <leo@famulari.name>
Date: Mon, 29 Feb 2016 02:23:43 -0500
Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
Fixes <http://bugs.gnu.org/22049>.
* gnu/packages/patches/ilmbase-testBox.patch,
gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
* gnu-system.am (dist_patch_DATA): Add them.
* gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
and patch/testBoxAlgo.
[arguments]: Add 'patch-for-i686' phase.
---
gnu-system.am | 2 +
gnu/packages/graphics.scm | 29 ++++++
gnu/packages/patches/ilmbase-testBox.patch | 121 +++++++++++++++++++++++++
gnu/packages/patches/ilmbase-testBoxAlgo.patch | 41 +++++++++
4 files changed, 193 insertions(+)
create mode 100644 gnu/packages/patches/ilmbase-testBox.patch
create mode 100644 gnu/packages/patches/ilmbase-testBoxAlgo.patch
diff --git a/gnu-system.am b/gnu-system.am
index 502993b..42673c5 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -529,6 +529,8 @@ dist_patch_DATA = \
gnu/packages/patches/icu4c-CVE-2014-6585.patch \
gnu/packages/patches/icu4c-CVE-2015-1270.patch \
gnu/packages/patches/icu4c-CVE-2015-4760.patch \
+ gnu/packages/patches/ilmbase-testBox.patch \
+ gnu/packages/patches/ilmbase-testBoxAlgo.patch \
gnu/packages/patches/imagemagick-test-segv.patch \
gnu/packages/patches/irrlicht-mesa-10.patch \
gnu/packages/patches/jasper-CVE-2007-2721.patch \
diff --git a/gnu/packages/graphics.scm b/gnu/packages/graphics.scm
index 560febe..0d99b4e 100644
--- a/gnu/packages/graphics.scm
+++ b/gnu/packages/graphics.scm
@@ -1,6 +1,7 @@
;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2015 Ludovic Courtès <ludo@gnu.org>
;;; Copyright © 2015 Tomáš Čech <sleep_walker@gnu.org>
+;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -91,6 +92,34 @@ many more.")
(base32
"1izddjwbh1grs8080vmaix72z469qy29wrvkphgmqmcm0sv1by7c"))))
(build-system gnu-build-system)
+
+ ;; ilmbase fails to build from source on the i686 architecture due to a
+ ;; rounding issue. These patches are applied only when building on or for
+ ;; i686.
+
+ ;; Please refer to:
+ ;; https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+ ;; https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
+ (native-inputs
+ (if (string-prefix? "i686" (or (%current-target-system) (%current-system)))
+ `(("patch/testBox"
+ ,(search-patch "ilmbase-testBox.patch"))
+ ("patch/testBoxAlgo"
+ ,(search-patch "ilmbase-testBoxAlgo.patch")))
+ '()))
+ (arguments
+ (if (string-prefix? "i686" (or (%current-target-system) (%current-system)))
+ `(#:phases
+ (modify-phases %standard-phases
+ (add-after 'unpack 'patch-for-i686
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((patches (list (assoc-ref inputs "patch/testBox")
+ (assoc-ref inputs "patch/testBoxAlgo"))))
+ (and (map (lambda (patches)
+ (zero?
+ (system* "patch" "-p1" "--input" patches)))
+ patches)))))))
+ '()))
(home-page "http://www.openexr.com/")
(synopsis "Utility C++ libraries for threads, maths, and exceptions")
(description
diff --git a/gnu/packages/patches/ilmbase-testBox.patch b/gnu/packages/patches/ilmbase-testBox.patch
new file mode 100644
index 0000000..a947df3
--- /dev/null
+++ b/gnu/packages/patches/ilmbase-testBox.patch
@@ -0,0 +1,121 @@
+Fix FTBFS on i686-linux due to rounding issue (see references).
+
+Fixes Guix bug #22049 (see below).
+
+Copied from Debian.
+
+Source:
+https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/testBoxAlgo.patch/
+
+References:
+https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+https://debbugs.gnu.org/cgi/bugreport.cgi?bug=22049
+https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
+https://anonscm.debian.org/cgit/pkg-phototools/ilmbase.git/commit/?id=ab28bb45cdad8adc32e345b777ab8e692b1d9a9c
+
+---
+
+Subject: testBox: allow fuzzy comparison of floats, doubles
+From: Steven Chamberlain <steven@pyro.eu.org>
+Date: Wed, 24 Feb 2016 01:10:11 +0000
+
+Allow for inexact values, as long as the error is smaller than the
+epsilon of the data type.
+
+On 32-bit x86, allow even greater discrepency at double
+precision, due to possible double-rounding. See
+https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+
+Index: ilmbase/ImathTest/testBox.cpp
+===================================================================
+--- ilmbase.orig/ImathTest/testBox.cpp
++++ ilmbase/ImathTest/testBox.cpp
+@@ -47,6 +47,58 @@ using namespace IMATH_INTERNAL_NAMESPACE
+
+ namespace {
+
++template <class T>
++bool
++approximatelyEqual (const T &p1, const T &p2)
++{
++ /* int and short should be exact */
++ return (p1 == p2);
++}
++
++bool
++approximatelyEqual (const Vec2<float> &p1, const Vec2<float> &p2)
++{
++ float e = limits<float>::epsilon();
++ float m = 0;
++
++ for (int i = 0; i < 2; ++i)
++ {
++ m = max (m, abs (p1[i]));
++ m = max (m, abs (p2[i]));
++ }
++
++ for (int i = 0; i < 2; ++i)
++ if (!equalWithAbsError (p1[i], p2[i], m * e))
++ return false;
++
++ return true;
++}
++
++bool
++approximatelyEqual (const Vec2<double> &p1, const Vec2<double> &p2)
++{
++#if defined(__i386__) || defined(_M_IX86)
++ /* double-rounding on 32-bit x86 may cause larger error:
++ use epsilon of float rather than double */
++ double e = limits<float>::epsilon();
++#else
++ double e = limits<double>::epsilon();
++#endif
++ double m = 0;
++
++ for (int i = 0; i < 2; ++i)
++ {
++ m = max (m, abs (p1[i]));
++ m = max (m, abs (p2[i]));
++ }
++
++ for (int i = 0; i < 2; ++i)
++ if (!equalWithAbsError (p1[i], p2[i], m * e))
++ return false;
++
++ return true;
++}
++
+ //
+ // Test case generation utility - create a vector of IMATH_INTERNAL_NAMESPACE::Vec{2,3,4}
+ // with all permutations of integers 1..T::dimensions().
+@@ -250,7 +302,8 @@ testExtendByPoint(const char *type)
+
+ IMATH_INTERNAL_NAMESPACE::Box<T> b;
+ b.extendBy(p);
+- assert(b.min == p && b.max == p);
++ assert (approximatelyEqual (b.min, p));
++ assert (approximatelyEqual (b.max, p));
+ }
+
+ //
+@@ -283,7 +336,8 @@ testExtendByPoint(const char *type)
+
+ b.extendBy(p);
+
+- assert(b.min == min && b.max == max);
++ assert (approximatelyEqual (b.min, min));
++ assert (approximatelyEqual (b.max, max));
+ }
+ }
+ }
+@@ -358,7 +412,8 @@ testExtendByBox(const char *type)
+ }
+ b.extendBy(IMATH_INTERNAL_NAMESPACE::Box<T>(p0, p1));
+
+- assert(b.min == min && b.max == max);
++ assert (approximatelyEqual (b.min, min));
++ assert (approximatelyEqual (b.max, max));
+ }
+ }
+ }
diff --git a/gnu/packages/patches/ilmbase-testBoxAlgo.patch b/gnu/packages/patches/ilmbase-testBoxAlgo.patch
new file mode 100644
index 0000000..06752e3
--- /dev/null
+++ b/gnu/packages/patches/ilmbase-testBoxAlgo.patch
@@ -0,0 +1,41 @@
+Fix FTBFS on i686-linux due to rounding issue (see references).
+
+Fixes Guix bug #22049 (see below).
+
+Copied from Debian.
+
+Source:
+https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/testBox.patch/
+
+References:
+https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+https://debbugs.gnu.org/cgi/bugreport.cgi?bug=22049
+https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
+https://anonscm.debian.org/cgit/pkg-phototools/ilmbase.git/commit/?id=ab28bb45cdad8adc32e345b777ab8e692b1d9a9c
+
+---
+
+Subject: testBoxAlgo: allow fuzzy match of b12 == b2
+From: Steven Chamberlain <steven@pyro.eu.org>
+Date: Wed, 24 Feb 2016 01:04:11 +0000
+
+Also fix a pre-existing typo.
+
+Index: ilmbase/ImathTest/testBoxAlgo.cpp
+===================================================================
+--- ilmbase.orig/ImathTest/testBoxAlgo.cpp
++++ ilmbase/ImathTest/testBoxAlgo.cpp
+@@ -886,10 +886,11 @@ boxMatrixTransform ()
+
+ assert (approximatelyEqual (b2.min, b4.min, e));
+ assert (approximatelyEqual (b2.max, b4.max, e));
+- assert (approximatelyEqual (b3.max, b4.max, e));
++ assert (approximatelyEqual (b3.min, b4.min, e));
+ assert (approximatelyEqual (b3.max, b4.max, e));
+
+- assert (b21 == b2);
++ assert (approximatelyEqual (b2.min, b21.min, e));
++ assert (approximatelyEqual (b2.max, b21.max, e));
+ assert (b31 == b3);
+
+ M[0][3] = 1;
--
2.7.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-02-29 8:37 ` Leo Famulari
@ 2016-03-01 13:36 ` Ludovic Courtès
2016-03-01 17:24 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: Ludovic Courtès @ 2016-03-01 13:36 UTC (permalink / raw)
To: Leo Famulari; +Cc: rennes, 22049
Leo Famulari <leo@famulari.name> skribis:
> From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
> Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
> From: Leo Famulari <leo@famulari.name>
> Date: Mon, 29 Feb 2016 02:23:43 -0500
> Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
>
> Fixes <http://bugs.gnu.org/22049>.
>
> * gnu/packages/patches/ilmbase-testBox.patch,
> gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
> * gnu-system.am (dist_patch_DATA): Add them.
> * gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
> and patch/testBoxAlgo.
> [arguments]: Add 'patch-for-i686' phase.
Cool!
I think it’d be reasonable to squash both patches in one file, and to
apply it unconditionally. WDYT?
So I gather upgrading to IlmBase 2.0.1 is not an option?
Thanks,
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-03-01 13:36 ` Ludovic Courtès
@ 2016-03-01 17:24 ` Leo Famulari
2016-03-01 18:01 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2016-03-01 17:24 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
On Tue, Mar 01, 2016 at 02:36:24PM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
>
> > From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
> > Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
> > From: Leo Famulari <leo@famulari.name>
> > Date: Mon, 29 Feb 2016 02:23:43 -0500
> > Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
> >
> > Fixes <http://bugs.gnu.org/22049>.
> >
> > * gnu/packages/patches/ilmbase-testBox.patch,
> > gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
> > * gnu-system.am (dist_patch_DATA): Add them.
> > * gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
> > and patch/testBoxAlgo.
> > [arguments]: Add 'patch-for-i686' phase.
>
> Cool!
>
> I think it’d be reasonable to squash both patches in one file, and to
> apply it unconditionally. WDYT?
Sure, I'll send an updated patch.
I'm wondering — does the current patch handle the conditional
application of the patch properly? Just looking for some feedback on the
approach...
>
> So I gather upgrading to IlmBase 2.0.1 is not an option?
2.0.1 would be a downgrade for us; we are on 2.2.0. I did try it without
success.
>
> Thanks,
> Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-03-01 17:24 ` Leo Famulari
@ 2016-03-01 18:01 ` Leo Famulari
2016-03-01 20:36 ` Ludovic Courtès
0 siblings, 1 reply; 16+ messages in thread
From: Leo Famulari @ 2016-03-01 18:01 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049
[-- Attachment #1: Type: text/plain, Size: 1433 bytes --]
On Tue, Mar 01, 2016 at 12:24:45PM -0500, Leo Famulari wrote:
> On Tue, Mar 01, 2016 at 02:36:24PM +0100, Ludovic Courtès wrote:
> > Leo Famulari <leo@famulari.name> skribis:
> >
> > > From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
> > > Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
> > > From: Leo Famulari <leo@famulari.name>
> > > Date: Mon, 29 Feb 2016 02:23:43 -0500
> > > Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
> > >
> > > Fixes <http://bugs.gnu.org/22049>.
> > >
> > > * gnu/packages/patches/ilmbase-testBox.patch,
> > > gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
> > > * gnu-system.am (dist_patch_DATA): Add them.
> > > * gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
> > > and patch/testBoxAlgo.
> > > [arguments]: Add 'patch-for-i686' phase.
> >
> > Cool!
> >
> > I think it’d be reasonable to squash both patches in one file, and to
> > apply it unconditionally. WDYT?
>
> Sure, I'll send an updated patch.
Attached.
>
> I'm wondering — does the current patch handle the conditional
> application of the patch properly? Just looking for some feedback on the
> approach...
>
> >
> > So I gather upgrading to IlmBase 2.0.1 is not an option?
>
> 2.0.1 would be a downgrade for us; we are on 2.2.0. I did try it without
> success.
>
> >
> > Thanks,
> > Ludo’.
>
>
>
[-- Attachment #2: 0001-gnu-ilmbase-Add-patch-to-fix-build-on-i686.patch --]
[-- Type: text/x-diff, Size: 6893 bytes --]
From 2f895108a3f476038b82e03645ee7f51e4ee1675 Mon Sep 17 00:00:00 2001
From: Leo Famulari <leo@famulari.name>
Date: Mon, 29 Feb 2016 02:23:43 -0500
Subject: [PATCH] gnu: ilmbase: Add patch to fix build on i686.
Fixes <http://bugs.gnu.org/22049>.
* gnu/packages/patches/ilmbase-fix-tests.patch: New file.
* gnu-system.am (dist_patch_DATA): Add it.
* gnu/packages/graphics.scm (ilmbase)[source]: Add patch.
---
gnu-system.am | 1 +
gnu/packages/graphics.scm | 4 +-
gnu/packages/patches/ilmbase-fix-tests.patch | 149 +++++++++++++++++++++++++++
3 files changed, 153 insertions(+), 1 deletion(-)
create mode 100644 gnu/packages/patches/ilmbase-fix-tests.patch
diff --git a/gnu-system.am b/gnu-system.am
index 7cfc485..5e2be0b 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -529,6 +529,7 @@ dist_patch_DATA = \
gnu/packages/patches/icu4c-CVE-2014-6585.patch \
gnu/packages/patches/icu4c-CVE-2015-1270.patch \
gnu/packages/patches/icu4c-CVE-2015-4760.patch \
+ gnu/packages/patches/ilmbase-fix-texts.patch \
gnu/packages/patches/imagemagick-test-segv.patch \
gnu/packages/patches/irrlicht-mesa-10.patch \
gnu/packages/patches/jasper-CVE-2007-2721.patch \
diff --git a/gnu/packages/graphics.scm b/gnu/packages/graphics.scm
index 560febe..3fdf07d 100644
--- a/gnu/packages/graphics.scm
+++ b/gnu/packages/graphics.scm
@@ -1,6 +1,7 @@
;;; GNU Guix --- Functional package management for GNU
;;; Copyright © 2015 Ludovic Courtès <ludo@gnu.org>
;;; Copyright © 2015 Tomáš Čech <sleep_walker@gnu.org>
+;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -89,7 +90,8 @@ many more.")
version ".tar.gz"))
(sha256
(base32
- "1izddjwbh1grs8080vmaix72z469qy29wrvkphgmqmcm0sv1by7c"))))
+ "1izddjwbh1grs8080vmaix72z469qy29wrvkphgmqmcm0sv1by7c"))
+ (patches (map search-patch '("ilmbase-fix-tests.patch")))))
(build-system gnu-build-system)
(home-page "http://www.openexr.com/")
(synopsis "Utility C++ libraries for threads, maths, and exceptions")
diff --git a/gnu/packages/patches/ilmbase-fix-tests.patch b/gnu/packages/patches/ilmbase-fix-tests.patch
new file mode 100644
index 0000000..fcaa37d
--- /dev/null
+++ b/gnu/packages/patches/ilmbase-fix-tests.patch
@@ -0,0 +1,149 @@
+Fix FTBFS on i686-linux due to rounding issue (see references).
+
+Fixes Guix bug #22049 (see below).
+
+Copied from Debian.
+
+Source:
+https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/testBoxAlgo.patch/
+https://sources.debian.net/src/ilmbase/2.2.0-11/debian/patches/testBox.patch/
+
+References:
+https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+https://debbugs.gnu.org/cgi/bugreport.cgi?bug=22049
+https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=815712
+https://anonscm.debian.org/cgit/pkg-phototools/ilmbase.git/commit/?id=ab28bb45cdad8adc32e345b777ab8e692b1d9a9c
+
+---
+
+Subject: testBoxAlgo: allow fuzzy match of b12 == b2
+From: Steven Chamberlain <steven@pyro.eu.org>
+Date: Wed, 24 Feb 2016 01:04:11 +0000
+
+Also fix a pre-existing typo.
+
+Index: ilmbase/ImathTest/testBoxAlgo.cpp
+===================================================================
+--- ilmbase.orig/ImathTest/testBoxAlgo.cpp
++++ ilmbase/ImathTest/testBoxAlgo.cpp
+@@ -886,10 +886,11 @@ boxMatrixTransform ()
+
+ assert (approximatelyEqual (b2.min, b4.min, e));
+ assert (approximatelyEqual (b2.max, b4.max, e));
+- assert (approximatelyEqual (b3.max, b4.max, e));
++ assert (approximatelyEqual (b3.min, b4.min, e));
+ assert (approximatelyEqual (b3.max, b4.max, e));
+
+- assert (b21 == b2);
++ assert (approximatelyEqual (b2.min, b21.min, e));
++ assert (approximatelyEqual (b2.max, b21.max, e));
+ assert (b31 == b3);
+
+ M[0][3] = 1;
+
+---
+
+Subject: testBox: allow fuzzy comparison of floats, doubles
+From: Steven Chamberlain <steven@pyro.eu.org>
+Date: Wed, 24 Feb 2016 01:10:11 +0000
+
+Allow for inexact values, as long as the error is smaller than the
+epsilon of the data type.
+
+On 32-bit x86, allow even greater discrepency at double
+precision, due to possible double-rounding. See
+https://lists.nongnu.org/archive/html/openexr-devel/2015-12/msg00001.html
+
+Index: ilmbase/ImathTest/testBox.cpp
+===================================================================
+--- ilmbase.orig/ImathTest/testBox.cpp
++++ ilmbase/ImathTest/testBox.cpp
+@@ -47,6 +47,58 @@ using namespace IMATH_INTERNAL_NAMESPACE
+
+ namespace {
+
++template <class T>
++bool
++approximatelyEqual (const T &p1, const T &p2)
++{
++ /* int and short should be exact */
++ return (p1 == p2);
++}
++
++bool
++approximatelyEqual (const Vec2<float> &p1, const Vec2<float> &p2)
++{
++ float e = limits<float>::epsilon();
++ float m = 0;
++
++ for (int i = 0; i < 2; ++i)
++ {
++ m = max (m, abs (p1[i]));
++ m = max (m, abs (p2[i]));
++ }
++
++ for (int i = 0; i < 2; ++i)
++ if (!equalWithAbsError (p1[i], p2[i], m * e))
++ return false;
++
++ return true;
++}
++
++bool
++approximatelyEqual (const Vec2<double> &p1, const Vec2<double> &p2)
++{
++#if defined(__i386__) || defined(_M_IX86)
++ /* double-rounding on 32-bit x86 may cause larger error:
++ use epsilon of float rather than double */
++ double e = limits<float>::epsilon();
++#else
++ double e = limits<double>::epsilon();
++#endif
++ double m = 0;
++
++ for (int i = 0; i < 2; ++i)
++ {
++ m = max (m, abs (p1[i]));
++ m = max (m, abs (p2[i]));
++ }
++
++ for (int i = 0; i < 2; ++i)
++ if (!equalWithAbsError (p1[i], p2[i], m * e))
++ return false;
++
++ return true;
++}
++
+ //
+ // Test case generation utility - create a vector of IMATH_INTERNAL_NAMESPACE::Vec{2,3,4}
+ // with all permutations of integers 1..T::dimensions().
+@@ -250,7 +302,8 @@ testExtendByPoint(const char *type)
+
+ IMATH_INTERNAL_NAMESPACE::Box<T> b;
+ b.extendBy(p);
+- assert(b.min == p && b.max == p);
++ assert (approximatelyEqual (b.min, p));
++ assert (approximatelyEqual (b.max, p));
+ }
+
+ //
+@@ -283,7 +336,8 @@ testExtendByPoint(const char *type)
+
+ b.extendBy(p);
+
+- assert(b.min == min && b.max == max);
++ assert (approximatelyEqual (b.min, min));
++ assert (approximatelyEqual (b.max, max));
+ }
+ }
+ }
+@@ -358,7 +412,8 @@ testExtendByBox(const char *type)
+ }
+ b.extendBy(IMATH_INTERNAL_NAMESPACE::Box<T>(p0, p1));
+
+- assert(b.min == min && b.max == max);
++ assert (approximatelyEqual (b.min, min));
++ assert (approximatelyEqual (b.max, max));
+ }
+ }
+ }
--
2.7.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-03-01 18:01 ` Leo Famulari
@ 2016-03-01 20:36 ` Ludovic Courtès
2016-03-02 0:20 ` Leo Famulari
0 siblings, 1 reply; 16+ messages in thread
From: Ludovic Courtès @ 2016-03-01 20:36 UTC (permalink / raw)
To: Leo Famulari; +Cc: rennes, 22049
Leo Famulari <leo@famulari.name> skribis:
> On Tue, Mar 01, 2016 at 12:24:45PM -0500, Leo Famulari wrote:
>> On Tue, Mar 01, 2016 at 02:36:24PM +0100, Ludovic Courtès wrote:
>> > Leo Famulari <leo@famulari.name> skribis:
>> >
>> > > From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
>> > > Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
>> > > From: Leo Famulari <leo@famulari.name>
>> > > Date: Mon, 29 Feb 2016 02:23:43 -0500
>> > > Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
>> > >
>> > > Fixes <http://bugs.gnu.org/22049>.
>> > >
>> > > * gnu/packages/patches/ilmbase-testBox.patch,
>> > > gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
>> > > * gnu-system.am (dist_patch_DATA): Add them.
>> > > * gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
>> > > and patch/testBoxAlgo.
>> > > [arguments]: Add 'patch-for-i686' phase.
>> >
>> > Cool!
>> >
>> > I think it’d be reasonable to squash both patches in one file, and to
>> > apply it unconditionally. WDYT?
>>
>> Sure, I'll send an updated patch.
>
> Attached.
>
[...]
>> > So I gather upgrading to IlmBase 2.0.1 is not an option?
>>
>> 2.0.1 would be a downgrade for us; we are on 2.2.0. I did try it without
>> success.
Oops, sorry. :-)
> From 2f895108a3f476038b82e03645ee7f51e4ee1675 Mon Sep 17 00:00:00 2001
> From: Leo Famulari <leo@famulari.name>
> Date: Mon, 29 Feb 2016 02:23:43 -0500
> Subject: [PATCH] gnu: ilmbase: Add patch to fix build on i686.
>
> Fixes <http://bugs.gnu.org/22049>.
>
> * gnu/packages/patches/ilmbase-fix-tests.patch: New file.
> * gnu-system.am (dist_patch_DATA): Add it.
> * gnu/packages/graphics.scm (ilmbase)[source]: Add patch.
Perfect, thanks again!
Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
* bug#22049: libreoffice compile error
2016-03-01 20:36 ` Ludovic Courtès
@ 2016-03-02 0:20 ` Leo Famulari
0 siblings, 0 replies; 16+ messages in thread
From: Leo Famulari @ 2016-03-02 0:20 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: rennes, 22049-done
On Tue, Mar 01, 2016 at 09:36:53PM +0100, Ludovic Courtès wrote:
> Leo Famulari <leo@famulari.name> skribis:
>
> > On Tue, Mar 01, 2016 at 12:24:45PM -0500, Leo Famulari wrote:
> >> On Tue, Mar 01, 2016 at 02:36:24PM +0100, Ludovic Courtès wrote:
> >> > Leo Famulari <leo@famulari.name> skribis:
> >> >
> >> > > From bf1f2a1c3621ba93ec99711ec78a79663acb6e82 Mon Sep 17 00:00:00 2001
> >> > > Message-Id: <bf1f2a1c3621ba93ec99711ec78a79663acb6e82.1456734966.git.leo@famulari.name>
> >> > > From: Leo Famulari <leo@famulari.name>
> >> > > Date: Mon, 29 Feb 2016 02:23:43 -0500
> >> > > Subject: [PATCH 1/1] gnu: ilmbase: Add patches to fix build on i686.
> >> > >
> >> > > Fixes <http://bugs.gnu.org/22049>.
> >> > >
> >> > > * gnu/packages/patches/ilmbase-testBox.patch,
> >> > > gnu/packages/patches/ilmbase-testBoxAlgo.patch: New files.
> >> > > * gnu-system.am (dist_patch_DATA): Add them.
> >> > > * gnu/packages/graphics.scm (ilmbase)[native-inputs]: Add patch/testBox
> >> > > and patch/testBoxAlgo.
> >> > > [arguments]: Add 'patch-for-i686' phase.
> >> >
> >> > Cool!
> >> >
> >> > I think it’d be reasonable to squash both patches in one file, and to
> >> > apply it unconditionally. WDYT?
> >>
> >> Sure, I'll send an updated patch.
> >
> > Attached.
> >
>
> [...]
>
> >> > So I gather upgrading to IlmBase 2.0.1 is not an option?
> >>
> >> 2.0.1 would be a downgrade for us; we are on 2.2.0. I did try it without
> >> success.
>
> Oops, sorry. :-)
>
> > From 2f895108a3f476038b82e03645ee7f51e4ee1675 Mon Sep 17 00:00:00 2001
> > From: Leo Famulari <leo@famulari.name>
> > Date: Mon, 29 Feb 2016 02:23:43 -0500
> > Subject: [PATCH] gnu: ilmbase: Add patch to fix build on i686.
> >
> > Fixes <http://bugs.gnu.org/22049>.
> >
> > * gnu/packages/patches/ilmbase-fix-tests.patch: New file.
> > * gnu-system.am (dist_patch_DATA): Add it.
> > * gnu/packages/graphics.scm (ilmbase)[source]: Add patch.
>
> Perfect, thanks again!
Pushed as 5e8276dcf4e3a, and closing.
>
> Ludo’.
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2016-03-02 0:21 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-29 3:42 bug#22049: libreoffice compile error rennes
2015-11-29 15:02 ` Ludovic Courtès
2015-11-29 16:50 ` rennes
2015-11-30 19:56 ` Leo Famulari
2015-11-30 20:14 ` Ludovic Courtès
2015-12-01 5:58 ` Leo Famulari
2015-12-03 20:01 ` Leo Famulari
2015-12-04 11:59 ` Ludovic Courtès
2016-02-28 18:06 ` Ludovic Courtès
2016-02-29 8:35 ` Leo Famulari
2016-02-29 8:37 ` Leo Famulari
2016-03-01 13:36 ` Ludovic Courtès
2016-03-01 17:24 ` Leo Famulari
2016-03-01 18:01 ` Leo Famulari
2016-03-01 20:36 ` Ludovic Courtès
2016-03-02 0:20 ` Leo Famulari
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).