<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Sep 12, 2013, at 8:48 , Jordan Rose <<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Sep 12, 2013, at 8:31 , Argyrios Kyrtzidis <<a href="mailto:akyrtzi@gmail.com">akyrtzi@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>On Sep 11, 2013, at 7:32 PM, David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">This commit causes no-op builds to relink which is slow. Why was this changed in the first place?</div></blockquote><div><br></div><div>Yes, I find this awful. Why don't we generate a temporary file and replace SVNVersion.inc only if it is different ?</div></div></blockquote><br></div><div>I tried several things here, but the underlying issue is that CMake doesn't know to do some action before recalculating dependencies. That means that if you update your repository and it doesn't touch any files in clangBasic, you get an updated SVNVersion.inc but not a rebuilt Clang. It's only on the <i>next</i> build that the change will be picked up, at which point you've already realized the problem.</div><div><br></div><div>I can add a guard option that turns this behavior on and off, but I couldn't think of a better answer that would just work.</div></div></blockquote><br></div><div>I've made this optional (defaulting to off) in r190613.</div><div><br></div><div>Jordan</div><br></body></html>