[Bug 459] Review request: nvidia-cuda-toolkit - NVIDIA CUDA Toolkit libraries

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=459


NicolasChauvet <kwizart at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




--- Comment #4 from NicolasChauvet <kwizart at gmail.com>  2009-03-24 17:05:46 ---
You have packaged cuda like if it was a library, but it is a more a compiler
than a library. I don't think using a -devel will make sense.

Usually, Fedora guideline are pretty clear on the name of the package, it need
to be the name of the "source tarball". In this case, it should be cudatoolkit.
I would agree to use only cuda (which is the kown product name), but having the
package named cudatoolkit and eventually provides cuda seems the prefered way.
I'm aware that historically, the package name has changed. (and may change with
newer version also).

- Does the package need to be (can be) multilib compliant or multiarch
compliant. At this time it is neither mutlilibs nor multiarches compliant.
(it may not be relevant)

- There is also a cuda-sdk (cuda-devel ?)
http://www.nvidia.com/object/thankyou_linux.html?url=/compute/cuda/2_1/SDK/cuda-sdk-linux-2.10.1215.2015-3233425.run
Is it needed along with cuda ?

* - Development/Languages seems more accurate to describe the rpm Group.

* - Does Cg is needed (bundled within) for this package ? 
  And which version it might work best with ?

* - The package are binary only, but nvidia currently only certify it on F-9
and EL-5. Also with the rpm strongerHash feature, rpm built on F-10 and beyond
could not be compatible with older distro. Thus I think the disttag must be
re-enabled.


* For the debuginfo, You can have a look on the rar review at rpmfusion to see
how debuginfo can be disabled without touching binaries, or with
xorg-x11-drv-nvidia\* to override the build id while extracting debuginfo.

* please use %{version} for the Source0/1.

* Please use install with -p whenever it is possible to prevent timestamp
change. (for example, when installing the initscript or when you convert the
doc in utf8). That will avoid.


* scriplets need to end with || : to avoid rpm transaction to be canceled if
something went wrong for any reason. (i don't know why an initscript would be
needed - will check later)


-- 
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.