Review Request: Nepomuk implementation for the comic applet

[Aaron Seigo]

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


code wide looks nice, and afaik is proper usage of nepomuk. 

build system wise ... it might be nice to make it optional rather than required, though. i'm ok with it being a hard requirement, but not everyone may appreciate that. at the very least, the check for nepomuk should happen in kdeplasma-addons/CMakeLists.txt and if it isn't found then the comic plasmoid shouldn't be build. even better would be a bit of #ifdef in the source code to make it truly optional (the library link line can remain; the nepomuk vars will just be "" in the not-there case, which is fine)

- Aaron


On 2009-02-28 10:13:07, Matthias Fuchs wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/207/
> -----------------------------------------------------------
> 
> (Updated 2009-02-28 10:13:07)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> When manually saving comics (context menu) additional information is stored using Nepomuk.
> 
> I'm not sure if the ontologies I use are the right ones (like subject) for what I do. Neither am I sure if the comic strip name (like XKCD) should be a tag itself rather than only the subject.
> 
> Please also look at the CMakeLists.txt file if I did it correctly.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeplasma-addons/applets/comic/CMakeLists.txt 933345 
>   /trunk/KDE/kdeplasma-addons/applets/comic/comic.cpp 933345 
> 
> Diff: http://reviewboard.kde.org/r/207/diff
> 
> 
> Testing
> -------
> 
> Seems to work nice.
> 
> 
> Thanks,
> 
> Matthias
> 
>