<div dir="ltr">New patch attached.  <div><br></div><div>GLEW detection will be somewhat spotty I suspect, the basic FindGLEW module built into CMake doesn't really cover it so I added some coverage to a custom one and added it to the cmake/modules subdir.  It was good enough for mine, and users can manually specify if they need to.<div><br></div></div><div>I haven't seen that any of the other changes ariovistus made were necessary to get gr-fosphor going, so I didn't include them.  Some are needed for earlier compiler versions, but not MSVC 2015.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Apr 24, 2016 at 1:52 AM, 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>
<span class=""><br>
> Do you mean in xxx_sink_c_impl, or are you thinking that GR would do it in<br>
> it's initial GUI setup?<br>
<br>
</span>In the xxx_sink_c_impl. Although since this is common this should<br>
probably be in the base_sink, most likely just after the glctx_init()<br>
call<br>
<span class=""><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 and<br>
>> automatically set it up.<br>
><br>
><br>
> Sure.  I don't disagree, but I'm not sure what other configurations would<br>
> require Glew, so since I couldn't vouch for anything but Win10-64/MSVC, I<br>
> left it as a manual option that can then be specified on the CMake command<br>
> line (which is what my build scripts are doing now).  If you can point in to<br>
> what configurations overall would need this, then happy to submit a new<br>
> change.<br>
<br>
</span>I don't think this is compiler specific so probably anything on<br>
windows would require it.<br>
<br>
I found out someone else tried to build gr-fosphor with MSVC a while<br>
back but he never to upstream :<br>
<br>
<a href="https://github.com/ariovistus/gr-fosphor/search?utf8=%E2%9C%93&q=glew" rel="noreferrer" target="_blank">https://github.com/ariovistus/gr-fosphor/search?utf8=%E2%9C%93&q=glew</a><br>
<br>
You can see there the GLEW things he had to do ... not sure if they're required.<br>
<br>
Cheers,<br>
<br>
   Sylvain<br>
<br>
</blockquote></div><br></div>