Bug 289777 - [PATCH] kded_keyboard fails to build (final): redefinition of "const char* LIST_SEPARATOR"
Summary: [PATCH] kded_keyboard fails to build (final): redefinition of "const char* LI...
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_keyboard (show other bugs)
Version: 4.9.4
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Andriy Rysin
URL:
Keywords: reproducible
Depends on:
Blocks:
 
Reported: 2011-12-25 11:51 UTC by esigra
Modified: 2013-02-07 03:50 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
patch that fixes the build (3.56 KB, patch)
2011-12-25 11:51 UTC, esigra
Details

Note You need to log in before you can comment on or make changes to this bug.
Description esigra 2011-12-25 11:51:59 UTC
Created attachment 67094 [details]
patch that fixes the build

Version:           unspecified (using KDE 4.7.3) 
OS:                Linux

kcm_keyboard fails to build with KDE4_ENABLE_FINAL=ON becasue LIST_SEPARATOR is defined in several places.

Reproducible: Always

Steps to Reproduce:
1. tar jxvf /usr/portage/distfiles/kde-workspace-4.7.3.tar.bz2
2. cd kde-workspace-4.7.3
3. mkdir build
4. cd build
5. cmake -DKDE4_ENABLE_FINAL=ON ..
6. make kded_keyboard

Actual Results:  
In file included from /home/work/programmering/kde-workspace-4.7.3/build/kcontrol/keyboard/kded_keyboard_final_cpp.cpp:8:0:                                                                                 
/home/work/programmering/kde-workspace-4.7.3/kcontrol/keyboard/keyboard_config.cpp:27:20: error: redefinition of 'const char* LIST_SEPARATOR'
/home/work/programmering/kde-workspace-4.7.3/kcontrol/keyboard/layout_memory_persister.cpp:44:20: error: 'const char* LIST_SEPARATOR' previously defined here

Expected Results:  
successful build
Comment 1 Andriy Rysin 2013-02-07 03:50:23 UTC
I've pushed a fix for this in 4.10.1: 22e0d313a304beb5916da85c8daa96cb105a78bd 
Unfortunately misspelled BUG tag so I'll close it manually.