hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kevin Wilfong <>
Subject Re: Problems with Arc/Phabricator
Date Wed, 11 Apr 2012 01:27:47 GMT
+John Sichi as he did a lot of the work on getting Hive to use Phabricator.

Is there anything preventing people from using Review Board for patches now, except maybe
convention?  As for applying patches, in either case, you can download the patch from the
JIRA site.

To be clear, I am not trying to aggressively force Phabricator on Hive, these are just suggestions.

From: Carl Steinbach <<>>
Date: Tue, 10 Apr 2012 18:10:44 -0700
To: Microsoft Office User <<>>
Cc: "<>" <<>>
Subject: Re: Problems with Arc/Phabricator

Ok, I'll submit an issue. But then what? Who is going to fix it? Perhaps in the meantime we
should just switch back to using reviewboard while we wait for Arc and Phabricator to mature
as open source projects.

On Tue, Apr 10, 2012 at 5:58 PM, Kevin Wilfong <<>>
There's a list of ways to report issues/get help on the side bar of this

It suggests #phabricator <irc://<>>
as the best
place to get help with issues.

On 4/10/12 5:42 PM, "Carl Steinbach" <<>>

>Arc won't let me apply the patch for HIVE-2646. I've also noticed some
>other problems (e.g. missing TEST PLAN in checkin comments), and was
>wondering how we can get these problems resolved? Since most of this work
>was done at Facebook, who is the designated point of contact for
>Arc/Phabricator issues going forward?
>Here's the error I get when applying HIVE-2646:
>% arc patch D2133 --trace
>Loading phutil library 'arc_jira_lib' from
>>>> [0] <exec> $ (cd '/Users/carl/Work/repos/hive-test'; git rev-parse
><<< [0] <exec> 13,599 us
>>>> [1] <conduit> differential.getdiff()
><<< [1] <conduit> 608,237 us
>>>> [2] <exec> $ (cd '/Users/carl/Work/repos/hive-test/'; git apply
><<< [2] <exec> 13,178 us
>Fatal error: Uncaught exception 'CommandException' with message 'Command
>'(cd '/Users/carl/Work/repos/hive-test/'; git apply --index --reject)'
>failed with error #128:
><stdin>:10: trailing whitespace.
><stdin>:111: trailing whitespace.
>    <fileset dir="${build.ivy.lib.dir}/default" includes="*.jar"
>fatal: corrupt patch at line 609
>' in /Users/carl/.local/pkg/libphutil/src/future/exec/ExecFuture.php:346
>Stack trace:
>#1 /Users/carl/.local/pkg/arcanist/scripts/arcanist.php(266):
>#2 {main}
>  thrown in
>on line 346

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message