<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 14, 2014 at 7:21 AM, Konrad Zemek <span dir="ltr"><<a href="mailto:konrad.zemek@gmail.com" target="_blank">konrad.zemek@gmail.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey,<br>
<div><div class="h5"><br>
2014-12-14 13:02 GMT+01:00 Myriam Schweingruber <<a href="mailto:myriam@kde.org">myriam@kde.org</a>>:<br>
> Hi all,<br>
><br>
><br>
> On Sat, Dec 13, 2014 at 11:07 PM, Matěj Laitl <<a href="mailto:matej@laitl.cz">matej@laitl.cz</a>> wrote:<br>
>><br>
>> On 13. 12. 2014 Konrad Zemek wrote:<br>
>> > gmock sources are still not packaged by distributions, and compiling<br>
>> > Amarok with tests on is still troublesome (I still use a cmake-gui<br>
>> > based approach where I manually set paths to my pre-compiled gmock<br>
>> > lib, as I outlined in an email some months ago).<br>
>> ><br>
>> > I solved the problem through the use of submodules and commited the<br>
>> > change to my personal scratch repo [1]. (...)<br>
>> ><br>
>> > If you find my approach agreeable, I will be happy to put it on<br>
>> > reviewboard.<br>
>><br>
>> Git submodule approach looks promising, however I have some concerns:<br>
>>  a) this makes test depend on 'your' github repositories; we cannot<br>
>> guarantee<br>
>> they won't go away etc.<br>
>>  b) this makes testing Amarok require internet connection, at least<br>
>> initially;<br>
>> this of shipping entire sources to build a distribution package etc.<br>
>>  c) circumvents source file checksumming etc. that many distributions do<br>
>> to<br>
>> enhance security<br>
>>  d) is it legally okay to redistribute googlemock, googletest? Using a git<br>
>> repo, shipping a tarball?<br>
>><br>
>> Still, I like the idea. a) seems easily fixable b), c) seems fixable by<br>
>> tweaking<br>
>> the way we create Amarok tarballs.<br>
>><br>
><br>
> I guess a) can be easily fixed if this goes to our git repo.<br>
> as for d) since googlemock is Free Software (New BSD 3 clause license, see<br>
> also <a href="https://code.google.com/p/googlemock/" target="_blank">https://code.google.com/p/googlemock/</a>), this shouldn't be a problem.<br>
<br>
</div></div>As for b) and c), I was imagining that `git submodule update --init`<br>
would become a standard step to fetch sources for creating a tarball<br>
or building tests. The auto-fetch is there just for convenience.<br>
<span class=""><br>
><br>
> Can we please make a release soon,  Matěj? There is one release blocker bug<br>
> which I still can reproduce and which falls in your speciality, but else we<br>
> are good for 2.9 since quite some time :)<br>
><br>
><br>
>><br>
>> > By the way, I noticed that importer tests are now guarded with<br>
>> > 'if(LINUX)' macro. There is no 'LINUX' platform in CMake, though, so<br>
>> > these tests are effectively disabled everywhere. I guess there were<br>
>> > some problems on non-linux systems?<br>
>><br>
>> Looks like a bug to me, feel free to investigate and fix, the test should<br>
>> run<br>
>> at least on Linux platforms (best if they are run everywhere).<br>
>><br>
> As for the LINUX tests: since the tests also run on Jenkins, maybe that is<br>
> the reason? subscribing Ben<br>
> Also Patrick can tell if the problem lies with building tests on Windows,<br>
> subscribing Patrick<br>
<br>
</span>Here's the commit log for the change:<br>
commit 726639b840e2a7a08fe68f04170f06b25a713c08<br>
Author: Daniel Meltzer <<a href="mailto:parallelgrapefruit@gmail.com">parallelgrapefruit@gmail.com</a>><br>
Date:   Fri May 16 20:09:17 2014 -0400<br>
Don't build the importer tests on windows either.  Same issue as mac<br>
with linking to a plugin<br>
<br>
Seems that there've been problems with linking tests on Windows and<br>
OS-X. For now I'll just change the line to `if( ${CMAKE_SYSTEM_NAME}<br>
MATCHES "Linux" )`.<br></blockquote><div><br></div><div>Yeah, we build them as plugins, and then link to the plugins.  Not possible on windows/os x.   </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
    Konrad<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Amarok-devel mailing list<br>
<a href="mailto:Amarok-devel@kde.org">Amarok-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/amarok-devel" target="_blank">https://mail.kde.org/mailman/listinfo/amarok-devel</a><br>
</div></div></blockquote></div></div></div>