Open correct libgssapi library version on OpenBSD

RESOLVED FIXED in mozilla12

Status

()

RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: gaston, Assigned: gaston)

Tracking

Trunk
mozilla12
x86
OpenBSD
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

7 years ago
Created attachment 542045 [details] [diff] [review]
Try to open libgssapi.so

Aftre leeengthy bikeshedding in https://bugzilla.mozilla.org/show_bug.cgi?id=650772, here's a patch to open the correct library on OpenBSD, where libgssapi.so.5.0 is present, and the loader will open it if asked for libgssapi.so. Not sure the component is the good one for extensions/auth subdir..
(Assignee)

Updated

7 years ago
Attachment #542045 - Flags: review?(kaie)

Updated

7 years ago
Attachment #542045 - Attachment is patch: true
(Assignee)

Comment 1

7 years ago
Erm... kai, ping ? we've been shipping this patch for ages.
(Assignee)

Comment 2

7 years ago
Created attachment 592426 [details] [diff] [review]
Open libgssapi.so for OpenBSD
Assignee: nobody → landry
Attachment #542045 - Attachment is obsolete: true
Attachment #592426 - Flags: review?(dmose)
Attachment #542045 - Flags: review?(kaie)
(Assignee)

Comment 3

7 years ago
Forgot to say it's a new patch against m-c, previous didn't apply.
(Assignee)

Comment 4

7 years ago
Comment on attachment 592426 [details] [diff] [review]
Open libgssapi.so for OpenBSD

Reassing on ludo's advice...
Attachment #592426 - Flags: review?(dmose) → review?(khuey)
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/e676b8f86ab1
Status: NEW → ASSIGNED
Keywords: checkin-needed
Target Milestone: --- → mozilla12
Version: unspecified → Trunk
Comment on attachment 592426 [details] [diff] [review]
Open libgssapi.so for OpenBSD

Also:
>         const char *const verLibNames[] = {
>             "libgssapi_krb5.so.2", /* MIT - FC, Suse10, Debian */
>             "libgssapi.so.4",      /* Heimdal - Suse10, MDK */
>-            "libgssapi.so.1"       /* Heimdal - Suse9, CITI - FC, MDK, Suse10*/
>+            "libgssapi.so.1",       /* Heimdal - Suse9, CITI - FC, MDK, Suse10*/
...before landing, I removed this----^ extra space character to keep these comments aligned after the patch's comma-insertion.
(Assignee)

Comment 7

7 years ago
ah, thanks... i was pretty sure having tried to keep the aligment, but seems i failed :)

Comment 8

7 years ago
https://hg.mozilla.org/mozilla-central/rev/e676b8f86ab1
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.