Closed Bug 1585435 Opened 6 years ago Closed 6 years ago

Broken error message when trying to submit a patch to phabricator

Categories

(Conduit :: moz-phab, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1563606

People

(Reporter: jfkthame, Unassigned)

Details

I was intending to upload a patch (from a contributor, originally attached in bugzilla) to phabricator so as to be able to trigger autoland for it.

However, moz-phab submit failed for me. I believe the failure was because the commit message specified me as reviewer, and it doesn't recognize that this is appropriate because the actual author is someone else (as recorded in the hg commit).

But not only did it fail to submit the patch; it also ran into an internal error when trying to give an error message (I assume):

Submitting 1 commit for review:
(New) 556822:de7a3eadff9a Bug 1401575 - Update Bulgarian hyphenation dictionary. r=jfkthame
Checking commits.. /de7a3eadff9a is based off non-public commit d565e8905d20
Warning: found 181 untracked files (will not be submitted):
Submit to Phabricator (YES/No/Always)? y

Creating new revision:
556822:de7a3eadff9a Bug 1401575 - Update Bulgarian hyphenation dictionary. r=jfkthame
Linting...
No lint engine configured for this project.
Running unit tests...
No unit test engine is configured for this project.
 SKIP STAGING  Phabricator does not support staging areas for this repository.
 Exception 
ArgumentCountError: Too few arguments to function PhutilErrorHandler::handleError(), 4 passed in /app/phabricator/externals/extensions/sentry/sentry/src/ErrorHandler.php on line 361 and exactly 5 expected
(Run with `--trace` for a full exception trace.)
CalledProcessError: Command '['/Users/jkew/.mozbuild/moz-phab/arcanist/bin/arc', 'diff', '--base', 'arc:this', '--allow-untracked', '--no-amend', '--no-ansi', '--message-file', '/var/folders/jq/5dbj29lj10l_thg8t9kq73qc0000gq/T/tmpkZMWar', '--create']' returned non-zero exit status 1
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.