[llvm-commits] [PATCH] Improved svn repo searching for 'make update'

Eric Christopher echristo at gmail.com
Sat Jan 26 12:33:21 PST 2013


Sure. If Benjamin is ok with it go ahead, I've just not had time to look.

-eric


On Sat, Jan 26, 2013 at 12:31 PM, Benjamin Kramer <benny.kra at gmail.com>wrote:

>
>   This is biting users all around and should go into trunk asap.
>
>
> ================
> Comment at: Makefile:255
> @@ +254,3 @@
> +define SUB-SVN-DIRS
> +function svnup() {
> +  dirs=`svn status --no-ignore $1 | awk '/I|\?      / {print $2}' |
> LC_ALL=C xargs svn info 2>/dev/null | awk '/^Path:\ / {print $2}'`;
> ----------------
> can you drop the function keyword? It's bash-specific and not needed.
>
>
> http://llvm-reviews.chandlerc.com/D326
>
> BRANCH
>   update_fix
>
> ARCANIST PROJECT
>   llvm
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130126/05091944/attachment.html>


More information about the llvm-commits mailing list