first package: pd-wiimote
Hans-Christoph Steiner
hans at at.or.at
Thu Sep 2 22:38:18 UTC 2010
On Sep 2, 2010, at 5:20 PM, Jonas Smedegaard wrote:
> On Thu, Sep 02, 2010 at 12:58:57PM -0400, Hans-Christoph Steiner
> wrote:
>>
>> 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.
>
> No, debhelper does not interfere with $(INSTALL_PROGRAM). For
> Debian packaging binaries should not be stripped during build - then
> dh_strip should deal with stripping afterwards.
>
> Problem is, the unusual filenames goes below the radar of dh_strip,
> and (according to Felipe) dh_strip cannot be spoonfed additional
> files to strip.
Ok, so unless anyone objects, I'm going to add this to all my pd
packages so they get stripped. That includes pd-motex. I guess that
means the package version gets incremented, since its currently in NEW?
.hc
----------------------------------------------------------------------------
All mankind is of one author, and is one volume; when one man dies,
one chapter is not torn out of the book, but translated into a better
language; and every chapter must be so translated.... -John Donne
More information about the pkg-multimedia-maintainers
mailing list