Package: debhelper
Version: 5.0.42
Severity: wishlist
Tags: patch

dh_fixperms only fixes permissions of `.h' header files in /usr/include,
but in Debian there are many C++ libraries that use `.hpp' instead.

Here is a simple patch to find `.hpp' headers too:

--- dh_fixperms.orig    2006-12-28 08:10:17.000000000 +0100
+++ dh_fixperms 2006-12-28 08:09:14.000000000 +0100
@@ -77,7 +77,7 @@
        
        # ..and header files ..
        complex_doit("find $tmp/usr/include  -type f",
-               "-name '*.h' $find_options -print0",
+               "-name '*.h' -or -name '*.hpp' $find_options -print0",
                "2>/dev/null | xargs -0r chmod 644");
        
        # .. and perl modules.


PS. A review of the files installed in /usr/include reveals that there are
headers with other extensions, such as `.hxx' or `.hh'. I didn't include
these headers in the patch because: 1) I found this bug while packaging
a library that uses `.hpp' files, 2) it looks like `.hpp' is more widely
used, 3) I'm not sure it is a good idea to include _everything_, and 4)
it is trivial to add them anyway.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to