Re: [llvm-dev] Upgrading phabricator

2016-10-04 Thread Eric Liu via cfe-commits
This is not due to the upgrade. There was long delay before Phabricator receives a commit, but the delay seems to be gone now. Let me know if you are still experiencing the delay. - Eric On Mon, Oct 3, 2016 at 4:25 PM Michał Górny wrote: > On Mon, 3 Oct 2016 13:47:08 + > Sjoerd Meijer via c

Re: [llvm-dev] Upgrading phabricator

2016-10-03 Thread Michał Górny via cfe-commits
On Mon, 3 Oct 2016 13:47:08 + Sjoerd Meijer via cfe-commits wrote: > I just committed to Clang and noticed that the corresponding Phabricator > ticket does not get automatically closed (I have "Differential Revision:" in > my commit message and believe I didn't make a typo). > Is it just me

RE: [llvm-dev] Upgrading phabricator

2016-10-03 Thread Sjoerd Meijer via cfe-commits
g to LLVM this still seems to work. Cheers, Sjoerd. -Original Message- From: llvm-dev [mailto:llvm-dev-boun...@lists.llvm.org] On Behalf Of Alex Bradbury via llvm-dev Sent: 30 September 2016 15:45 To: Eric Liu Cc: llvm-dev; Clang Dev; Eric Liu via llvm-commits; cfe-commits Subject: Re: [llvm

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Mehdi Amini via cfe-commits
> On Sep 29, 2016, at 11:04 PM, Eric Liu wrote: > > I've switched the default email format to be plain text only now. This option > should be per-user configurable, but somehow it is not shown in the > "Settings"; I'll try if I can make the option personalized. > > Regarding new features and

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Eric Liu via cfe-commits
@Alex I tried glancing through emails in llvm-commits, and I found "[PATCH]" makes it easier to tell revisions from patches, and I guess people are also used to "[PATCH]" after all these years :) @Zachary Yes! This option can be configured in "Email Preferences" in your personal setting, but I'm n

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Zachary Turner via cfe-commits
Is there any way to disable emails when a revision is closed? I always find those annoying, especially when it's a mass closing to clean out your dashboard On Fri, Sep 30, 2016 at 7:45 AM Alex Bradbury via llvm-dev < llvm-...@lists.llvm.org> wrote: > On 30 September 2016 at 14:21, Eric Liu via llv

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Alex Bradbury via cfe-commits
On 30 September 2016 at 14:21, Eric Liu via llvm-commits wrote: > Thanks for the feedback Aaron! :) > > I've disabled it. I think the annoying part really is the status (e.g. > Request, Closed etc) in the tag, and I am wondering if a tag with just line > numbers like "(N Loc)" would be better. But

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Aaron Ballman via cfe-commits
On Fri, Sep 30, 2016 at 9:21 AM, Eric Liu wrote: > Thanks for the feedback Aaron! :) > > I've disabled it. I think the annoying part really is the status (e.g. > Request, Closed etc) in the tag, and I am wondering if a tag with just line > numbers like "(N Loc)" would be better. But I'm not really

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Eric Liu via cfe-commits
Thanks for the feedback Aaron! :) I've disabled it. I think the annoying part really is the status (e.g. Request, Closed etc) in the tag, and I am wondering if a tag with just line numbers like "(N Loc)" would be better. But I'm not really sure about the trade-off here. - Eric On Fri, Sep 30, 20

Re: [llvm-dev] Upgrading phabricator

2016-09-30 Thread Aaron Ballman via cfe-commits
On Fri, Sep 30, 2016 at 2:04 AM, Eric Liu via cfe-commits wrote: > I've switched the default email format to be plain text only now. This > option should be per-user configurable, but somehow it is not shown in the > "Settings"; I'll try if I can make the option personalized. Thank you for workin

Re: [llvm-dev] Upgrading phabricator

2016-09-29 Thread Eric Liu via cfe-commits
I've switched the default email format to be plain text only now. This option should be per-user configurable, but somehow it is not shown in the "Settings"; I'll try if I can make the option personalized. Regarding new features and bug fixes in this upgrade, I don't really have a list since the P

Re: [llvm-dev] Upgrading phabricator

2016-09-29 Thread Mehdi Amini via cfe-commits
One of the new “feature” is that emails are HTML only right now. Not quite nice for the archive (or for interacting by email). See for instance: http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160926/172081.html (Also the funky "[Changed Subscribers] “ in the title) Another issue is

Re: [llvm-dev] Upgrading phabricator

2016-09-29 Thread Sanjoy Das via cfe-commits
It looks like the new phabricator sends html email by default. I personally prefer text email. What do others think? Is this configurable in the new installation? Thanks, -- Sanjoy Zachary Turner via llvm-commits wrote: You mentioned this was for an upgrade. Are there any major new features

Re: [llvm-dev] Upgrading phabricator

2016-09-29 Thread Zachary Turner via cfe-commits
You mentioned this was for an upgrade. Are there any major new features or bugfixes to be aware of? On Thu, Sep 29, 2016 at 9:26 PM Eric Liu via llvm-commits < llvm-comm...@lists.llvm.org> wrote: > Hi all, > > Phabricator is (finally) back online! Let me know if you have any feedback > or problem

Re: [llvm-dev] Upgrading phabricator

2016-09-29 Thread Eric Liu via cfe-commits
Hi all, Phabricator is (finally) back online! Let me know if you have any feedback or problem :) Thanks, Eric On Thu, Sep 29, 2016 at 10:23 PM Eric Liu wrote: > According to top and iotop, mysqld is still working, and the progress bar > did move by a little bit, so I think it's just really slo