<div dir="ltr">Hello Lubomir,<br><br>As GSoC coding period have begun, I wanted to update you with my progress till now, I've pushed some more commits to my git branch, I will push some more commits on the weekend as I am having my finals exams now. BTW Have you managed to build subsurface with Grantlee yet or you still getting the same linking error?<br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0px 0px 0px 40px;border:none;padding:0px"><div dir="ltr">Gehad</div></blockquote></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 12, 2015 at 9:09 PM, Gehad Elrobey <span dir="ltr"><<a href="mailto:gehadelrobey@gmail.com" target="_blank">gehadelrobey@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Tue, May 12, 2015 at 11:24 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"><span>On 7 May 2015 at 09:28, Lubomir I. Ivanov <<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>> wrote:<br>
</span><span>> Dirk and others, what do you think - should we make this new library<br>
> dependency (Grantlee) optional like we do with Marble?<br>
> Should the printing be completely disabled if the user decides to<br>
> build with NO_GRANTLEE or should we fall back to the current Qt-only<br>
> based printing (creates bloat as we are going to have 2 printing<br>
> modules)?<br>
><br>
> also GSoC wise, do you think the new printing code should enter master<br>
> while WIP or should it do so all at once when mostly done?<br>
><br>
<br>
</span>hello Gehad,<br>
<br>
to summarize what Drik just confirmed:<br>
- we use NO_GRANTLEE and align it to NO_PRINTING i.e. if Grantlee is<br>
not used we don't allow printing.<br>
- only one printing module at a time. we only work on the new printing<br>
module and replace the old one completely!<br>
- we can still use Github for the reviews, but we are going to make<br>
pull requests more often, so that people can test the code in master.<br>
- once i review certain patches, i'm going to pull them in my github,<br>
sign them off and send the pull requests myself<br>
<br>
having said that i need to find more free time to do the reviews!<br>
<br>
lubomir<br>
--<br>
</blockquote></div><br></div></div>Hello Lubomir,</div><div class="gmail_extra"><br></div><div class="gmail_extra">ACK, thanks for the clarification.<span class="HOEnZb"><font color="#888888"><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">regards,</div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div dir="ltr"><div>Gehad</div></div></div></blockquote></div></div>
</div>