Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 190925 - media-gfx/lprof needs patch for lcms-1.17
Summary: media-gfx/lprof needs patch for lcms-1.17
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Printing Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-31 23:06 UTC by Hal Engel
Modified: 2007-10-10 18:43 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
lcms-1.17.patch (lcms-117.patch,348 bytes, patch)
2007-08-31 23:07 UTC, Hal Engel
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Hal Engel 2007-08-31 23:06:27 UTC
Because of changes in the type defs in LCMS 1.17 it is no longer API compatible with earlier versions.  As a result at least one application, LProf, will not compile.  LProf makes use of LCMS is ways that most applications do not and as a result it very tightly coupled to LCMS.  So it may be the only application currently in portage that will exhibit this problem.  A fix would be to patch src/liblprof/lcmsprf.h by adding the following:

#if LCMS_VERSION > 116
typedef int BOOL;
#endif

There is already a patch for this available on the LProf sourceforge patches tracker at 

http://sourceforge.net/tracker/index.php?func=detail&aid=1786078&group_id=146038&atid=764372


Reproducible: Always

Steps to Reproduce:
1. emerge =media-libs/lcms-1.17
2. emerge lprof

This will fail with a compile error about BOOL not being defined.

3.

Actual Results:  
LProf ebuild fails.

Expected Results:  
LProf ebuild works.
Comment 1 Hal Engel 2007-08-31 23:07:37 UTC
Created attachment 129735 [details, diff]
lcms-1.17.patch

Here is the patch from the lprof patch tracker.
Comment 2 Stefan Schweizer (RETIRED) gentoo-dev 2007-10-10 18:43:28 UTC
Thank you, fixed. Sorry for taking so long.