On Mon, Aug 6, 2012 at 11:16 PM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank" class="cremed">dgregor@apple.com</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="im"><br>
On Aug 6, 2012, at 1:11 PM, Sebastian Pop <<a href="mailto:spop@codeaurora.org" class="cremed">spop@codeaurora.org</a>> wrote:<br>
<br>
> Ping^4.<br>
><br>
> Hi, I'm waiting for a month now for an OK to commit.<br>
> It seems that really nobody except me cares for this change.<br>
> Does it really matter to somebody if I'm pushing this change out to svn?<br>
><br>
> How should I proceed in timeout cases like this?<br>
<br>
</div>Patch looks good, please go ahead and commit. If this happens again, please CC me. If I can't review the patch, I'll find someone who can. Sorry about that!</blockquote><div><br></div><div>It's also largely my fault. I've been neglecting CMake patches. =/ Sebastian pinged me on IRC and I still didn't get to it.</div>
<div><br></div><div>FWIW, the first patch makes perfect sense.</div><div><br></div><div>The second patch I'm not as clear on... I feel like either:</div><div><br></div><div>1) We shouldn't build (or test) c-index-test and friends if they necessitate a static library and that isn't supported in the CMake configuration, or</div>
<div><br></div><div>2) We should link c-index-test and friends dynamically when explicitly requested to do so.</div><div><br></div><div><br></div><div>I think I prefer #2. What do you think?</div></div></div>