Hi doko, Thank you for the comments.
On Sun, 2016-01-03 at 18:55 +0100, Matthias Klose wrote: > some comments about the v4 patch: > > - Please could you put your generated tarball to some place? The original source is very huge (~1GiB+), and it will grow up to ~2GiB after adding the ppc64el "source" contents. However my current ISP is accounting the traffic flow, so you know. :-( > - could you just remove the man-* patches? Yes, seems that my python scripts for autofixing work well, so there remains no reason to keep them. > - could you remove the MISSING lines in the symbols files? > - dh_strip is overridden to do nothing. same thing should be > done with dh_strip_nondeterminism. Yes, I'll keep these in mind. > - dh_install --list-missing lists some non-installed files. > Are these all not installed by intent? Just the opposite, almost all of them are installed. And the very long list of --list-missing is a historical problem for the CUDA package ...... Due to path change (e.g. from /usr/opt/ to /usr/ ), --list-missing considers them not installed. in d/rules, there are some $RM operations to decrease the length of --list-missing, however, it doesn't work all the time. 1. introduce a "--list-missing-by-checksum" to debhelper ? this is very slow but may be used by other packages. 2. fix paths in debian/tmp before dh_install. this will only work in the CUDA package. > dh_install: usr/CUDA_Toolkit_Release_Notes.txt exists in debian/tmp but is > not > installed to anywhere [...] By the way, the missing CUDA support for ppc64el blocks 7.5 (missing it is a mistake. ppc64el support is available since CUDA version 7.0). See BUG #808883 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=808883 So I plan to 1. prepare CUDA-7.0.28+ds1-1, adding missing ppc64el "source". 2. migrate some changes from this BUG into the +ds1 upload.