Issue 124157 - CoinMP header should be included with "coin/" if SYSTEM_COINMP is defined
Summary: CoinMP header should be included with "coin/" if SYSTEM_COINMP is defined
Status: UNCONFIRMED
Alias: None
Product: Calc
Classification: Application
Component: code (show other issues)
Version: 4.1.0-dev
Hardware: All All
: P3 Normal (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-31 19:07 UTC by Ted Ralphs
Modified: 2014-01-31 19:07 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Ted Ralphs 2014-01-31 19:07:02 UTC 
When the option --with-system-coinmp is used in building Open Office and the include directory is determined by pkgconfig, the directory already includes the trailing "coin/" and thus, including "coin/CoinMP.h" fails. This inclusion works fine if the header file is in a system location, since CoinMP installs into <prefix>/coin by default. The issue is with these lines in sccomp/source/solver/solver.cxx:

#ifdef SYSTEM_COINMP
#include <coin/CoinMP.h>
#else
#include <coinmp/CoinMP.h>
#endif

The second line should be

#include <CoinMP.h>

if the include directory is determined by pkg-config (which is currently how things work).