<br><br><div class="gmail_quote">On Thu, Mar 14, 2013 at 4:21 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">
I mean, if we just remove the conditionals and do<br>
<div class="im"><br>
#define DMS_DBUS_SERVICE (qgetenv("NEPOMUK_FAKE_DMS_DBUS_SERVICE").isEmpty() ?<br>
"org.kde.nepomuk.DataManagement" :<br>
qgetenv("NEPOMUK_FAKE_DMS_DBUS_SERVICE").constData())<br>
<br>
</div>in release mode, this will be slow for all the people *not* running<br>
the test, who will have to look up a variable all the time for no<br>
reason.<br></blockquote><div><br>I just looked at the code. I see what you mean.<br><br>Do you think it would be better to just discard the entire test and move it into the test framework? It's basically checking if the dbus interface works. With the nepomuk test library, we actually run our own dbus-server, so we won't need any of these fake dbus service hacks. <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>
On 13 March 2013 18:32, Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>> wrote:<br>
><br>
><br>
> On Thu, Mar 14, 2013 at 3:36 AM, Simeon Bird <<a href="mailto:bladud@gmail.com">bladud@gmail.com</a>> wrote:<br>
>><br>
>> The reason the test is failing is that it runs in release mode and we<br>
>> have in genericdatamanagementjob_p.h:<br>
>><br>
>> #ifndef NDEBUG<br>
>> #define DMS_DBUS_SERVICE<br>
>> (qgetenv("NEPOMUK_FAKE_DMS_DBUS_SERVICE").isEmpty() ?<br>
>> "org.kde.nepomuk.DataManagement" :<br>
>> qgetenv("NEPOMUK_FAKE_DMS_DBUS_SERVICE").constData())<br>
>> #else<br>
>> #define DMS_DBUS_SERVICE "org.kde.nepomuk.DataManagement"<br>
>> #endif<br>
>><br>
>> so it only pays attention to the fake data management service in debug<br>
>> mode.<br>
>><br>
>> I'm not sure how to fix this...getenv is probably too expensive in<br>
>> release mode, right?<br>
><br>
><br>
> Well, it is a test. How does it matter if it is expensive?<br>
><br>
>><br>
>><br>
>> Simeon<br>
>><br>
>><br>
>> On 11 March 2013 14:31, Vishesh Handa <<a href="mailto:me@vhanda.in">me@vhanda.in</a>> wrote:<br>
>> > To mailing list people<br>
>> ><br>
>> > I know these emails might be a tad bit annoying, but they will only be<br>
>> > sent<br>
>> > if some test is failing or if nepomuk is not building. Currently 3 tests<br>
>> > are<br>
>> > failing. Until they are fixed, we will probably get an email each time<br>
>> > someone pushes something to the stable branch.<br>
>> ><br>
>> > It would be awesome if someone could fix the tests! :D<br>
>> ><br>
>> ><br>
>> > On Mon, Mar 11, 2013 at 11:58 PM, KDE CI System <<a href="mailto:null@kde.org">null@kde.org</a>> wrote:<br>
>> >><br>
>> >> See <<a href="http://build.kde.org/job/nepomuk-core_stable/changes" target="_blank">http://build.kde.org/job/nepomuk-core_stable/changes</a>><br>
>> >><br>
>> >> _______________________________________________<br>
>> >> Nepomuk mailing list<br>
>> >> <a href="mailto:Nepomuk@kde.org">Nepomuk@kde.org</a><br>
>> >> <a href="https://mail.kde.org/mailman/listinfo/nepomuk" target="_blank">https://mail.kde.org/mailman/listinfo/nepomuk</a><br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > --<br>
>> > Vishesh Handa<br>
>> ><br>
>> > _______________________________________________<br>
>> > Nepomuk mailing list<br>
>> > <a href="mailto:Nepomuk@kde.org">Nepomuk@kde.org</a><br>
>> > <a href="https://mail.kde.org/mailman/listinfo/nepomuk" target="_blank">https://mail.kde.org/mailman/listinfo/nepomuk</a><br>
>> ><br>
><br>
><br>
><br>
><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>