Review Request: plasma-netbook: fix for mid to netbook renaming

Roderick Greening roderick.greening at gmail.com
Thu Aug 13 02:02:57 CEST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1303/
-----------------------------------------------------------

(Updated 2009-08-13 00:02:57.317900)


Review request for Plasma and Artur de Souza (MoRpHeUz).


Changes
-------

Updated to reflect current situation and testing.


Summary (updated)
-------

As plasma-netbook was transitioned from being called MID to Netbook, not all things were updated to account for this. Specifically, in this case, was the minimal kwin window decorations for use in plasma-netbook. 

This patch updates the references to names, files, namespaces and classes to Netbook (from Mid).


Diffs
-----

  trunk/playground/base/plasma/netbook/kwin/CMakeLists.txt 1010511 
  trunk/playground/base/plasma/netbook/kwin/mid/CMakeLists.txt 1010511 
  trunk/playground/base/plasma/netbook/kwin/mid/mid.cpp 1010511 
  trunk/playground/base/plasma/netbook/kwin/mid/mid.desktop 1010511 
  trunk/playground/base/plasma/netbook/kwin/mid/mid.h 1010511 
  trunk/playground/base/plasma/netbook/kwin/netbook/CMakeLists.txt PRE-CREATION 
  trunk/playground/base/plasma/netbook/kwin/netbook/netbook.desktop PRE-CREATION 
  trunk/playground/base/plasma/netbook/kwin/netbook/netbook.h PRE-CREATION 
  trunk/playground/base/plasma/netbook/kwin/netbook/netbook.cpp PRE-CREATION 

Diff: http://reviewboard.kde.org/r/1303/diff


Testing (updated)
-------

The patch is a fairly simple one, only updating the names to the correct new naming scheme. Builds and compiles as it previously did.

Unfortunately, the window decoration is still broken, and does not load, which is the same behaviour exhibited when the plugin was called kwin3_mid.so. I expect there is some other issue preventing it from loading properly.

This patch is still necessary to move forward with the naming change.


Thanks,

Roderick



More information about the Plasma-devel mailing list