Bug 287212

Summary: kcontrol installs keyboard test files in wrong location
Product: [Applications] systemsettings Reporter: Andrea Scarpino <scarpino>
Component: kcm_keyboardAssignee: Andriy Rysin <arysin>
Status: RESOLVED FIXED    
Severity: minor CC: kde, rdieter
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Andrea Scarpino 2011-11-22 00:03:54 UTC
Version:           unspecified (using Devel) 
OS:                Linux

I built the latest kde-workspace from git and it installed these files in a wrong location.

$ ls /build/src/build/kcontrol/keyboard/tests/config/
base.1.1.xml  base.bad.xml  base.xml

I built it in a chroot, and /build/src/build is the dir in which I build kde-workspace.

My cmake options are:
        cmake ../kde-workspace \
        -DCMAKE_BUILD_TYPE=Release \
        -DCMAKE_SKIP_RPATH=ON \
        -DCMAKE_INSTALL_PREFIX=/usr \
        -DWITH_Xmms=OFF \
        -DWITH_Googlegadgets=OFF \
        -DWITH_libgps=OFF \
        -DWITH_OpenGLES=ON \
        -DKWIN_BUILD_WITH_OPENGLES=ON


Reproducible: Always

Steps to Reproduce:
build the kde-workspace from git

Actual Results:  
Everything works but those files are installed in a wrong location.

Expected Results:  
Files are installed in the right location.

KDE from git
Comment 1 Nicolas L. 2011-11-22 01:28:06 UTC
this is because there is an error in kcontrol/keyboard/tests/CMakeLists.txt

install( DIRECTORY config DESTINATION ${CMAKE_CURRENT_BINARY_DIR} )

what should be the good behaviour ? not to install test ? or if we need to install, where should it be ?
Comment 2 Christophe Marin 2011-11-22 01:53:24 UTC
Git commit ecb88c2d999b5a69eda67b72b8e2a74cb56ee331 by Christophe Giboudeaux.
Committed on 22/11/2011 at 02:53.
Pushed by cgiboudeaux into branch 'master'.

Don't install files in the build dir.
BUG: 287212

M  +3    -1    kcontrol/keyboard/tests/CMakeLists.txt

http://commits.kde.org/kde-workspace/ecb88c2d999b5a69eda67b72b8e2a74cb56ee331