543089 - XP_MACOSX should be defined when cross-compiling using toolwhip

Status: RESOLVED FIXED

(NSPR :: NSPR, defect)

Description

[Boris Zbarsky [:bzbarsky]]

Attachment: Proposed fix

Right now, the NSPR configure doesn't set XP_MACOSX unless it finds /System/Library/Frameworks/Carbon.framework/Carbon which means that trying to cross-compile Gecko using toolwhip fails to produce a properly working binary.  In particular, prlink ends up unable to load dylibs, which means NSS doesn't work correctly.

The proposed patch leaves the behavior as it is now by default, but if MACOS_SDK_DIR is explicitly set looks for Carbon in that SDK instead of looking for it in the "system default sdk".  This lets one use --with-macos-sdk for victory.

Comment 1

[Wan-Teh Chang]

bz: your patch should be fine.  But NSPR should be able
to load dylibs when XP_MACOSX is not defined.  Only the
ability to load CFM & CFBundle plugins should be affected.
We should find out why...

Comment 2

[Wan-Teh Chang]

I see.  I was confused between loading "loadable bundles"
and loading dylibs.  The loading of dylibs is only enabled
if XP_MACOSX is defined, which I believe is overly strict.

In any case bz's patch is fine by me, but let's have Ted
review it authoritatively.

Comment 3

[Boris Zbarsky [:bzbarsky]]

> I was confused between loading "loadable bundles" and loading dylibs.  The
> loading of dylibs is only enabled if XP_MACOSX is defined

Yep, that's exactly what I was seeing.

Comment 4

[(not currently active) Ted Mielczarek]

Comment on attachment 424303 [details] [diff] [review]
Proposed fix

This looks fine. Would be easier to review if you hadn't included configure in the diff. :)

I'll land this for you on NSPR HEAD.

Comment 5

[(not currently active) Ted Mielczarek]

Checking in configure;
/cvsroot/mozilla/nsprpub/configure,v  <--  configure
new revision: 1.266; previous revision: 1.265
done
Checking in configure.in;
/cvsroot/mozilla/nsprpub/configure.in,v  <--  configure.in
new revision: 1.270; previous revision: 1.269
done