Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] rng-tools: fix configure to work with new jitterentropy library.
@ 2020-12-03 23:44 lonwillett
  2020-12-04  1:16 ` [PR REVIEW] " ericonr
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: lonwillett @ 2020-12-03 23:44 UTC (permalink / raw)
  To: ml

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

There is a new pull request by lonwillett against master on the void-packages repository

https://github.com/lonwillett/void-packages rng-tools-new-jitterentropy-fix
https://github.com/void-linux/void-packages/pull/26933

rng-tools: fix configure to work with new jitterentropy library.
The recent update to the jitterentropy library broke rng-tools
configure. This patch fixes it. The source is the upstream commit:
https://github.com/nhorman/rng-tools/commit/daff4e2dd9d3abb74829f41e72607681163bd309.

A patch file from https://github.com/void-linux/void-packages/pull/26933.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-rng-tools-new-jitterentropy-fix-26933.patch --]
[-- Type: text/x-diff, Size: 2095 bytes --]

From 6f0d1fb92041b5b2ee0534b1eb45178f2ba2097b Mon Sep 17 00:00:00 2001
From: Lon Willett <xgit@lonw.net>
Date: Fri, 4 Dec 2020 00:32:08 +0100
Subject: [PATCH] rng-tools: fix configure to work with new jitterentropy
 library.

The recent update to the jitterentropy library broke rng-tools
configure. This patch fixes it. The source is the upstream commit:
https://github.com/nhorman/rng-tools/commit/daff4e2dd9d3abb74829f41e72607681163bd309.
---
 ...configure-ac-jitterentropy-needs-pthread.patch | 15 +++++++++++++++
 srcpkgs/rng-tools/template                        |  2 +-
 2 files changed, 16 insertions(+), 1 deletion(-)
 create mode 100644 srcpkgs/rng-tools/patches/0004-configure-ac-jitterentropy-needs-pthread.patch

diff --git a/srcpkgs/rng-tools/patches/0004-configure-ac-jitterentropy-needs-pthread.patch b/srcpkgs/rng-tools/patches/0004-configure-ac-jitterentropy-needs-pthread.patch
new file mode 100644
index 00000000000..4e9147b3b5e
--- /dev/null
+++ b/srcpkgs/rng-tools/patches/0004-configure-ac-jitterentropy-needs-pthread.patch
@@ -0,0 +1,15 @@
+New jitterentropy library broke configure. This patch fixes it.
+
+Upstream patch: https://github.com/nhorman/rng-tools/commit/daff4e2dd9d3abb74829f41e72607681163bd309
+
+--- a/configure.ac	2020-12-03 20:59:11.086238518 +0100
++++ b/configure.ac	2020-12-03 21:00:40.978593727 +0100
+@@ -91,7 +91,7 @@
+ 		AC_SEARCH_LIBS(jent_version,jitterentropy,
+ 				[AM_CONDITIONAL([JITTER], [true])
+ 				AC_DEFINE([HAVE_JITTER],1,[Enable JITTER])],
+-				AC_MSG_NOTICE([No Jitterentropy library found]))
++				AC_MSG_NOTICE([No Jitterentropy library found]),-lpthread)
+ 	], [AC_MSG_NOTICE([Disabling JITTER entropy source])]
+ )
+ 
diff --git a/srcpkgs/rng-tools/template b/srcpkgs/rng-tools/template
index f81c9bf3d85..57f9cb42cf3 100644
--- a/srcpkgs/rng-tools/template
+++ b/srcpkgs/rng-tools/template
@@ -1,7 +1,7 @@
 # Template file for 'rng-tools'
 pkgname=rng-tools
 version=6.10
-revision=3
+revision=4
 wrksrc=${pkgname}-${version}
 build_style=gnu-configure
 configure_args="--sbindir=/usr/bin --without-pkcs11 --without-rtlsdr"

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

* Re: [PR REVIEW] rng-tools: fix configure to work with new jitterentropy library.
  2020-12-03 23:44 [PR PATCH] rng-tools: fix configure to work with new jitterentropy library lonwillett
@ 2020-12-04  1:16 ` ericonr
  2020-12-04  1:16 ` ericonr
  2020-12-04  2:32 ` [PR PATCH] [Merged]: " ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2020-12-04  1:16 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/26933#discussion_r535763610

Comment:
Is a revbump necessary? It depends on `jitterentropy` right now:

```
~ ➜ xbps-query -Rx rng-tools
musl>=1.1.21_1
libsysfs>=2.1.0_1
jitterentropy>=3.0.0_1
jansson>=2.4_1
libcurl>=7.19_1
libcrypto46>=3.1.1_1
```

Having the patch is good, I just want to avoid an unnecessary rebuild.

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

* Re: [PR REVIEW] rng-tools: fix configure to work with new jitterentropy library.
  2020-12-03 23:44 [PR PATCH] rng-tools: fix configure to work with new jitterentropy library lonwillett
  2020-12-04  1:16 ` [PR REVIEW] " ericonr
@ 2020-12-04  1:16 ` ericonr
  2020-12-04  2:32 ` [PR PATCH] [Merged]: " ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2020-12-04  1:16 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/26933#discussion_r535764136

Comment:
It is necessary! This issue doesn't happen on musl :) So the packages differ between the two libcs.

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

* Re: [PR PATCH] [Merged]: rng-tools: fix configure to work with new jitterentropy library.
  2020-12-03 23:44 [PR PATCH] rng-tools: fix configure to work with new jitterentropy library lonwillett
  2020-12-04  1:16 ` [PR REVIEW] " ericonr
  2020-12-04  1:16 ` ericonr
@ 2020-12-04  2:32 ` ericonr
  2 siblings, 0 replies; 4+ messages in thread
From: ericonr @ 2020-12-04  2:32 UTC (permalink / raw)
  To: ml

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

There's a merged pull request on the void-packages repository

rng-tools: fix configure to work with new jitterentropy library.
https://github.com/void-linux/void-packages/pull/26933

Description:
The recent update to the jitterentropy library broke rng-tools
configure. This patch fixes it. The source is the upstream commit:
https://github.com/nhorman/rng-tools/commit/daff4e2dd9d3abb74829f41e72607681163bd309.

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

end of thread, other threads:[~2020-12-04  2:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-03 23:44 [PR PATCH] rng-tools: fix configure to work with new jitterentropy library lonwillett
2020-12-04  1:16 ` [PR REVIEW] " ericonr
2020-12-04  1:16 ` ericonr
2020-12-04  2:32 ` [PR PATCH] [Merged]: " ericonr

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 NNTP newsgroup(s).