<div dir="ltr">Sure. If Benjamin is ok with it go ahead, I've just not had time to look.<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jan 26, 2013 at 12:31 PM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  This is biting users all around and should go into trunk asap.<br>
<br>
<br>
================<br>
Comment at: Makefile:255<br>
@@ +254,3 @@<br>
<div class="im">+define SUB-SVN-DIRS<br>
+function svnup() {<br>
+  dirs=`svn status --no-ignore $1 | awk '/I|\?      / {print $2}' | LC_ALL=C xargs svn info 2>/dev/null | awk '/^Path:\ / {print $2}'`;<br>
</div>----------------<br>
can you drop the function keyword? It's bash-specific and not needed.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D326" target="_blank">http://llvm-reviews.chandlerc.com/D326</a><br>
<br>
BRANCH<br>
  update_fix<br>
<br>
ARCANIST PROJECT<br>
  llvm<br>
</blockquote></div><br></div>