GNU bug report logs - #34048
[PATCH] gnu: z3: Update to 4.8.4.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 34048 in the body.
You can then email your comments to 34048 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Amin Bandali <bandali <at> gnu.org>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: z3: Update to 4.8.4.
Date: Fri, 11 Jan 2019 18:13:53 -0500

[0001-gnu-z3-Update-to-4.8.4.patch (text/x-patch, inline)]

From 2cd82564402e4363db581bde066766d779b6af1a Mon Sep 17 00:00:00 2001
From: Amin Bandali <bandali <at> gnu.org>
Date: Fri, 11 Jan 2019 18:08:42 -0500
Subject: [PATCH] gnu: z3: Update to 4.8.4.

* gnu/packages/maths.scm (z3): Update to 4.8.4.
---
 gnu/packages/maths.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/maths.scm b/gnu/packages/maths.scm
index 4138c7a3b..2c5108388 100644
--- a/gnu/packages/maths.scm
+++ b/gnu/packages/maths.scm
@@ -3937,7 +3937,7 @@ as equations, scalars, vectors, and matrices.")
 (define-public z3
   (package
     (name "z3")
-    (version "4.8.3")
+    (version "4.8.4")
     (home-page "https://github.com/Z3Prover/z3")
     (source (origin
               (method git-fetch)
@@ -3946,7 +3946,7 @@ as equations, scalars, vectors, and matrices.")
               (file-name (git-file-name name version))
               (sha256
                (base32
-                "0p5gdmhd32x6zwmx7j5cgwh4jyfxa9yapym95nlmyfaqzak92qar"))))
+                "014igqm5vwswz0yhz0cdxsj3a6dh7i79hvhgc3jmmmz3z0xm1gyn"))))
     (build-system gnu-build-system)
     (arguments
      `(#:phases
-- 
2.20.1

Message #8 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Amin Bandali <bandali <at> gnu.org>
Cc: 34048 <at> debbugs.gnu.org
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Sat, 12 Jan 2019 18:30:38 -0500

[Message part 1 (text/plain, inline)]

On Fri, Jan 11, 2019 at 06:13:53PM -0500, Amin Bandali wrote:
> From 2cd82564402e4363db581bde066766d779b6af1a Mon Sep 17 00:00:00 2001
> From: Amin Bandali <bandali <at> gnu.org>
> Date: Fri, 11 Jan 2019 18:08:42 -0500
> Subject: [PATCH] gnu: z3: Update to 4.8.4.
> 
> * gnu/packages/maths.scm (z3): Update to 4.8.4.

Thanks!

I tested this patch and found that arachne-pnr fails its test suite
after upgrading z3.

Can you take a look?

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Amin Bandali <bandali <at> gnu.org>
To: Leo Famulari <leo <at> famulari.name>
Cc: 34048 <at> debbugs.gnu.org
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Sat, 12 Jan 2019 22:57:02 -0500

Hi Leo,

Thanks for the reply.

[...]

>
> I tested this patch and found that arachne-pnr fails its test suite
> after upgrading z3.
>
> Can you take a look?
>

Do you have test suite logs from before and after applying the patch?
Running ‘guix build arachne-pnr’ seems to fail for me on both ‘master’
and on my local ‘z3-4.8.4’ branch with my patch applied, so I’m not sure
if it’s my patch that’s broken its test suite.

Also, I’m a bit of Guix newbie, and I’m curious about if, and how, you
test all the packages, only those that depend on Z3, or if you happened
to stumble upon arachne-pnr’s failure.

Thanks,
amin

Message #14 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Efraim Flashner <efraim <at> flashner.co.il>
To: Amin Bandali <bandali <at> gnu.org>
Cc: 34048 <at> debbugs.gnu.org, Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Sun, 13 Jan 2019 09:52:25 +0200

[Message part 1 (text/plain, inline)]

On Sat, Jan 12, 2019 at 10:57:02PM -0500, Amin Bandali wrote:
> Hi Leo,
> 
> Thanks for the reply.
> 
> [...]
> 
> >
> > I tested this patch and found that arachne-pnr fails its test suite
> > after upgrading z3.
> >
> > Can you take a look?
> >
> 
> Do you have test suite logs from before and after applying the patch?
> Running ‘guix build arachne-pnr’ seems to fail for me on both ‘master’
> and on my local ‘z3-4.8.4’ branch with my patch applied, so I’m not sure
> if it’s my patch that’s broken its test suite.
> 
> Also, I’m a bit of Guix newbie, and I’m curious about if, and how, you
> test all the packages, only those that depend on Z3, or if you happened
> to stumble upon arachne-pnr’s failure.
> 

'guix refresh -l z3' shows:
Building the following 2 packages would ensure 3 dependent packages are rebuilt: arachne-pnr <at> 0.0-1-52e69ed20 cubicle <at> 1.1.2

so in theory it's inexpensive to test the packages. Based on
arachne-pnr's version number, I'd check if there's a later commit that
doesn't fail the test suite.

-- 
Efraim Flashner   <efraim <at> flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Amin Bandali <bandali <at> gnu.org>
To: Efraim Flashner <efraim <at> flashner.co.il>
Cc: Danny Milosavljevic <dannym <at> scratchpost.org>, 34048 <at> debbugs.gnu.org,
 Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Sun, 13 Jan 2019 10:16:42 -0500

[Message part 1 (text/plain, inline)]

Hi Efraim,

[...]

> 'guix refresh -l z3' shows:
> Building the following 2 packages would ensure 3 dependent packages are rebuilt: arachne-pnr <at> 0.0-1-52e69ed20 cubicle <at> 1.1.2
>

TIL, thanks!

>
> so in theory it's inexpensive to test the packages. Based on
> arachne-pnr's version number, I'd check if there's a later commit that
> doesn't fail the test suite.
>

Based on your suggestion, I tried bumping arachne-pnr from its current
version in Guix from way back in 2016 to latest master from September
2018 (see the attached patch).

The build phase fails, but it seems to be for another reason than its
current failure.  However, I’ve never used arachne-pnr and don’t know
much about it.

I’ve Cc’ed Danny who originally added the package.  Danny, would you be
able to have a look and see if anything can be done to fix arachne-pnr?

The failure seems to be due to a missing txt file:

,----
| make: *** No rule to make target '/usr/local/share/icebox/chipdb-384.txt', needed by 'share/arachne-pnr/chipdb-384.bin'.  Stop.
`----

Best,
amin

[0001-gnu-arachne-pnr-Update-to-840bdfdeb.patch (text/x-patch, attachment)]

Message #20 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Amin Bandali <bandali <at> gnu.org>
Cc: 34048 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>,
 Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Tue, 15 Jan 2019 16:48:03 +0100

[Message part 1 (text/plain, inline)]

Hi,

I've updated arachne-pnr, icestorm (please make sure to also update this when
you update arachne-pnr since arachne-pnr tests require icestorm) and yosys on
guix master now.

