[PATCH] D37951: [zorg] Allow specification of svn update mode remotely

Galina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 13:06:07 PDT 2017


gkistanova accepted this revision.
gkistanova added a comment.
This revision is now accepted and ready to land.

Not sure the naming is good, though. Since this is a build property, it might be a good idea to be a bit more specific of what kind of mode is that, what do you think?
How about "checkout_mode" or "svn_mode"?

And just as a generic note. With this patch only some builders would accept this property.


https://reviews.llvm.org/D37951





More information about the llvm-commits mailing list