<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 4, 2015 at 11:04 AM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">CCing the mailing list.<br>
<br>
On 4 May 2015 at 02:23, Gehad <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
<span class="">> On 05/02/2015 07:14 PM, Lubomir I. Ivanov wrote:<br>
>> On 29 April 2015 at 02:00, Gehad <<a href="mailto:gehadelrobey@gmail.com">gehadelrobey@gmail.com</a>> wrote:<br>
>>><br>
>>> This is my fork:<br>
>>> <a href="https://github.com/Gehadelrobey/subsurface" target="_blank">https://github.com/Gehadelrobey/subsurface</a><br>
>>> I'll push to 'custom-print' branch and rebase it with master regularly.<br>
>>><br>
>><br>
>> hello Gehad,<br>
>> here are some comments on the two commits that i see.<br>
>><br>
>> <a href="https://github.com/Gehadelrobey/subsurface/commit/781f0f98167096c045df1426667078962b96f101" target="_blank">https://github.com/Gehadelrobey/subsurface/commit/781f0f98167096c045df1426667078962b96f101</a><br>
>><br>
>> CSS coding style related<br>
>><br>
>> .innerContainer {<br>
>>     height: 85%;<br>
>>     ...<br>
>> }<br>
>><br>
>> note that:<br>
>> - the opening brace is on the same line as the class name<br>
>> - "height : ..." should be without space after "height" and before<br>
>> ":". space only after ":".<br>
>><br>
><br>
</span>> Fixed and pushed to the branch.<br>
<br>
thanks, looks good.<br>
<span class=""><br>
><br>
>> <a href="https://github.com/Gehadelrobey/subsurface/commit/af2376ff2e9277d938d29c95c3999443f8023c56" target="_blank">https://github.com/Gehadelrobey/subsurface/commit/af2376ff2e9277d938d29c95c3999443f8023c56</a><br>
>><br>
>> i need to see if if this works from me (on win32):<br>
>> find_package(Grantlee5)<br>
>><br>
>> for instance, it finds marble using find_package, but the resulted<br>
>> variables were bogus if i recall correctly.<br>
>><br>
>> even if AFAIK both libraries don't use pkg-config we may have to add<br>
>> the optional:<br>
>> GRANTLEE_FROM_PKGCONFIG<br>
>> MARBLE_FROM_PKGCONFIG<br>
>><br>
><br>
</span>> I changed the cmake file a little bit.<br>
><br>
> -find_package(Grantlee5)<br>
> -set(GRANTLEE_LIBRARIES Grantlee5::Templates)<br>
> +if(GRANTLEE_FROM_PKGCONFIG)<br>
> +       pkg_config_library(LIBGRANTLEE libgrantlee-templates5 REQUIRED)<br>
> +       set(GRANTLEE_LIBRARIES "")<br>
> +else()<br>
> +       find_package(Grantlee5)<br>
> +       set(GRANTLEE_LIBRARIES Grantlee5::Templates)<br>
> +endif()<br>
><br>
> I installed libgrantlee-dev but couldn't find the package with<br>
> pkg-config, So I didn't succeed to link subsurface to libgrantlee5.<br>
><br>
><br>
<br>
that's probably because marble and grantlee AFAIK don't use pkg-config.<br>
<span class=""><br>
like i said:<br>
> unlike cmake, qmake allowed "PKGCONFIG += libmarble", which was very handy.<br>
<br>
</span>i need to figure a way how to approach the cmake issue before i<br>
continue reviewing, as i simply can't compile ATM.<br>
then i also need to look into the rest of the patches that you have pushed.<br></blockquote><div><br></div><div>If you use windows - Open cmake-gui and point the source dir ( the one that has CMakeLists.txt ) on the  project line edit,<br></div><div>and select your build dir.<br></div><div><br>>From the drop down menu, select your generator ( I'm guessing it's a mingw-generator ) <br><br></div><div>Then click configure<br></div><div>It will fail, but it will show you what failed, then you can click on the failed fields to point it to the right location.<br><br></div><div>each of those failed fields is a -DFAILED_FIELD_NAME=c:/path/to/something  that you can add to the command line if you prefer .<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>
lubomir<br>
--<br>
_______________________________________________<br>
subsurface mailing list<br>
<a href="mailto:subsurface@subsurface-divelog.org">subsurface@subsurface-divelog.org</a><br>
<a href="http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface" target="_blank">http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface</a><br>
</div></div></blockquote></div><br></div></div>