<div dir="ltr">On Tue, Apr 14, 2015 at 5:41 AM, Steve Borho <span dir="ltr"><<a href="mailto:steve@borho.org" target="_blank">steve@borho.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><span style="color:rgb(34,34,34)">I'm open to suggestions (perhaps LEGACY_WINDOWS) but you have to keep in</span><br></div></div>
mind the cmake cache which keeps old options around until you manually<br>
delete the cache or delete the build folder. So WINXP_SUPPORT will still<br>
show up in the list of options for everyone for quite some time, and<br>
they might be quite confused if they turned on WINXP_SUPPORT and it had<br>
no effect.<br>
<br>
We could change the logic to be something like this:<br>
<br>
option(LEGACY_WINDOWS "Make binaries compatible with Windows XP and Vista" OFF)<br>
if(LEGACY_WINDOWS OR WINXP_SUPPORT)<br>
 ...<br>
<br>
which keeps the old option still effective, but then we have two options<br>
that have the same effect. This is why I left the old option name in<br>
place and just updated the description to mention XP and Vista.<br></blockquote><div><br></div><div>Maybe I'm being too picky. Your patch looks OK, you could just go with it and see how many users actually have an issue with it, might end up being no-one at all.</div><div><br></div><div>Tim</div></div></div></div>