<div dir="ltr"><div class="gmail_default" style="font-size:small">Ben,</div><div class="gmail_default" style="font-size:small">I was actually thinking of having a look at it last night because of all the changes. Thanks for already doing this!<br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Much appreciated.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Regards</div><div class="gmail_default" style="font-size:small">   Andrew</div><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jan 12, 2019 at 7:21 AM Ben Boeckel <<a href="mailto:ben.boeckel@kitware.com">ben.boeckel@kitware.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sat, Jan 12, 2019 at 07:13:02 +1100, Andrew Maclean wrote:<br>
> I am surprised and amazed this works with the new system.<br>
<br>
I updated it since it was requested here. It also turned out to be quite<br>
helpful for determining real dependencies of things in ParaView and<br>
examples.<br>
<br>
> When I wrote it, it:<br>
> 1) Only looked at the header files in your code and, from their name,<br>
> worked out the module to use. Ad a consequence of this we had to manually<br>
> include e.g. some modules like the OpenGL stuff.<br>
<br>
That's pretty much true still.<br>
<br>
> 2) Does detect modules excluded from the wrapping.<br>
<br>
Not hard now; it's still just a keyword in `vtk.module`. Though it<br>
doesn't do so right now.<br>
<br>
> 3) It was written before external modules were introduced so it may not<br>
> properly work with them.<br>
<br>
It only cares about `vtk.module` files (just like it cared only about<br>
`module.cmake` before. If it can find them in the path given, I don't<br>
think it should matter too much.<br>
<br>
> The intent was to provide to the user a guide as to what modules should be<br>
> included so they may still have to add extras.<br>
<br>
Yeah. I imagine the hard-coded if-then extensions were done because of<br>
this. Doing `IMPLEMENTABLE`/`IMPLEMENTS` logic should make it work for<br>
any module.<br>
<br>
--Ben<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">___________________________________________<br>Andrew J. P. Maclean<br><br>___________________________________________</div>