<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:12.8px">Any code in lib/ shouldn't have to care about doing this.</span> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:12.8px">Any </span><span style="font-size:12.8px">required setup must be done in the GR C++ part</span></blockquote><div><br></div><div>Do you mean in xxx_sink_c_impl, or are you thinking that GR would do it in it's initial GUI setup?  </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"> <span style="font-size:12.8px">Also using a magic 'USING_GLEW' that must somehow be setup manually is </span> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:12.8px">not acceptable. Need some CMake logic to detect when this is required </span><span style="font-size:12.8px">and automatically set it up.</span></blockquote><br style="font-size:12.8px"><div>Sure.  I don't disagree, but I'm not sure what other configurations would require Glew, so since I couldn't vouch for anything but Win10-64/MSVC, I left it as a manual option that can then be specified on the CMake command line (which is what my build scripts are doing now).  If you can point in to what configurations overall would need this, then happy to submit a new change.  </div><div><br></div><div>Geof</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Apr 23, 2016 at 9:28 PM, Sylvain Munaut <span dir="ltr"><<a href="mailto:246tnt@gmail.com" target="_blank">246tnt@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Thanks for the patches. Comments below.<br>
<span class=""><br>
<br>
> 1- Allows compilation on MSVC by conditionally adding __attribute(format<br>
> tags based on the compiler<br>
<br>
</span>I merged an equivalent patch, it'll be pushed whenever I push my next<br>
series of update for fosphor.<br>
<span class=""><br>
<br>
> 2- Allows builders to use Glew when required, making a GlewInit call when<br>
> USING_GLEW is defined. If that is not defined, there are no changes.<br>
<br>
</span>Any code in lib/ shouldn't have to care about doing this. It expects<br>
that all the required GL setup has been done prior to calling it. Any<br>
required setup must be done in the GR C++ part<br>
<br>
Also using a magic 'USING_GLEW' that must somehow be setup manually is<br>
not acceptable. Need some CMake logic to detect when this is required<br>
and automatically set it up.<br>
<br>
<br>
Cheers,<br>
<br>
   Sylvain<br>
<br>
</blockquote></div><br></div>