[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: plasma-mobile and metadata.desktop files
From:       Sebastian =?utf-8?q?K=C3=BCgler?= <sebas () kde ! org>
Date:       2011-07-08 10:31:36
Message-ID: 201107081231.36682.sebas () kde ! org
[Download RAW message or body]

Hi,

[Put active@kde.org in CC:]

On Friday, July 08, 2011 10:48:07 Aaron J. Seigo wrote:
> today i started poking a bit more deeply into the plasma-mobile
> repository.  this is the first in a series of emails as a result :)

I feel some house-cleaning is coming up :-)

> a lot of stuff is written with QML and using Plasma::Packages. awesome :) 
> however, the metada.desktop files in the packages are duplicated with the 
> "proper" name (e.g. plasma-applet-somethingorother.desktop).
> 
> this is messy, error prone and translators will hate us. so i added a
> simple  two line macro to the top level CMakeLists.txt file call
> install_metadata_file. you can now just do:
> 
> install_metadata_file(package/metadata.desktop 
>                                      plasma-applet-appletstrip.desktop)
> 
> and it all magically works without a copy of the .desktop file in the
> source  repo.

I've just recently started to install the metadatafile like that using CMake's 
RENAME, which is also easy enough, not sure if we really need a macro here, I 
assume it just does the renaming?

> finally, we really need to get away from names like plasma-applet-
> appletstrip.desktop .. they should be universally unique and probably
> should  be more like plasma-applet-org.kde.appletstrip.desktop.

Good style suggestion.

> finally, a question: why are the items in plasma-mobile/containments/
> install  with a plasma-applet-foo.desktop instead of a
> plasma-containment-foo.desktop?

No really good reason, although in the webdashboard case (the other is applet 
strip), it can be used as a containment or as an applet.

Cheers,
-- 
sebas

http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]