Page MenuHomePhabricator
Feed Advanced Search

Jan 14 2021

aleb added a comment to T12404: Implement a first-party SMTP client.

Recently smtp-relay.gmail.com stopped accepting email from our Phabricator instance because it turns out Phabricator was sending HELO localhost.localdomain instead of HELO smtp-relay.gmail.com when doing the SMTP connection.

Jan 14 2021, 6:45 PM · Infrastructure, Mail

Jan 7 2019

aleb added a comment to T13227: Figure out if Google Auth needs to be updated before Google+ shutdown on March 7, 2019.

There is a Migrate from Google+ Sign-In howto for web apps.

Jan 7 2019, 9:05 AM · Auth

Aug 14 2017

aleb added a comment to T12742: `arc land` shows only part of the error that causes the final squash to fail.

The error truncating behavior affects also Diffusion > R > Status, where only part of the stderr is shown, making it impossible to debug the problem without hacking Phabricator:

ab@phabricator-1-vm:/opt/bitnami$ diff apps/phabricator/libphutil/src/future/exec/CommandException.php.original apps/phabricator/libphutil/src/future/exec/CommandException.php
59c59
<     $limit = 1000;
---
>     $limit = 1000000000;
Aug 14 2017, 11:09 AM · Bug Report

Aug 10 2017

aleb updated the question details for Q631: Why does `arc land` truncate the output it receives from `git commit`?.
Aug 10 2017, 3:26 PM · Arcanist
aleb updated the answer details for Q631: Why does `arc land` truncate the output it receives from `git commit`? (Answer 561).
Aug 10 2017, 3:26 PM
aleb added Q631: Why does `arc land` truncate the output it receives from `git commit`? (Answer 561).
Aug 10 2017, 3:25 PM

Jun 28 2017

aleb asked Q631: Why does `arc land` truncate the output it receives from `git commit`?.
Jun 28 2017, 3:17 PM · Arcanist

May 22 2017

aleb added a comment to T12742: `arc land` shows only part of the error that causes the final squash to fail.

Arcanist hides part of the stderr to prevent console spam. But what problem does this fix exactly? How is it a problem that 5GB of output are printed to stderr? The way I see it, if the tool called by arc prints a lot of info to stderr, it should be assumed it is printed for a good reason, be it 1KB or 5PB.

May 22 2017, 6:32 PM · Bug Report
aleb added a comment to T12742: `arc land` shows only part of the error that causes the final squash to fail.

The same as what you do with any tool which produces 5GB of output: wonder what happened, then try again and redirect the output to a file and deal with it. Currently the user has to run manually the steps printed when using --trace, and then they will get to that 5GB output anyway.

May 22 2017, 3:32 PM · Bug Report
aleb created T12742: `arc land` shows only part of the error that causes the final squash to fail.
May 22 2017, 2:30 PM · Bug Report

Apr 3 2017

aleb added a comment to T12498: Cannot get the details about the reviewers of a revision through Conduit.

We need to check programmatically whether all the reviewers approved. Any suggestion how this could be done?

Apr 3 2017, 4:02 PM · Feature Request
aleb added a comment to T12497: Cannot set a mailing list user to watch a project.

Could you reopen and set it as feature request?

Apr 3 2017, 3:55 PM · Mailing Lists, Bug Report
aleb created T12498: Cannot get the details about the reviewers of a revision through Conduit.
Apr 3 2017, 3:11 PM · Feature Request
aleb created T12497: Cannot set a mailing list user to watch a project.
Apr 3 2017, 2:57 PM · Mailing Lists, Bug Report

Mar 21 2017

aleb added a comment to T4200: Building OS packages and install scripts.

Commenting here since you merged T12429. I find surprising you put Arcanist in the same bucket as Phabricator. I think you don't even have to do all the work to make Arcanist users happy. I might be wrong, but I think there will be people adding support for Arcanist to their favorite package manager (I'm ignoring Windows), if only they'd have tags or marked releases to work with?

Mar 21 2017, 5:52 PM · Guides, Setup
aleb created T12429: Make it easier to install and update arcanist on Mac.
Mar 21 2017, 12:13 PM · Feature Request

Oct 20 2016

aleb added a comment to T11036: Put subproject columns on workboards.

Ok, thanks for the explanation!

Oct 20 2016, 2:19 PM · Workboards (v3), Projects (v3)

Oct 18 2016

aleb added a comment to T11036: Put subproject columns on workboards.

As you described, the model would be very complex considering a task can be in more than one subprojects, or a milestone. Why is it needed to have separate columns for subprojects in a parent project's workboard?

Oct 18 2016, 10:44 AM · Workboards (v3), Projects (v3)

Apr 8 2016

chad awarded Arcanist Workflows a Pterodactyl token.
Apr 8 2016, 5:38 PM

Mar 20 2016

aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

Why is the list of diffs in a task page ordered by creation time?

Mar 20 2016, 10:05 PM · Maniphest, Feature Request

Mar 18 2016

aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

We have not seen any evidence that installs which do not use git-phab experience this problem,

Mar 18 2016, 2:04 PM · Maniphest, Feature Request

Jan 23 2016

aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

I think there is a misunderstanding. The problem is not caused by git-phab, it only becomes more visible with git-phab.

Jan 23 2016, 6:15 PM · Maniphest, Feature Request

Jan 5 2016

aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

Could you please reconsider this task considering what I said in the previous comment?

Jan 5 2016, 11:21 AM · Maniphest, Feature Request

Dec 15 2015

aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

We use git-phab, a tool which creates a task per branch and keeps the commits in the branch and the diffs of the task in sync. https://phabricator.freedesktop.org/diffusion/GITPHAB/ What this means is that it allows us to change the order of the diffs more easily, so in our case it will happen more often!

Dec 15 2015, 9:45 PM · Maniphest, Feature Request
aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

When the reviewer looks at the task to review the diffs, he needs to start somewhere and end somewhere. This is where the diffs list help: the reviewer will start at the top and finish at the bottom. Currently the diffs are ordered by creation time. Normally this works ok because the newer diffs depend on the older diffs. But if a newer diff is made a dependency of an older diff, the order of the list is not relevant.

Dec 15 2015, 6:52 PM · Maniphest, Feature Request
aleb added a comment to T9986: I registered and I got a "No Create Permission" error.
In T9986#149260, @chad wrote:

Afterwards I clicked on Create Task and it worked, it did not give me any other error.

Where, specifically. "Create Task" is a permission-locked form on this install.

Dec 15 2015, 3:33 PM · Bug Report
aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

Sorry, those two pages are ridiculously long, I won't read them.

Dec 15 2015, 3:29 PM · Maniphest, Feature Request
aleb added a comment to T9986: I registered and I got a "No Create Permission" error.

Not sure what you mean. Afterwards I clicked on Create Task and it worked, it did not give me any other error.

Dec 15 2015, 3:24 PM · Bug Report
aleb added a comment to T9986: I registered and I got a "No Create Permission" error.

Can you provide steps on how to reproduce this bug, it's not clear from your report. See Contributing Bug Reports.

Dec 15 2015, 11:40 AM · Bug Report
aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

For example here D506 should be between D560 and D561, not at the top:
https://phabricator.freedesktop.org/T3491

Dec 15 2015, 11:33 AM · Maniphest, Feature Request
aleb created T9987: Diffs in a task should be sorted by the dependency tree.
Dec 15 2015, 9:35 AM · Maniphest, Feature Request
aleb created T9986: I registered and I got a "No Create Permission" error.
Dec 15 2015, 9:32 AM · Bug Report