On Sep 2, 2010, at 3:36 AM, Roman Haefeli wrote:

On Thu, 2010-09-02 at 01:07 -0400, Hans-Christoph Steiner wrote:
Looks pretty good to me, but I'm just learning myself :)

Thanks for having a look.
One thing I like to mention: The upstream sources come with a Makefile
based on a apparently old Makefile template for libdirs. It was pretty
broken and created superfluous directories in debian/. I copied over the
current template, adapted it and applied it as a quilt patch. Is that
the proper way to do it?

 I am
wondering about the strip stuff:

override_dh_strip:
 strip --remove-section=.comment --remove-section=.note --strip-
unneeded \
 debian/pd-wiimote/usr/lib/pd/extra/wiimote/wiimote.pd_linux

My guess is that this is needed in order to properly strip
the .pd_linux so binaries?


Yeah, dh_strip apparently does not consider .pd_linux files as shared
objects and also there is no way to force it by passing it file names as
arguments.

I used the strip command from a mail by Felipe Sateler [1]

[1]
http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/2010-August/012336.html

Roman

I was under the impression that dh would set the strip options in the $ (INSTALL_PROGRAM) for the Makefile, so that an explicit strip wasn't necessary. Is that true? If so, I'll change up the pd packages to use INSTALL_PROGRAM and INSTALL_DATA properly. I don't know why I didn't do this to begin with.

.hc


----------------------------------------------------------------------------

Man has survived hitherto because he was too ignorant to know how to realize his wishes. Now that he can realize them, he must either change them, or perish. -William Carlos Williams



_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to