Bug 66476

Summary:	nx-x11-1.4.0-r2 fails to compile on PPC
Product:	Gentoo Linux	Reporter:	Nick Lassonde <ziktar>
Component:	Current packages	Assignee:	Stuart Herbert (RETIRED) <stuart>
Status:	VERIFIED UPSTREAM
Severity:	normal
Priority:	High
Version:	2004.1
Hardware:	PPC
OS:	All
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	63757
Attachments:	patch for next_be for PPC

Description Nick Lassonde 2004-10-05 17:42:23 UTC

nx-x11-1.4.0-r2 fails to compile on PPC. It first fails to compile as I've got gcc 3.4 (since 3.4 is now marked stable for PPC). However, after applying the patches from #62378, it gets further along compiling, and then fails again.

This time, the failure is in the nxdesktop component, regarding "next_be" being undefined. (Sorry, I don't have the full output anymore...)

I looked this up, and found that rdesktop had the same parse.h, but had moved next_be out of it's ifdef block and to the end of the file. (The ifdef regards big-endian and data-alignment).

Anyway, I'm doing a bad job describing the bug, but I'll attached a patch which moves next_be to the end of the parse.h just like the current version of rdesktop.

(See http://cvs.sourceforge.net/viewcvs.py/rdesktop/rdesktop/parse.h and look at revision 1.10)

On the other hand, you could also turn on alignment for PPC... From what I've read, PPC doesn't require data alignment, but is faster with it enabled. But, I'm not an expert on this stuff, so I'm just happy this patch let me compile...

Comment 1 Nick Lassonde 2004-10-05 17:43:14 UTC

Created attachment 41189 [details, diff]
patch for next_be for PPC

Comment 2 Nick Lassonde 2004-10-05 17:44:07 UTC

Added block of meta-bug #63757

Comment 3 Stuart Herbert (RETIRED) gentoo-dev

2005-01-02 01:12:21 UTC

This has been fixed in nx-x11-1.4.0-r3, which should appear on your local rsync mirror in about an hour or so.

Best regards,
Stu

Comment 4 Stuart Herbert (RETIRED) gentoo-dev

2005-02-17 02:30:31 UTC

Closing old bug