We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 895

Summary: [PATCH] Build fails on PowerPC64 if no libspe2 is installed.
Product: SDL Reporter: sovewabe
Component: buildAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: major    
Priority: P2    
Version: 1.2.14   
Hardware: PowerPC   
OS: Linux   
Attachments: Check if libspe2 is available

Description sovewabe 2009-12-07 09:32:46 UTC
Created attachment 454 [details]
Check if libspe2 is available

Building on PowerPC64 results in:

[...]
In file included from ./src/video/ps3/SDL_ps3events_c.h:33,
                 from ./src/video/ps3/SDL_ps3video.c:32:
./src/video/ps3/SDL_ps3video.h:34:21: error: libspe2.h: No such file or
directory
In file included from ./src/video/ps3/SDL_ps3events_c.h:33,
                 from ./src/video/ps3/SDL_ps3video.c:32:
./src/video/ps3/SDL_ps3video.h:75: error: expected specifier-qualifier-list
before 'spe_context_ptr_t'
./src/video/ps3/SDL_ps3video.c:78: error: expected '=', ',', ';', 'asm' or
'__attribute__' before 'fb_writer_spu'
./src/video/ps3/SDL_ps3video.c: In function 'PS3_VideoInit':
./src/video/ps3/SDL_ps3video.c:167: error: 'spu_data_t' has no member named
'program'
./src/video/ps3/SDL_ps3video.c:167: error: 'fb_writer_spu' undeclared (first
use in this function)
[...]

The problem is that the ps3 video driver is being build regardless if libspe2
is installed or not. The attached patch fixes this problem.

[ Note: The other headers that are being checked for are coming from the linux
headers package on powerpc64. As a result these are also available if you're on
a generic powerpc64 machine and not a PS3 only. ]
Comment 1 Sam Lantinga 2009-12-11 07:15:05 UTC
Your fix is in subversion, thanks!