[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#698499: marked as done (unblock: khronos-opencl-headers/1.2-2012.11.30-2)

Your message dated Sat, 19 Jan 2013 15:15:17 +0100
with message-id <20130119141517.GJ5676@radis.cristau.org>
and subject line Re: Bug#698499: unblock: khronos-opencl-headers/1.2-2012.11.30-2
has caused the Debian Bug report #698499,
regarding unblock: khronos-opencl-headers/1.2-2012.11.30-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
698499: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=698499
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package khronos-opencl-headers

This is a new upstream snapshot with some fixes w.r.t to deprecated
APIs. It finally updates the C++ bindings to support 1.2 (instead of
only 1.1) while the other headers have been for 1.2 for a long time.

Unfortunately they introduced some bugs, so I had to add some patches to
clean this up. And I added a few tests to run during package build to
ensure the headers can be used actually :-)

There is also a fix to add a missing M-A: allowed.

The only new feature introduced is support for some newer AMD
extensions, but that should not affect anything else.

The diffstat is huge, primarily because the C++ bindings were reduced
from supporting 128 template parameters to only 32 ... but nothing in
Debian used (or even could use because of compile errors) them so far,
so that can't break anything.

 cl.hpp                                         |71868 +------------------------
 ...
 21 files changed, 4875 insertions(+), 67440 deletions(-)

The diffstat excluding cl.hpp looks much nicer:

 cl.h                                           |  107 +++++++++----------------
 cl_d3d10.h                                     |    2 
 cl_d3d11.h                                     |    2 
 cl_dx9_media_sharing.h                         |    2 
 cl_ext.h                                       |   48 ++++++++++-
 cl_gl.h                                        |   34 +++----
 cl_gl_ext.h                                    |    2 
 cl_platform.h                                  |   79 +++++++++++++++---
 debian/changelog                               |   18 ++++
 debian/control                                 |    5 -
 debian/patches/AMD_extensions.patch            |   76 +++++++++++++++++
 debian/patches/CL_EXT_SUFFIX__VERSION_1_2.diff |   17 +++
 debian/patches/clSetPrintfCallback.patch       |   21 ++++
 debian/patches/series                          |    3 
 debian/rules                                   |    6 +
 debian/t/Makefile                              |   20 ++++
 debian/t/cl_hpp.cpp                            |    1 
 debian/t/opencl_h.c                            |    1 
 debian/t/opencl_h.cpp                          |    1 
 opencl.h                                       |    2 
 20 files changed, 339 insertions(+), 108 deletions(-)

And this is the diffstat for c-headers-only.diff that has the patches
clSetPrintfCallback and CL_EXT_SUFFIX__VERSION_1_2 applied and
whitespace only changes ignored:

 cl.h                   |   44 +++++++++----------------------
 cl_d3d10.h             |    2 -
 cl_d3d11.h             |    2 -
 cl_dx9_media_sharing.h |    2 -
 cl_ext.h               |   48 ++++++++++++++++++++++++++++++----
 cl_gl.h                |   10 ++-----
 cl_gl_ext.h            |    2 -
 cl_platform.h          |   69 +++++++++++++++++++++++++++++++++++++++++++------
 opencl.h               |    2 -
 9 files changed, 127 insertions(+), 54 deletions(-)

khronos-opencl-headers (1.2-2012.11.30-2) unstable; urgency=low

  * opencl-headers: Set Multi-Arch: allowed.  (Closes: #698495)

khronos-opencl-headers (1.2-2012.11.30-1) unstable; urgency=low

  * New upstream release.  (Closes: #695124, #684444) 
  * Run some trivial compile tests during package build.
  * CL_EXT_SUFFIX__VERSION_1_2.diff: New, fix typo in macro name.
  * clSetPrintfCallback.patch: New, add missing declaration.
  * AMD_extensions.patch: New, add a few extensions from AMD that are not yet
    in the headers published by khronos. Thanks Vincent Danjean!
    (Closes: #695136)

Thanks,

Andreas

unblock khronos-opencl-headers/1.2-2012.11.30-2

Attachment: c-headers-only.diff.gz
Description: GNU Zip compressed data

Attachment: koh_1.2-2012.11.30-2.diff.gz
Description: GNU Zip compressed data


--- End Message ---
--- Begin Message --- 
On Sat, Jan 19, 2013 at 14:13:40 +0100, Andreas Beckmann wrote:

> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package khronos-opencl-headers
> 
> This is a new upstream snapshot with some fixes w.r.t to deprecated
> APIs. It finally updates the C++ bindings to support 1.2 (instead of
> only 1.1) while the other headers have been for 1.2 for a long time.
> 
> Unfortunately they introduced some bugs, so I had to add some patches to
> clean this up. And I added a few tests to run during package build to
> ensure the headers can be used actually :-)
> 
> There is also a fix to add a missing M-A: allowed.
> 
> The only new feature introduced is support for some newer AMD
> extensions, but that should not affect anything else.
> 
> The diffstat is huge, primarily because the C++ bindings were reduced
> from supporting 128 template parameters to only 32 ... but nothing in
> Debian used (or even could use because of compile errors) them so far,
> so that can't break anything.
> 
None of this sounds like it warrants a freeze exception, sorry.

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: