From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] gnu: Add dlib. Date: Wed, 24 Aug 2016 13:26:08 -0400 Message-ID: <20160824172608.GA24668@jasmine> References: <20160816234507.GA24224@jasmine> <87h9ajzc1z.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160818202353.GB2393@jasmine> <8737m1yqru.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160821201757.GA21038@jasmine> <6004b842-2434-9cdc-7a31-04231eac5941@uq.edu.au> <87k2f957wt.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <8760qr5o8n.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <87eg5e4g4r.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:46659) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bcbwc-0003bj-Ig for guix-devel@gnu.org; Wed, 24 Aug 2016 13:26:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bcbwX-0002nS-F0 for guix-devel@gnu.org; Wed, 24 Aug 2016 13:26:29 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:57175) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bcbwU-0002mV-WA for guix-devel@gnu.org; Wed, 24 Aug 2016 13:26:25 -0400 Content-Disposition: inline In-Reply-To: <87eg5e4g4r.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Marius Bakke Cc: guix-devel , Alex Kost On Wed, Aug 24, 2016 at 11:26:28AM +0100, Marius Bakke wrote: > There are a couple of things going on in this thread: > > 1. Segfault on x86_64. This seems to have been resolved simply by > updating OpenBLAS. At least, I'm no longer able to reproduce it even > with LAPACK in inputs. So, that should fix the Hydra x86_64 build. > Can the OpenBLAS update be cherry-picked to master? I'd say it depends on whether the OpenBLAS users are building successfully on core-updates, but unfortunately core-updates is currently failing early in the bootstrap process [0]. Can you take a look at `guix refresh -l dlib` and pick some important looking applications to test with the updated OpenBLAS? [0] https://hydra.gnu.org/jobset/gnu/core-updates http://lists.gnu.org/archive/html/guix-devel/2016-08/msg01390.html > 2. i686 test failures. Updating OpenBLAS fixed 1/5 errors. The remaining > four are reproducible on 32-bit Ubuntu, so they do not seem Guix > related. Upstream has been notified. > > 3. ARM failures. I don't have ARM hardware to test on, but I'm guessing > it's similar to i686 (i.e. not directly Guix related). Maybe dlib is 64-bit only? If that's the case, we can disable it on those architectures. > Adding "#:parallel-build? #f" had no effect on tests, indeed the check > phase does not seem to use the previously built dlib; it builds it again > without parallel-build. I will try reproducing the non-reproducibility > on some higher end hardware, hopefully this week. It's weird that it rebuilds the application again for the tests. Which build is actually installed in the case of success? Is it worth an upstream bug report? > I've also found that FFTW is no longer used, apparently due to thread > safety issues. So I'd appreciate if the following patch can be added. > Apologies for not catching the missing reference earlier, I will be more > careful in the future (fftw was added in the last minute..). Can you hold this patch locally in your "dlib fixes" queue? It would trigger a rebuild of dlib on Hydra, but I don't see the point when we expect the build to fail near the end anyways.