Openwall GNU/*/Linux - a small security-enhanced Linux distro for servers
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 4 Jan 2012 17:29:02 +0400
From: Solar Designer <solar@...nwall.com>
To: john-dev@...ts.openwall.com
Subject: Re: getting the CUDA & OpenCL stuff into -jumbo

On Wed, Jan 04, 2012 at 02:13:19PM +0100, Samuele Giovanni Tonon wrote:
> this is a patch to be applied on latest jumbo (jumbo-5) .
> 
> As you can see there are little to nothing changes inside of Makefile.
> 
> I was able to cleanly compile it, on both nvidia and Ati machines .

Thanks!  Unfortunately, I think this patch breaks non-OpenCL builds and
thus is unsuitable to be merged into -jumbo as-is.  Specifically, I
think the *_fmt_plug.c files introduced by the patch will be picked up
in all builds, and I see no logic to disable them or their contents when
OpenCL is not available.  I think you need to rename them to *_fmt.c
(not "plug") and have the formats registered in john.c explicitly only
in OpenCL builds (similarly to what we do for formats that require
OpenSSL 0.9.8+).

Alexander

Powered by blists - more mailing lists

Your e-mail address:

Powered by Openwall GNU/*/Linux - Powered by OpenVZ