Closed Bug 66185 Opened 24 years ago Closed 23 years ago

Need to get PSM working on the trunk for AIX

Categories

(Core Graveyard :: Security: UI, defect, P5)

1.0 Branch
Other
AIX
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jdunn, Assigned: jdunn)

References

Details

Attachments

(2 files)

Need to get an AIX PSM working on the trunk.
Blocks: 18688
Status: NEW → ASSIGNED
QA Contact: ckritzer → jdunn
The work I have done for the AIX (4.3.2, xlC_r) psm is based on my work for the HP-UX 11 one (see 64650). Here is the breakdown of the changes -security/coreconf/AIX.mk change to use xlc_r for 'cc' compiles, this reduces warnings... -security/coreconf/AIX4.3.mk do NOT use svld on aix4.3.2 (i believe this is the old 'dlopen' style of loading libs). Instead link with -bso. Also turn 'on' berok during DSO creation - to flag errors. Don't 'hard link' to /usr/lib/xlC/lib. -security/psm/server/Makefile Similar changes from HP's... Add an AIX section for NSPR XPCOM OS_LIBS & MKPROG definitions -security/psm/server/ctrlconn.c AIX requires the cast form xpcomGetProcessDir to processDir. -security/psm/server/manifest.mn & psmmain.c & main.cpp HP changes - main() belongs in a C++ file. -security/psm/server/servutil.c AIX and OSF both require the memsetting of servaddr. So might as well just do it everywhere! -security/psm/server/start-psm & start-regxpcom Adding AIX's LIBPATH env variable for finding shared libs...
Depends on: 64650
Jim: you haven't attached your patch. Regarding your change to security/psm/server/servutil.c, I've just changed PR_InitializeNetAddr() to memset the PRNetAddr, so you no longer need that memset call.
Attached patch AIX psm diffs (deleted) — Splinter Review
jdunn: can you make the diffs unified diffs?
Attached patch new set of diffs with (deleted) — Splinter Review
Sorry about the spam. An unset priority should be less than P5.
Priority: -- → P5
Jim: 1. coreconf/AIX.mk: I will not take this change. Using xlC_r to compile our C code has worked just fine. 2. coreconf/AIX4.3.mk: we probably should take this change but I'd like to understand it more. We are very close to NSS 3.2 RTM so very likely I will consider this after NSS 3.2 ships (Feb. 14). 3. psm/server/Makefile: the right fix is to set MKPROG to $(CCC) on all platforms and delete the whole CPLUSPLUSRUNTIME thing. This is the missing piece of the main.cpp+psmmain.c solution.
The aix tinderbox on the Ports page is now building with --enable-crypto. Is this bug still relevant?
This is finally fixed 0.9.1 builds (and beyond) just compile and work... thanks for everyone's help!
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
CRYPTO, not GENERAL
Component: Security: General → Security: Crypto
Verified per reporter's comment.
Status: RESOLVED → VERIFIED
Mass changing Security:Crypto to PSM
Component: Security: Crypto → Client Library
Product: Browser → PSM
Version: other → 2.1
Mass changing Security:Crypto to PSM
Product: PSM → Core
Version: psm2.1 → 1.0 Branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: