all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#37825] [PATCH] gnu: bitcoin-abc: Update to 0.20.4.
@ 2019-10-19 18:48 Guillaume Le Vaillant
  2019-10-31 23:23 ` bug#37825: " Marius Bakke
  0 siblings, 1 reply; 4+ messages in thread
From: Guillaume Le Vaillant @ 2019-10-19 18:48 UTC (permalink / raw)
  To: 37825; +Cc: Guillaume Le Vaillant

* gnu/packages/finance.scm (bitcoin-abc): Update to 0.20.4.
  [source]: Fix uri.
  [inputs]: Use bdb-5.3 (bdb-4.8 is not supported).
  [arguments]: Add 'fix-tests' phase.
---
 gnu/packages/finance.scm | 26 +++++++++++++++++++++++---
 1 file changed, 23 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 960e3adeb3..0ab6489aeb 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -1133,15 +1133,35 @@ Trezor wallet.")
   (package
     (inherit bitcoin-core)
     (name "bitcoin-abc")
-    (version "0.19.8")
+    (version "0.20.4")
     (source (origin
               (method url-fetch)
               (uri (string-append "https://download.bitcoinabc.org/"
-                                  version "/linux/src/bitcoin-abc-"
+                                  version "/src/bitcoin-abc-"
                                   version ".tar.gz"))
               (sha256
                (base32
-                "0ndvkxv5m8346bdhfqzgdiz1k9wyjycj05jp7daf9pml3cw79sz5"))))
+                "0fld54z3l7z7k5n35rrjichjnx37j9xp0rv8i69m3x4qfj1xk2np"))))
+    (inputs
+     `(("bdb" ,bdb-5.3)
+       ("boost" ,boost)
+       ("libevent" ,libevent)
+       ("miniupnpc" ,miniupnpc)
+       ("openssl" ,openssl)
+       ("protobuf" ,protobuf)
+       ("qtbase" ,qtbase)))
+    (arguments
+     (substitute-keyword-arguments (package-arguments bitcoin-core)
+       ((#:phases phases)
+        `(modify-phases ,phases
+           (add-after 'unpack 'fix-tests
+             ;; Disable 'check-devtools' test which tries to run a
+             ;; python script that doesn't exist.
+             (lambda _
+               (substitute* "Makefile.in"
+                 (("^check-local: check-devtools")
+                  "check-local:"))
+               #t))))))
     (home-page "https://www.bitcoinabc.org/")
     (synopsis "Bitcoin ABC peer-to-peer full node for the Bitcoin Cash protocol")
     (description
-- 
2.23.0

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

* bug#37825: [PATCH] gnu: bitcoin-abc: Update to 0.20.4.
  2019-10-19 18:48 [bug#37825] [PATCH] gnu: bitcoin-abc: Update to 0.20.4 Guillaume Le Vaillant
@ 2019-10-31 23:23 ` Marius Bakke
  2019-11-01 10:18   ` [bug#37825] " Guillaume Le Vaillant
  0 siblings, 1 reply; 4+ messages in thread
From: Marius Bakke @ 2019-10-31 23:23 UTC (permalink / raw)
  To: Guillaume Le Vaillant, 37825-done

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

Guillaume Le Vaillant <glv@posteo.net> writes:

> * gnu/packages/finance.scm (bitcoin-abc): Update to 0.20.4.
>   [source]: Fix uri.
>   [inputs]: Use bdb-5.3 (bdb-4.8 is not supported).
>   [arguments]: Add 'fix-tests' phase.

[...]

> +    (arguments
> +     (substitute-keyword-arguments (package-arguments bitcoin-core)
> +       ((#:phases phases)
> +        `(modify-phases ,phases
> +           (add-after 'unpack 'fix-tests
> +             ;; Disable 'check-devtools' test which tries to run a
> +             ;; python script that doesn't exist.
> +             (lambda _
> +               (substitute* "Makefile.in"
> +                 (("^check-local: check-devtools")
> +                  "check-local:"))
> +               #t))))))

Do you know why this script is not distributed?  Should we provide it,
or was it a packaging mistake upstream?

In any case I pushed this patch in
28d46d9d4317180b348ad5baeddad8ef25ed430a.  Thank you!

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

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

* [bug#37825] [PATCH] gnu: bitcoin-abc: Update to 0.20.4.
  2019-10-31 23:23 ` bug#37825: " Marius Bakke
@ 2019-11-01 10:18   ` Guillaume Le Vaillant
  2019-11-03 17:11     ` Marius Bakke
  0 siblings, 1 reply; 4+ messages in thread
From: Guillaume Le Vaillant @ 2019-11-01 10:18 UTC (permalink / raw)
  To: Marius Bakke; +Cc: 37825


Marius Bakke skribis:

> Guillaume Le Vaillant <glv@posteo.net> writes:
>
>> * gnu/packages/finance.scm (bitcoin-abc): Update to 0.20.4.
>>   [source]: Fix uri.
>>   [inputs]: Use bdb-5.3 (bdb-4.8 is not supported).
>>   [arguments]: Add 'fix-tests' phase.
>
> [...]
>
>> +    (arguments
>> +     (substitute-keyword-arguments (package-arguments bitcoin-core)
>> +       ((#:phases phases)
>> +        `(modify-phases ,phases
>> +           (add-after 'unpack 'fix-tests
>> +             ;; Disable 'check-devtools' test which tries to run a
>> +             ;; python script that doesn't exist.
>> +             (lambda _
>> +               (substitute* "Makefile.in"
>> +                 (("^check-local: check-devtools")
>> +                  "check-local:"))
>> +               #t))))))
>
> Do you know why this script is not distributed?  Should we provide it,
> or was it a packaging mistake upstream?
>
> In any case I pushed this patch in
> 28d46d9d4317180b348ad5baeddad8ef25ed430a.  Thank you!

I'm not sure if it's a packaging mistake or a makefile mistake (if these
tests are not supposed to be run on a release version). They are tests
to check that some development tools work correctly (e.g. tools to
generate a header file which is included in the release tarball, to
update the copyright years in the headers, etc).

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

* [bug#37825] [PATCH] gnu: bitcoin-abc: Update to 0.20.4.
  2019-11-01 10:18   ` [bug#37825] " Guillaume Le Vaillant
@ 2019-11-03 17:11     ` Marius Bakke
  0 siblings, 0 replies; 4+ messages in thread
From: Marius Bakke @ 2019-11-03 17:11 UTC (permalink / raw)
  To: Guillaume Le Vaillant; +Cc: 37825

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

Guillaume Le Vaillant <glv@posteo.net> writes:

> Marius Bakke skribis:
>
>> Guillaume Le Vaillant <glv@posteo.net> writes:
>>
>>> * gnu/packages/finance.scm (bitcoin-abc): Update to 0.20.4.
>>>   [source]: Fix uri.
>>>   [inputs]: Use bdb-5.3 (bdb-4.8 is not supported).
>>>   [arguments]: Add 'fix-tests' phase.
>>
>> [...]
>>
>>> +    (arguments
>>> +     (substitute-keyword-arguments (package-arguments bitcoin-core)
>>> +       ((#:phases phases)
>>> +        `(modify-phases ,phases
>>> +           (add-after 'unpack 'fix-tests
>>> +             ;; Disable 'check-devtools' test which tries to run a
>>> +             ;; python script that doesn't exist.
>>> +             (lambda _
>>> +               (substitute* "Makefile.in"
>>> +                 (("^check-local: check-devtools")
>>> +                  "check-local:"))
>>> +               #t))))))
>>
>> Do you know why this script is not distributed?  Should we provide it,
>> or was it a packaging mistake upstream?
>>
>> In any case I pushed this patch in
>> 28d46d9d4317180b348ad5baeddad8ef25ed430a.  Thank you!
>
> I'm not sure if it's a packaging mistake or a makefile mistake (if these
> tests are not supposed to be run on a release version). They are tests
> to check that some development tools work correctly (e.g. tools to
> generate a header file which is included in the release tarball, to
> update the copyright years in the headers, etc).

I see, thanks for the explanation.  Perhaps we are simply using the
wrong Makefile target?  By default gnu-build-system runs 'make check' in
the 'check' phase: some packages use 'make test' or 'make ci' instead.

Something to look out for in the next version.  :-)

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

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

end of thread, other threads:[~2019-11-03 17:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-19 18:48 [bug#37825] [PATCH] gnu: bitcoin-abc: Update to 0.20.4 Guillaume Le Vaillant
2019-10-31 23:23 ` bug#37825: " Marius Bakke
2019-11-01 10:18   ` [bug#37825] " Guillaume Le Vaillant
2019-11-03 17:11     ` Marius Bakke

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.