KDE/kdevplatform

[Andreas Pakulat]

On 14.07.07 18:01:22, Dukju Ahn wrote:
> SVN commit 687905 by dukjuahn:
> 
> Fix the parent app name of envsetting kcm from kdevelop to kdevplatformproject.
> Also, slight modification in projectCtrl to load kdevplatformproject kcm.
> 
> 
>  M  +2 -2      project/settings/kcm_kdev_envsettings.desktop  
>  M  +3 -1      shell/projectcontroller.cpp  
> 
> 
> --- trunk/KDE/kdevplatform/project/settings/kcm_kdev_envsettings.desktop #687904:687905
> @@ -7,8 +7,8 @@
>  X-KDE-ModuleType=Library
>  X-KDE-Library=kdev_envsettings
>  X-KDE-FactoryName=kcm_kdev_envsettings
> -X-KDE-ParentApp=kdevelop
> -X-KDE-ParentComponents=kdevelop
> +X-KDE-ParentApp=kdevplatformproject
> +X-KDE-ParentComponents=kdevplatformprojecta

This is not fully right. The parent app should be kdevplatform (see
shell/settings/kcm_kdev_uisettings.desktop).

>  Name=Environment
> --- trunk/KDE/kdevplatform/shell/projectcontroller.cpp #687904:687905
> @@ -79,7 +79,9 @@
>              return;
>          if( !m_cfgDlgs.contains( proj ) )
>          {
> -            m_cfgDlgs[proj] = new KSettings::Dialog( findPluginsForProject( proj ),
> +            QStringList pluginsForPrj = findPluginsForProject( proj );
> +            pluginsForPrj << "kdevplatformproject"; // for project-wide env settings.
> +            m_cfgDlgs[proj] = new KSettings::Dialog( pluginsForPrj,

And this is not needed then anymore.

Andreas

-- 
You could live a better life, if you had a better mind and a better body.