Created attachment 2730 [details]
k3b: fix FTBFS

Attached patch fixes both issues.

About the artsc stuff, the problem is that include dir is "/opt/trinity/include/artsc", but TDE_INCLUDE_DIR is "/opt/trinity/include/tde" and ARTS_INCLUDE_DIRS is "/opt/trinity/include/tde/arts" .
Notice that artsc is installed outside the "tde" subfolder.

Francois, Slavek,
wouldn't it be better if we change the install destionation of artsc.h and artsc_export.h from /opt/trinity/include/artsc to /opt/trinity/include/arts?
Or is there a special reason why artsc include files are installed separately?

anyhow it is quite strange there are no issues in building under Debian.
What options are you passing to cmake? Just wondering.

Hello, you can check the entire build log for r14.0.x here:
https://build.opensuse.org/public/build/home:PunisherHD:Trinity:r14.0.x/CentOS_6/i586/k3b/_log
(same problem in r14.1.0)

The cmake command was:
/usr/bin/cmake -DCMAKE_VERBOSE_MAKEFILE=ON -DCMAKE_INSTALL_PREFIX:PATH=/usr -DCMAKE_INSTALL_LIBDIR:PATH=/usr/lib -DINCLUDE_INSTALL_DIR:PATH=/usr/include -DLIB_INSTALL_DIR:PATH=/usr/lib -DSYSCONF_INSTALL_DIR:PATH=/etc -DSHARE_INSTALL_PREFIX:PATH=/usr/share -DCMAKE_SKIP_RPATH:BOOL =ON -DBUILD_SHARED_LIBS:BOOL=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo '-DCMAKE_C_FLAGS=-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i686 -mtune=atom -fasynchronous-unwind-tables -I/usr/include/ffmpeg -DNDEBUG' '-DCMAKE_CXX_FLAGS=-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i686 -mtune=atom -fasynchronous-unwind-tables -I/usr/include/ffmpeg -DNDEBUG' -DCMAKE_SKIP_RPATH=OFF -DCMAKE_INSTALL_RPATH=/opt/trinity/lib -DCMAKE_NO_BUILTIN_CHRPATH=ON -DCMAKE_VERBOSE_MAKEFILE=ON -DCMAKE_PROGRAM_PATH=/opt/trinity/bin -DWITH_GCC_VISIBILITY=OFF -DCMAKE_INSTALL_PREFIX=/opt/trinity -DBIN_INSTALL_DIR=/opt/trineity/bin -DCONFIG_INSTALL_DIR=/etc/trinity -DINCLUDE_INSTALL_DIR=/opt/trinity/include/tde -DLIB_INSTALL_DIR=/opt/trinity/lib -DSHARE_INSTALL_PREFIX=/opt/trinity/share -DWITH_ALL_OPTIONS=ON -DWITH_MUSICBRAINZ=ON -DWITH_FFMPEG_ALL_CODECS=ON -DWITH_MUSEPACK=ON ..

I'm not sure why the artsc headers are outside TDE's include directory.
In fact, I believe that originally, both 'arts' and 'artsc' directories were under '/opt/trinity/include' directly (not tde subfolder), and that there was a mistake during the cmake conversion.

(In reply to Michele Calgaro from comment #3)
> anyhow it is quite strange there are no issues in building under Debian.
> What options are you passing to cmake? Just wondering.

On Debian and Ubuntu variable INCLUDE_INSTALL_DIR is not explicitly specified for CMake builds and therefore is used the default value ${CMAKE_INSTALL_PREFIX/include} == without '/tde' at the end.

(In reply to Michele Calgaro from comment #2)
> Francois, Slavek,
> wouldn't it be better if we change the install destionation of artsc.h and
> artsc_export.h from /opt/trinity/include/artsc to /opt/trinity/include/arts?
> Or is there a special reason why artsc include files are installed
> separately?

My plan is to adjust the patch so that it was used artsc-config to determine the exact path of artc inculdes. In future will be better instead of artsc-config binary to prepare appropriate pkgconfig file.

Or it would be better to focus on the future and prepare pkgconfig file now? What is your opinion?

In my opinion better to focus on future right now, avoid doing two things when only one is enough. Perhaps we can put this bug for R14.0.5 unless you think it will be a very quick fix

(In reply to Michele Calgaro from comment #7)
> In my opinion better to focus on future right now, avoid doing two things
> when only one is enough. Perhaps we can put this bug for R14.0.5 unless you
> think it will be a very quick fix

Ok, I will give this direction. I assume that the patches will be ready quickly.

Part from attachment 2730 [details] to fix ffmpeg includes pushed to GIT in hash a30375c6 (master) and bd93d759 (r14.0.x).

Part to fix artsc includes I reworked and pushed to GIT in hash 1602d89a (master) and b1200055 (r14.0.x).

For CMake build assumes an updated arts - with pkg-config file for artsc.