From an unrelated thread:> ... This will need to be fixed before we can commit the patch. Since Phabricator automatically closed http://reviews.llvm.org/D12761 <http://reviews.llvm.org/D12761>, I've created a new revision. The attached diff is a very a slightly modified version of your original patch ...I have wanted to reopen a Phabricator revision too many times already, but couldn’t. I don’t know who the admin of reviews.llvm.org <http://reviews.llvm.org/> is (CC’ing Chandler as my wild guess). Could we configure it so that revisions can be reopened? Kuba -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160607/90fe37bc/attachment.html>
Tobias Grosser via llvm-dev
2016-Jun-07 19:15 UTC
[llvm-dev] Allow reopening on Phabricator
On 06/07/2016 09:14 PM, Kuba Brecka via llvm-dev wrote:> From an unrelated thread: > >> ... This will need to be fixed before we can commit the patch. Since >> Phabricator automatically closed http://reviews.llvm.org/D12761, I've >> created a new revision. The attached diff is a very a slightly >> modified version of your original patch ... > > I have wanted to reopen a Phabricator revision too many times already, > but couldn’t. I don’t know who the admin of reviews.llvm.org > <http://reviews.llvm.org> is (CC’ing Chandler as my wild guess). Could > we configure it so that revisions can be reopened?Manuel is the one to CC. Tobias
Manuel Klimek via llvm-dev
2016-Jun-08 04:16 UTC
[llvm-dev] Allow reopening on Phabricator
If you can find out whether it's possible with the latest phab, I can update our instance. On Tue, Jun 7, 2016, 9:15 PM Tobias Grosser <tobias at grosser.es> wrote:> On 06/07/2016 09:14 PM, Kuba Brecka via llvm-dev wrote: > > From an unrelated thread: > > > >> ... This will need to be fixed before we can commit the patch. Since > >> Phabricator automatically closed http://reviews.llvm.org/D12761, I've > >> created a new revision. The attached diff is a very a slightly > >> modified version of your original patch ... > > > > I have wanted to reopen a Phabricator revision too many times already, > > but couldn’t. I don’t know who the admin of reviews.llvm.org > > <http://reviews.llvm.org> is (CC’ing Chandler as my wild guess). Could > > we configure it so that revisions can be reopened? > > Manuel is the one to CC. > > Tobias > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160608/18d13b70/attachment.html>