rra at debian.org
Sun Jun 12 03:38:14 UTC 2011
Andreas Beckmann <debian at abeckmann.de> writes:
> could you upload libthrust to unstable/NEW? This comes bundled with the
> cuda toolkit 4.0, but since Thrust is available under a free license I
> prefer to package it separately from its free source :-)
Sorry about the delay in reviewing this! It looks great. I just noticed
a couple of things that I think would be worth fixing.
1. debian/control declares a dependency on debhelper 8, but debian/compat
says compat level 7. I'd just bump that up to 8.
2. The package long description in debian/control reads a bit like
marketing literature in a few spots. It's not a big deal, but how
about this phrasing instead?
Thrust is a CUDA library of parallel algorithms with an interface
resembling the C++ Standard Template Library (STL). Thrust provides a
flexible high-level interface for GPU programming by providing a rich
collection of data parallel primitives such as scan, sort, and reduce.
It then automatically selects an efficient implementation without
requiring source modifications.
The Compute Unified Device Architecture (CUDA) enables NVIDIA
graphics processing units (GPUs) to be used for massively parallel
general purpose computation.
That adds in a few more details from the Quick Start Guide.
You can also go ahead and add DM-Upload-Allowed: yes to the package so
that once it gets through NEW (and your key is registered), you can upload
further updates directly.
Russ Allbery (rra at debian.org) <http://www.eyrie.org/~eagle/>
More information about the pkg-nvidia-devel