Gentoo Archives: gentoo-commits

From: Kent Fredric <kentnl@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: dev-perl/Digest-GOST/, dev-perl/Digest-GOST/files/
Date: Mon, 16 Oct 2017 22:39:34
Message-Id: 1508193248.a63a99f38ca651a6a266369a360eda0189f1d116.kentnl@gentoo
1 commit: a63a99f38ca651a6a266369a360eda0189f1d116
2 Author: Kent Fredric <kentnl <AT> gentoo <DOT> org>
3 AuthorDate: Mon Oct 16 22:30:54 2017 +0000
4 Commit: Kent Fredric <kentnl <AT> gentoo <DOT> org>
5 CommitDate: Mon Oct 16 22:34:08 2017 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=a63a99f3
7
8 dev-perl/Digest-GOST: Fix broken linking on BigEndian bug #608214
9
10 -r1 bump because people who previously built this withour running tests
11 have a silently broken install.
12
13 Closes: https://bugs.gentoo.org/608214
14 Package-Manager: Portage-2.3.8, Repoman-2.3.3
15
16 ...-0.60.0.ebuild => Digest-GOST-0.60.0-r1.ebuild} | 1 +
17 .../files/Digest-GOST-0.60.0-bigendian-link.patch | 43 ++++++++++++++++++++++
18 2 files changed, 44 insertions(+)
19
20 diff --git a/dev-perl/Digest-GOST/Digest-GOST-0.60.0.ebuild b/dev-perl/Digest-GOST/Digest-GOST-0.60.0-r1.ebuild
21 similarity index 90%
22 rename from dev-perl/Digest-GOST/Digest-GOST-0.60.0.ebuild
23 rename to dev-perl/Digest-GOST/Digest-GOST-0.60.0-r1.ebuild
24 index 126317e2ab8..e7f28397c7e 100644
25 --- a/dev-perl/Digest-GOST/Digest-GOST-0.60.0.ebuild
26 +++ b/dev-perl/Digest-GOST/Digest-GOST-0.60.0-r1.ebuild
27 @@ -23,3 +23,4 @@ DEPEND="${RDEPEND}
28 >=virtual/perl-Test-Simple-0.820.0
29 )
30 "
31 +PATCHES=( "${FILESDIR}/${P}-bigendian-link.patch" )
32
33 diff --git a/dev-perl/Digest-GOST/files/Digest-GOST-0.60.0-bigendian-link.patch b/dev-perl/Digest-GOST/files/Digest-GOST-0.60.0-bigendian-link.patch
34 new file mode 100644
35 index 00000000000..bc43f7e079a
36 --- /dev/null
37 +++ b/dev-perl/Digest-GOST/files/Digest-GOST-0.60.0-bigendian-link.patch
38 @@ -0,0 +1,43 @@
39 +From c7dda2667b15cb9cf44856bd0d8efc9321eca354 Mon Sep 17 00:00:00 2001
40 +From: Sergei Trofimovich <slyfox@g.o>
41 +Date: Mon, 16 Oct 2017 10:27:32 +0100
42 +Subject: Fix link failure on big-endian platforms.
43 +
44 +On powerpc64 tests failed as:
45 +
46 +```
47 + Failed 2/2 subtests
48 + ===( 2;0 0/? 0/? 0/? )=========================================Can't load '.../Digest-GOST-0.06/blib/arch/auto/Digest/GOST/GOST.so' for
49 + module Digest::GOST: .../Digest-GOST-0.06/blib/arch/auto/Digest/GOST/GOST.so: undefined symbol: rhash_u32_swap_copy at /usr/lib64/perl5/5.24.3/powerpc64-linux/DynaLoader.pm line 193.
50 +```
51 +
52 +rhash_u32_swap_copy() is a function local to this package.
53 +Seems to be used only for big-endian case.
54 +
55 +The fix is to add missing 'src/byte_order.c' file.
56 +
57 +All tests pass with patch applied.
58 +
59 +Bug: https://bugs.gentoo.org/608214
60 +Bug: https://rt.cpan.org/Public/Bug/Display.html?id=120092
61 +Bug: https://github.com/gray/digest-gost/pull/1
62 +Signed-off-by: Sergei Trofimovich <slyfox@g.o>
63 +---
64 + GOST.xs | 1 +
65 + 1 file changed, 1 insertion(+)
66 +
67 +diff --git a/GOST.xs b/GOST.xs
68 +index 1dfd33c..4ea6f35 100644
69 +--- a/GOST.xs
70 ++++ b/GOST.xs
71 +@@ -6,6 +6,7 @@
72 + #include "ppport.h"
73 +
74 + #include "src/gost.c"
75 ++#include "src/byte_order.c"
76 +
77 + static int
78 + hex_encode (char *dest, const unsigned char *src, int len) {
79 +--
80 +2.14.2
81 +