By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 2488 - Build issue: tdeaddons FTBFS on RHEL 5 because SDL is not detected
Summary: Build issue: tdeaddons FTBFS on RHEL 5 because SDL is not detected
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdeaddons (show other bugs)
Version: R14.0.1 [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks: R14.0.2
  Show dependency tree / graph
 
Reported: 2015-07-28 06:07 CDT by Francois Andriot
Modified: 2015-08-30 07:37 CDT (History)
4 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
tdeaddons : fix SDL detection (654 bytes, patch)
2015-07-28 06:11 CDT, Francois Andriot 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Francois Andriot 2015-07-28 06:07:39 CDT 
Hello, on older distribution (e.g. RHEL 5), the SDL library comes without the precious "pkgconfig/sdl.pc" file.
Then, the SDL library is not detected by tdeaddons/ConfigureChecks.cmake , and the build fails.

The cmake detection should be able to fall back to detecting the SDL.h or SDL.so file when sdl.pc does not exists.

See amarok/ConfigureChecks.cmake for an example.
Comment 1 Francois Andriot 2015-07-28 06:11:12 CDT 
Created attachment 2541 [details]
tdeaddons : fix SDL detection
Comment 2 Michele Calgaro 2015-07-28 17:53:55 CDT 
Francois,
at first sight the patch looks good. Please hold on pushing the patch until R14.0.1 packages are fully built.
Comment 3 Slávek Banko 2015-08-30 07:37:19 CDT 
Pushed to GIT in hash 19183b5d (master) and d1e0006b (r14.0.x).