[Message part 2 (application/pgp-signature, inline)]

Message #23 received at 34048 <at> debbugs.gnu.org (full text, mbox):

From: Amin Bandali <bandali <at> gnu.org>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 34048 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>,
 Leo Famulari <leo <at> famulari.name>
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Thu, 17 Jan 2019 23:27:40 -0500

[Message part 1 (text/plain, inline)]

Hi Danny,

On 2019-01-15  4:48 PM, Danny Milosavljevic wrote:
> Hi,
>
> I've updated arachne-pnr, icestorm (please make sure to also update this when
> you update arachne-pnr since arachne-pnr tests require icestorm) and yosys on
> guix master now.
>

Thanks; I’ll keep your advice about them in mind if I end up changing
them in the future.

With Danny’s changes the build phase for arachne-pnr passes but the
check phase fails on one of the test cases, with or without my z3 4.8.4
update patch.  I’ve attached the corresponding logs:

- gwa4kkg00rzp96jkqv52fzz4g197q4-arachne-pnr-0.0-2-840bdfdeb.drv.bz2:
  with z3 4.8.3 (without my patch)
- s1lqkz49nlp147m96bnn0d0yl21dgi-arachne-pnr-0.0-2-840bdfdeb.drv.bz2:
  with z3 4.8.4 (with my patch)

A quick glance at the ediff of the two files, I didn’t see anything
suggesting that the failure may be due to updating z3 from 4.8.3 to
4.8.4.  It might be a good idea to apply the z3 update patch and try
resolving arachne-pnr’s test suite failure in a separate issue.

Thanks,
amin

[gwa4kkg00rzp96jkqv52fzz4g197q4-arachne-pnr-0.0-2-840bdfdeb.drv.bz2 (application/x-bzip2, attachment)]

[s1lqkz49nlp147m96bnn0d0yl21dgi-arachne-pnr-0.0-2-840bdfdeb.drv.bz2 (application/x-bzip2, attachment)]

Message #28 received at 34048-done <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Amin Bandali <bandali <at> gnu.org>
Cc: Danny Milosavljevic <dannym <at> scratchpost.org>, 34048-done <at> debbugs.gnu.org,
 Efraim Flashner <efraim <at> flashner.co.il>
Subject: Re: [bug#34048] [PATCH] gnu: z3: Update to 4.8.4.
Date: Fri, 18 Jan 2019 11:14:52 -0500

[Message part 1 (text/plain, inline)]

On Thu, Jan 17, 2019 at 11:27:40PM -0500, Amin Bandali wrote:
> With Danny’s changes the build phase for arachne-pnr passes but the
> check phase fails on one of the test cases, with or without my z3 4.8.4
> update patch.  I’ve attached the corresponding logs:
> 
> - gwa4kkg00rzp96jkqv52fzz4g197q4-arachne-pnr-0.0-2-840bdfdeb.drv.bz2:
>   with z3 4.8.3 (without my patch)
> - s1lqkz49nlp147m96bnn0d0yl21dgi-arachne-pnr-0.0-2-840bdfdeb.drv.bz2:
>   with z3 4.8.4 (with my patch)
> 
> A quick glance at the ediff of the two files, I didn’t see anything
> suggesting that the failure may be due to updating z3 from 4.8.3 to
> 4.8.4.  It might be a good idea to apply the z3 update patch and try
> resolving arachne-pnr’s test suite failure in a separate issue.

Thanks for investigating. I've pushed the z3 update as
6654f8c1447d80c4899c4234306801407315b31f

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.