Bug 305759 - Build fails due to missing header, although CMake reports that all external packages have been found
Summary: Build fails due to missing header, although CMake reports that all external p...
Status: RESOLVED WORKSFORME
Alias: None
Product: kmag
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: sarang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-25 13:00 UTC by Frank Reininghaus
Modified: 2012-09-24 21:14 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frank Reininghaus 2012-08-25 13:00:03 UTC
Just tried to build KMag from git master. Compilation aborts with the following error:

In file included from /home/kde-devel/kde/src/KDE/kmag/kmagzoomview.cpp:53:0:
/home/kde-devel/kde/include/kdeaccessibilityclient/accessibleobject.h:28:35: fatal error: atspi/atspi-constants.h: No such file or directory

But CMake reported that "All external packages have been found". Looks like a check is missing.

BTW, I could not find any package in OpenSuse that provides that header with a quick search. There is a package libatspi0, but it contains only .so files, no headers.

Reproducible: Always
Comment 1 Christoph Feck 2012-08-25 16:09:14 UTC
at-spi2-core-devel ?
Comment 2 Frank Reininghaus 2012-08-26 08:15:11 UTC
Thanks Christoph, that package indeed contains this header file. However, the build still aborts with the same error message. The header file is located at

/usr/include/at-spi-2.0/atspi/atspi-constants.h

and apparently cannot be found there by the compiler. So it seems that not only the CMake check, but also the include dirs should probably be fixed.
Comment 3 Frederik Gladhorn 2012-09-24 08:33:05 UTC
Can you pull libkdeaccessibilityclient and also post here what "pkg-config  --cflags atspi-2" outputs for you?
(Right now it probably doesn't compile because I just renamed the lib according to the review comments, but KMag should be updated in two hours or so...)
Comment 4 Frank Reininghaus 2012-09-24 21:14:14 UTC
It builds fine now, thanks!

(In reply to comment #3)
> Can you pull libkdeaccessibilityclient and also post here what "pkg-config 
> --cflags atspi-2" outputs for you?

~/kde/src/KDE/libkdeaccessibilityclient$ pkg-config --cflags atspi-2
-I/usr/include/at-spi-2.0 -I/usr/include/dbus-1.0 -I/usr/lib64/dbus-1.0/include