Gentoo Archives: gentoo-commits

From: Alexis Ballier <aballier@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: media-libs/ilmbase/, media-libs/ilmbase/files/
Date: Thu, 28 Sep 2017 12:50:25
Message-Id: 1506602707.24675bd3c2c26e2a612932e0471af99c34a4f1b0.aballier@gentoo
1 commit: 24675bd3c2c26e2a612932e0471af99c34a4f1b0
2 Author: Jonathan Scruggs <j.scruggs <AT> gmail <DOT> com>
3 AuthorDate: Sat Sep 23 13:01:19 2017 +0000
4 Commit: Alexis Ballier <aballier <AT> gentoo <DOT> org>
5 CommitDate: Thu Sep 28 12:45:07 2017 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=24675bd3
7
8 media-libs/ilmbase: Revision bump to 2.2.0-r1
9
10 * Changed ebuild to EAPI 6
11 * Changed ebuild to use multilib-minimal
12 * Added patch to remove register keyword for GCC7 Compat
13 * Added tabs in the metadata.xml file
14
15 .../ilmbase-2.2.0-Remove-register-keyword.patch | 79 ++++++++++++++++++++++
16 media-libs/ilmbase/ilmbase-2.2.0-r1.ebuild | 30 ++++++++
17 media-libs/ilmbase/metadata.xml | 9 ++-
18 3 files changed, 115 insertions(+), 3 deletions(-)
19
20 diff --git a/media-libs/ilmbase/files/ilmbase-2.2.0-Remove-register-keyword.patch b/media-libs/ilmbase/files/ilmbase-2.2.0-Remove-register-keyword.patch
21 new file mode 100644
22 index 00000000000..3fd95aceda9
23 --- /dev/null
24 +++ b/media-libs/ilmbase/files/ilmbase-2.2.0-Remove-register-keyword.patch
25 @@ -0,0 +1,79 @@
26 +From 6d297f35c5dbfacc8a5e94f33b986db7ab468db9 Mon Sep 17 00:00:00 2001
27 +From: Larry Gritz <lg@××××××××××.com>
28 +Date: Fri, 30 Sep 2016 15:58:27 -0700
29 +Subject: [PATCH] Remove 'register' keyword.
30 +
31 +'register' is a relic of K&R-era C, it's utterly useless in modern
32 +compilers. It's been deprecated in C++11, and therefore will generate
33 +warnings when encountered -- and many packages that use OpenEXR's public
34 +headers use -Werr to turn warnings into errors. Starting in C++17, the
35 +keyword is removed entirely, and thus will certainly be a build break
36 +for that version of the standard. So it's time for it to go.
37 +---
38 + IlmBase/Half/half.cpp | 6 +++---
39 + IlmBase/Half/half.h | 4 ++--
40 + IlmBase/Imath/ImathMatrix.h | 8 ++++----
41 + 3 files changed, 9 insertions(+), 9 deletions(-)
42 +
43 +diff --git a/Half/half.cpp b/Half/half.cpp
44 +index 609aaa1..09a50aa 100644
45 +--- a/Half/half.cpp
46 ++++ b/Half/half.cpp
47 +@@ -94,9 +94,9 @@ half::convert (int i)
48 + // of float and half (127 versus 15).
49 + //
50 +
51 +- register int s = (i >> 16) & 0x00008000;
52 +- register int e = ((i >> 23) & 0x000000ff) - (127 - 15);
53 +- register int m = i & 0x007fffff;
54 ++ int s = (i >> 16) & 0x00008000;
55 ++ int e = ((i >> 23) & 0x000000ff) - (127 - 15);
56 ++ int m = i & 0x007fffff;
57 +
58 + //
59 + // Now reassemble s, e and m into a half:
60 +diff --git a/Half/half.h b/Half/half.h
61 +index f78e4f6..da418d6 100644
62 +--- a/Half/half.h
63 ++++ b/Half/half.h
64 +@@ -459,7 +459,7 @@ half::half (float f)
65 + // to do the float-to-half conversion.
66 + //
67 +
68 +- register int e = (x.i >> 23) & 0x000001ff;
69 ++ int e = (x.i >> 23) & 0x000001ff;
70 +
71 + e = _eLut[e];
72 +
73 +@@ -470,7 +470,7 @@ half::half (float f)
74 + // bits and combine it with the sign and exponent.
75 + //
76 +
77 +- register int m = x.i & 0x007fffff;
78 ++ int m = x.i & 0x007fffff;
79 + _h = e + ((m + 0x00000fff + ((m >> 13) & 1)) >> 13);
80 + }
81 + else
82 +diff --git a/Imath/ImathMatrix.h b/Imath/ImathMatrix.h
83 +index 3e96c2f..1d7f8dc 100644
84 +--- a/Imath/ImathMatrix.h
85 ++++ b/Imath/ImathMatrix.h
86 +@@ -2527,11 +2527,11 @@ Matrix44<T>::multiply (const Matrix44<T> &a,
87 + const Matrix44<T> &b,
88 + Matrix44<T> &c)
89 + {
90 +- register const T * IMATH_RESTRICT ap = &a.x[0][0];
91 +- register const T * IMATH_RESTRICT bp = &b.x[0][0];
92 +- register T * IMATH_RESTRICT cp = &c.x[0][0];
93 ++ const T * IMATH_RESTRICT ap = &a.x[0][0];
94 ++ const T * IMATH_RESTRICT bp = &b.x[0][0];
95 ++ T * IMATH_RESTRICT cp = &c.x[0][0];
96 +
97 +- register T a0, a1, a2, a3;
98 ++ T a0, a1, a2, a3;
99 +
100 + a0 = ap[0];
101 + a1 = ap[1];
102 +--
103 +2.14.1
104 +
105
106 diff --git a/media-libs/ilmbase/ilmbase-2.2.0-r1.ebuild b/media-libs/ilmbase/ilmbase-2.2.0-r1.ebuild
107 new file mode 100644
108 index 00000000000..0fbeb31e701
109 --- /dev/null
110 +++ b/media-libs/ilmbase/ilmbase-2.2.0-r1.ebuild
111 @@ -0,0 +1,30 @@
112 +# Copyright 1999-2017 Gentoo Foundation
113 +# Distributed under the terms of the GNU General Public License v2
114 +
115 +EAPI=6
116 +inherit multilib-minimal
117 +
118 +DESCRIPTION="OpenEXR ILM Base libraries"
119 +HOMEPAGE="http://openexr.com/"
120 +SRC_URI="http://download.savannah.gnu.org/releases/openexr/${P}.tar.gz"
121 +
122 +LICENSE="BSD"
123 +SLOT="0/12" # based on SONAME
124 +KEYWORDS="~amd64 -arm ~arm64 ~hppa ~ia64 ~ppc ~ppc64 ~sparc ~x86 ~amd64-fbsd ~x86-fbsd ~amd64-linux ~x86-linux ~x64-macos ~x86-solaris"
125 +IUSE="static-libs"
126 +
127 +DEPEND="virtual/pkgconfig[${MULTILIB_USEDEP}]"
128 +
129 +DOCS=( AUTHORS ChangeLog NEWS README )
130 +MULTILIB_WRAPPED_HEADERS=( /usr/include/OpenEXR/IlmBaseConfig.h )
131 +
132 +PATCHES=( "${FILESDIR}/${P}-Remove-register-keyword.patch" )
133 +
134 +multilib_src_configure() {
135 + # Disable use of ucontext.h wrt #482890
136 + if use hppa || use ppc || use ppc64; then
137 + export ac_cv_header_ucontext_h=no
138 + fi
139 +
140 + ECONF_SOURCE=${S} econf "$(use_enable static-libs static)"
141 +}
142
143 diff --git a/media-libs/ilmbase/metadata.xml b/media-libs/ilmbase/metadata.xml
144 index 0427ec20bfd..fc9e500d69e 100644
145 --- a/media-libs/ilmbase/metadata.xml
146 +++ b/media-libs/ilmbase/metadata.xml
147 @@ -1,7 +1,10 @@
148 <?xml version="1.0" encoding="UTF-8"?>
149 <!DOCTYPE pkgmetadata SYSTEM "http://www.gentoo.org/dtd/metadata.dtd">
150 <pkgmetadata>
151 -<maintainer type="project">
152 -<email>media-video@g.o</email>
153 -</maintainer>
154 + <maintainer type="project">
155 + <email>media-video@g.o</email>
156 + </maintainer>
157 + <upstream>
158 + <remote-id type="github">openexr/openexr</remote-id>
159 + </upstream>
160 </pkgmetadata>