[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add dlib.
From: |
Ben Woodcroft |
Subject: |
Re: [PATCH] gnu: Add dlib. |
Date: |
Wed, 24 Aug 2016 18:51:59 -0400 |
User-agent: |
K-9 Mail for Android |
On 24 August 2016 3:08:58 PM GMT-04:00, Marius Bakke <address@hidden> wrote:
>
>>> 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?
>
>We rely on cmake-build-system for everything but the check phase, which
>doesn't have a "proper" check target. I'm no cmake expert, but we can
>probably prevent re-build by not including the main application in
>test/CmakeLists.txt and instead copy/link in our already-built version:
>
>https://github.com/davisking/dlib/blob/master/dlib/test/CMakeLists.txt#L15
The cmake build system has an #:out-of-tree? option, maybe set that to #f?
Ben
- Re: [PATCH] gnu: Add dlib., (continued)
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/17
- Re: [PATCH] gnu: Add dlib., Leo Famulari, 2016/08/18
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/19
- Re: [PATCH] gnu: Add dlib., Leo Famulari, 2016/08/21
- Re: [PATCH] gnu: Add dlib., Ben Woodcroft, 2016/08/21
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/22
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/23
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/24
- Re: [PATCH] gnu: Add dlib., Leo Famulari, 2016/08/24
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/24
- Re: [PATCH] gnu: Add dlib.,
Ben Woodcroft <=
- Re: [PATCH] gnu: Add dlib., Marius Bakke, 2016/08/30
- Re: [PATCH] gnu: Add dlib., Leo Famulari, 2016/08/31
- Re: [PATCH] gnu: Add dlib., Leo Famulari, 2016/08/18