<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 4, 2015 at 8:13 PM, 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 class="">On 4 June 2015 at 20:02, Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
> On Thu, Jun 04, 2015 at 05:58:10PM +0300, Lubomir I. Ivanov wrote:<br>
>> So this is the initial work from Gehad. I'm looking forward to getting this in<br>
>> master as he needs to continute the work into the template specific logic.<br>
>> Rebasing all the time while waiting on me for reviews must be a pain, so this<br>
>> is pending.<br>
><br>
> OK, I added a few comments, some are arguably cosmetic, some are about<br>
> hardcoding things. None should prevent me from pulling this in order to<br>
> make your life easier.<br>
><br>
> But please make it a high priority to address the comments I made on<br>
> github.<br>
><br>
<br>
</span>replies:<br>
<br>
<a href="https://github.com/torvalds/subsurface/commit/3f3937f908b5c0044aef1caa196511469103daa7#commitcomment-11525543" target="_blank">https://github.com/torvalds/subsurface/commit/3f3937f908b5c0044aef1caa196511469103daa7#commitcomment-11525543</a><br>
> Why is this hard coded to A4? This should simply use the printer's default page size<br>
<br>
intentional. i did not object about these hardcoded values for testing purposes.<br>
*any* size should be supported in the final implementation, otherwise<br>
it would be incomplete.<br>
<br>
<a href="https://github.com/torvalds/subsurface/commit/75b5d7e9e34a694a47f8ce01cf90bcf31fbc98eb#commitcomment-11525638" target="_blank">https://github.com/torvalds/subsurface/commit/75b5d7e9e34a694a47f8ce01cf90bcf31fbc98eb#commitcomment-11525638</a><br>
> I realize that using NO_PRINTING is consistent with what we do elsewhere. Still...<br>
> if(NOT NO_PRINTING) sounds really silly<br>
> let's turn this around into one if/else/endif that starts with<br>
> if(NO_PRINTING)<br>
> -- handle that case<br>
> else()<br>
> -- handle the printing case<br>
> endif()<br>
<br>
Gehad, can you fix what Dirk requests here?<br></blockquote><div><br>Sure, I ll push a fix for that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> I will however turn this code off by default until we figured out the<br>
> building of Grantlee<br>
><br>
<br>
</span>"from source" for all targets might be the correct choice to present<br>
in the docs.<br>
<span class=""><br>
>> As discussed this does break the current printing module in the expense of<br>
>> not maintaining a couple of modules (old vs new) durring GSoC.<br>
>> For users that follow master and don't want to install Grantlee please use<br>
>> "cmake -DNO_PRINTING" but mind that Grantlee will be a hard dependency if you<br>
>> ever want to print with Subsurface.<br>
><br>
> Nope, for now we'll do it the other way around. You'll need to add<br>
> -DNO_PRINTING=OFF in order to enable this in master<br>
><br>
<br>
</span>should Gehad modify the cmake file so that NO_PRINTING is ON by default?<br>
<span class=""><br>
>> Some of the patches at this point simply remove old code and add some of the<br>
>> new logic which is WIP for the time being.<br>
>><br>
>> Earlier today I've sent another patch, which is for the sake of me being able<br>
>> to build with NO_MARBLE:<br>
>> [PATCH] GlobeGPS: add empty function for NO_MARBLE<br>
><br>
> I'll add that one as well.<br>
><br>
<br>
</span>thanks.<br>
<br>
lubomir<br>
--<br>
</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></div>