<br><br><div class="gmail_quote">On Wed, Mar 20, 2013 at 8:54 AM, Simeon Bird <span dir="ltr"><<a href="mailto:bladud@gmail.com" target="_blank">bladud@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey Vishesh,<br>
<br>
I took a look at this this evening - it all looks fine and works as<br>
expected for me.<br>
And it is plainly a good idea :).<br>
The code looks good to me too, so consider this my "ship it!".<br>
<br>
Only things that broke are:<br>
1) the kcm and nepomukcontroller didn't recognise the file indexing<br>
service anymore<br></blockquote><div><br>fixed.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) nepomukctl needs updating.<br></blockquote><div><br>fixed<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
but these should be easy enough to fix.<br>
<div class="HOEnZb"><div class="h5"><br>
Simeon<br>
<br>
> On 10 March 2013 14:08, Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>> wrote:<br>
>> Hey Simeon<br>
>><br>
>> I've been meaning to implement a Nepomuk::Service2 class which would serve<br>
>> as an upgrade from the normal Service class. Given the problems we were<br>
>> having with properly shutting down the storage service, I decided to<br>
>> implement it. It's currently in the feature/service2 branch in nepomuk-core.<br>
>><br>
>> The main difference is that now each service installs its own process,<br>
>> instead of installing a library which is then loaded by the<br>
>> nepomukservicestub. This results in somewhat lighter processes, and it makes<br>
>> debugging a lot easier as we will no longer get reports that say<br>
>> "nepomukservicestub" is acting up.<br>
>><br>
>> Additionally, it simplifies the entire parent child relationship for each<br>
>> service. Now each service is the top most parent QObject, and it is deleted<br>
>> after the QApplication::exec finishes. No more wacky static variables.<br>
>><br>
>> Problems -<br>
>> 1. I've kinda broken the /servicecontrol name and description functions.<br>
>> Implementing this would require parsing the desktop files, which I still<br>
>> haven't done.<br></div></div></blockquote><div><br>Still trying to fix this. I'll merge after fixing this and some more testing.<br><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
>> 2. The nepomukservicestub used to catch interrupts and exit gracefully -<br>
>> This isn't the case any more. I'm not sure why.<br></div></div></blockquote><div><br>fixed this by installing a custom signal handler<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
>><br>
>> Do you think you could have a look at the code when you get some time?<br>
>> Please :)<br>
>><br>
>> I was also thinking of renaming the nepomukserver to nepomuk_control, since<br>
>> it isn't really a server, but I'm afraid that might clash with the<br>
>> nepomukcontroller.<br></div></div></blockquote><div><br>Any comments on renaming nepomukserver? <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb">
<div class="h5">
>><br>
>> --<br>
>> Vishesh Handa<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><span style="color:rgb(192,192,192)">Vishesh Handa</span><br>