hhellyer added a comment. Should I be able to deliver these changes now? When I try following the instructions here: http://llvm.org/docs/Phabricator.html the patch downloads and applies correctly but whether I use arc via the git or svn commit methods I'm ultimately prompted for a password for subversion. I'm not sure if I should have a password or if that means the patch doesn't match what's expected in some way. I don't know how clever phabricator's svn commit hooks are and whether they check against the patch before prompting for authentication.
(Or is there a "Land this" button on phabricator I'm missing?) From: Greg Clayton <clayb...@gmail.com> To: Howard Hellyer/UK/IBM@IBMGB, clayb...@gmail.com Date: 13/06/2016 21:28 Subject: Re: [PATCH] http://reviews.llvm.org/D20565: Add MemoryRegionInfo to SB API clayborg accepted this revision. This revision is now accepted and ready to land. http://reviews.llvm.org/D20565 Unless stated otherwise above: IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU http://reviews.llvm.org/D20565 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
