526192 - Don't link against libsoftokn.so

Status: RESOLVED INVALID

(Directory :: LDAP C SDK, defect)

Description

[Mike Hommey [:glandium]]

Attachment: Patch against seamonkey 2.0

Disclaimer: I'm not sure I should file this under LDAP C SDK, considering this is something I would like to land in Seamonkey, nor who should be reviewing this.

LDAP C SDK builds link against the libsoftokn.so file, while this one is supposed to be dynamically loaded. Please note that Windows is not affected because an unrelated change (switching from using -l to using .lib) removed softokn from the list.

Comment 1

[Anton Bobrov]

Mark, Rich, any idea why we link against softkn directly ? perhaps historically we had to ? anyway i dont see why we need to link directly against it anymore given that libnss has it as its own dependency. this patch looks good to me.

Comment 2

[Rich Megginson]

(In reply to comment #1)
> Mark, Rich, any idea why we link against softkn directly ? perhaps historically
> we had to ? anyway i dont see why we need to link directly against it anymore
> given that libnss has it as its own dependency. this patch looks good to me.

Right.  Historically, we had to, otherwise we would get either build time or runtime link errors.  I think NSS fixed this a long time ago, so we should not link with softokn anymore.

Comment 3

[Mike Hommey [:glandium]]

This is now obsoleted by the move of the c-sdk build system to moz.build in comm-central.