On Tue, 26 Oct 2010, Evan Broder wrote:

Package: alpine
Version: 2.01-1
Severity: normal
Tags: patch

alpine uses symbols from libkrb5 and libcrypto, but does not link
those libraries directly.

Currently this does not cause a build failure because libkrb5 is
linked as a result of linking libgssapi, and libcrypto is linked by
linking libssl.

However, it does cause build failures on Ubuntu Natty due to changes
in the linker configuration, and it will cause similar failures in
Wheezy for the same reason (see
http://wiki.debian.org/ToolChain/DSOLinking)

I've attached a patch that fixes the issue in a fairly minimal way as
a straw man.

Please let me know if you're not planning to upload fixes during the
Squeeze freeze, so that I can go ahead and fix this in Ubuntu.

I don't know where you got "alpine 2.01" in your version string. But this is almost definitely a good idea to change.

Your patch is kind of excessive -- I don't tthink you really mean to do this:

+ case "$host" in
+-  *-linux-gnu*|*-k*bsd*-gnu*|*-*-gnu*)
++  *-linux-gnu*|*-k*bsd*-gnu*)

But in general I'm kind of struggling to understand this patch format where there's two nested patches.

The patches aren't in a good format to go upstream. Can you improve them so they can be? You're changing configure.ac but then also patching configure; best to just re-run autogen.sh if possible.

I'll take a closer look at this on Monday, but that's my thoughts for now.

-- Asheesh.

--
Q:      What do you call the scratches that you get when a female
        sheep bites you?
A:      Ewe nicks.



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to