nspr and firefox3 on powerpc

[Joe Marcus Clarke]

On Mon, 2008-11-03 at 23:20 +0100, Ed Schouten wrote:
> Hello all,
> 
> I saw both the nspr and firefox3 ports were maintained by the GNOME
> folks, so I'd better send these patches to you, to ask if I can commit
> them to CVS.
> 
> Last week I installed FreeBSD -CURRENT on my PowerBook G4 12". So far a
> lot of things start to work, but I noticed I wasn't able to compile my
> favourite browser on it. I've attached two patches, one for nspr and one
> for firefox3 to make them compile properly.
> 
> nspr:
> 
> The patch for nspr is quite obvious. _freebsd.cfg now returns an #error,
> because the platform is unsupported. Add the proper size definitions.
> I've copied these from _linux.cfg. Turns out the _openbsd.cfg and
> _netbsd.cfg definitions are quite probably a little broken, because they
> define PR_BYTES_PER_INT64 to 4. These bits should be correct, though.
> 
> firefox3:
> 
> This patch is a little more complex. The changes to the port Makefile:
> it turns out the `cc -dumpspecs | grep -m 1 pthread' trick doesn't work
> on powerpc, because the flag it tries to obtain is stored on the second
> line it returns. Make the pattern a little more accurate to make it work
> properly. GECKO_PTHREAD_LIBS is now -lpthread instead of a lot of random
> garbage.
> 
> The fixes to the other file look a little strange, but that's mainly
> because diff(1) makes a mess out of it. The only thing it does, is make
> FreeBSD/powerpc use some of the Linux/powerpc xptcall bits that we need
> instead of bailing out because we're running an unknown architecture.
> We already do similar things for FreeBSD/{alpha,sparc64} and looking at
> the xptcall code it is a pretty sane decision to do this.
> 
> Any comments?

We're fine with you committing PPC support for these ports.  These
patches look okay to me.  The pthread hack still works on i386 and
amd64.  Thanks for doing this work!

Joe

-- 
PGP Key : http://www.marcuscom.com/pgp.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: This is a digitally signed message part
Url : http://lists.freebsd.org/pipermail/freebsd-gnome/attachments/20081103/0feebdc0/attachment.pgp