Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
tag: conduit-4
Commits on May 20, 2012
  1. @epriestley

    Dispatch a "diff.willBuildMessage" event from Arcanist

    epriestley authored
    Summary:
    Hive has custom integration which relies on prefilling a bunch of information from JIRA. This is currently broken and accomplished in a roundabout way. Instead, add an event that the integration can listen for. See next diff.
    
    @Girish, I guess you're official Phabricator/Hive support now?
    
    Test Plan: Ran the Hive/JIRA workflows in Git and SVN, see next diff.
    
    Reviewers: Girish, btrahan, ashutoshc
    
    Reviewed By: btrahan
    
    CC: aran
    
    Maniphest Tasks: T1206
    
    Differential Revision: https://secure.phabricator.com/D2449
Commits on May 19, 2012
  1. Say until when the reviewers are away

    vrana authored
    Test Plan: `arc diff` with reviewers away.
    
    Reviewers: btrahan, epriestley
    
    Reviewed By: epriestley
    
    CC: aran, epriestley
    
    Differential Revision: https://secure.phabricator.com/D2496
Commits on May 18, 2012
  1. Warn user about specifying reviewers that are away

    vrana authored
    Summary:
    There are different options how to implement this:
    We can also generate the warning in `validateField()` and handle it in all callsites.
    This is sufficient for me and simple enough.
    
    Test Plan: `arc diff` revision with reviewer away/not away.
    
    Reviewers: btrahan, jungejason
    
    Reviewed By: jungejason
    
    CC: epriestley, aran
    
    Differential Revision: https://secure.phabricator.com/D2493
Commits on May 17, 2012
  1. @epriestley

    Merge pull request #28 from phleet/hg-multiple-heads

    epriestley authored
    Make arc diff work with multiple heads in hg
  2. @jlfwong
  3. @epriestley

    Merge pull request #27 from phleet/hg-bookmark

    epriestley authored
    Make arc diff not crash on hg bookmarks
Commits on May 16, 2012
  1. @jlfwong
Commits on May 12, 2012
  1. Provide user-friendly error in arc diff --update

    Nick Harper authored
    Summary:
    If you forget to provide an argument for --update and have another argument
    following it (e.g. HEAD^), we should provide a nice error message instead
    of passing that argument through to a conduit call and then printing the
    conduit error.
    
    Test Plan: ran 'arc diff --update HEAD^' and got a nice error message
    
    Reviewers: epriestley, jungejason, vrana
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2468
Commits on May 11, 2012
  1. @epriestley

    Fix "local commits" for Git and Mercurial to align with user expectat…

    epriestley authored
    …ions, and fix a parsing bug
    
    Summary:
      - "git log" still includes "\n", so we're currently generating nonsense hashes like "\nabd9879bab86ad78ab...".
      - Correct the log range we use in Git. See comment. When users perform merges, their expectations about what the "included commits" and what the included changes are are different. Represent them with two different ranges.
      - Same deal for Mercurial
    
    Test Plan: Ran "arc which" in various contexts.
    
    Reviewers: btrahan, aurelijus, Makinde
    
    Reviewed By: Makinde
    
    CC: aran, nh, jungejason
    
    Maniphest Tasks: T873
    
    Differential Revision: https://secure.phabricator.com/D2460
  2. @epriestley

    Make "arc which" really really really verbose

    epriestley authored
    Summary: Provide far more information about what "arc diff" intends to do.
    
    Test Plan: Ran "arc which" in a variety of circumstances.
    
    Reviewers: btrahan, Makinde
    
    Reviewed By: Makinde
    
    CC: aran
    
    Maniphest Tasks: T1183
    
    Differential Revision: https://secure.phabricator.com/D2454
  3. Remove arc export's dependency on a working copy when using --diff

    Nick Harper authored
    Summary:
    When getting an encoding, we should query the server for the encoding of the
    project that we're exporting from, not the project that we're running arc in
    (arc might not be in a working copy).
    
    Test Plan: ran arc export with --diff and didn't get a workflow exception
    
    Reviewers: epriestley, jungejason, vrana, davidreuss
    
    Reviewed By: jungejason
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2455
Commits on May 10, 2012
  1. @epriestley

    arc land: add --delete-remote, raise an error for landing a branch on…

    epriestley authored
    …to itself
    
    Summary:
      - Allow users to delete the remote copy of a feature branch as well as the local one, for workflows that push feature branches. We test if the remote exists before trying to delete it.
      - Raise a better warning when you misuse "arc land".
      - I also wrote some documentation about this, see next diff.
    
    Test Plan:
      - Tried to land a branch onto itself.
      - Ran "arc land --delete-remote" on feature branches with and without remote feature branches.
    
    Reviewers: aurelijus, btrahan
    
    Reviewed By: aurelijus
    
    CC: aran
    
    Maniphest Tasks: T1204
    
    Differential Revision: https://secure.phabricator.com/D2445
  2. @epriestley

    Fix 'rev' index error for Mercurial

    epriestley authored
    Summary: I renamed this in D2437 for greater consistency with everything else, but missed this use of the old key.
    
    Test Plan: idk lmk?
    
    Reviewers: Makinde
    
    Reviewed By: Makinde
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2452
Commits on May 9, 2012
  1. @epriestley

    Ship complete commit messages to Phabricator from Arcanist

    epriestley authored
    Summary: Currently, we ship only the summary, but we need to ship the whole thing for T1189.
    
    Test Plan: Added var_dump() + die, ran in git and hg working copies, verified 'message' included the whole message.
    
    Reviewers: csilvers, btrahan, Makinde
    
    Reviewed By: csilvers
    
    CC: aran
    
    Maniphest Tasks: T1189
    
    Differential Revision: https://secure.phabricator.com/D2437
  2. @epriestley

    "arc upgrade", to automatically upgrade arc (client changes)

    epriestley authored
    Summary:
      - Try to limit the pain of //future// version bumps by making arc self-updating.
      - When the server needs a newer version, prompt the user to update.
      - (We need them to reissue their command because we may already have loaded classes which have changed in the update.)
      - Make the message sound exciting!
    
    Test Plan: Artifically bumped server forward, ran "arc list", got to upgrade!
    
    Reviewers: Makinde, nh, jungejason, btrahan
    
    Reviewed By: nh
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2435
  3. Catch ArcanistUsageException when getting git parent log info

    Nick Harper authored
    Summary:
    ArcanistDiffWorkflow::getGitParentLogInfo() calls
    ArcanistDifferentialCommitMessage::newFromRawCorpus() which may throw a
    ArcanistUsageException if the parent commit message is malformed (specifically,
    a bad "Differential Revision:" line); this should not stop arc diff.
    
    Test Plan: successfully ran arc diff where the parent commit message was malformed.
    
    Reviewers: epriestley, jungejason
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2434
Commits on May 8, 2012
  1. Allow --verbatim and -m in arc diff

    Nick Harper authored
    Summary:
    There's no reason these should be exclusive: -m is used only on update for the
    update message, and --verbatim doesn't affect the udpate message. It's also
    useful to allow both of these, if say I want to update my test plan by editing
    my git commit message and also provide a message for the update from the
    command line.
    
    Test Plan:
    ran arc diff with --verbatim and -m and saw my message from -m was used, as
    well as my updates in the commit message went through.
    
    Reviewers: jungejason, epriestley, vrana
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2426
Commits on May 7, 2012
  1. @epriestley

    Allow 'arc' to run without '.arcconfig'

    epriestley authored
    Summary:
    This is mostly an onboarding thing, but also allows "arc upload", "arc download", and "arc paste" to work anywhere on the system.
    
      - Try to read the Phabricator install URI from arc global config if we can't find ".arcconfig".
      - Build a WorkingCopy anyway if we can't find ".arcconfig", as long as we can find ".svn", ".git", or ".hg".
      - Make all the workflows handle "no project ID" at least somewhat gracefully.
    
    Test Plan:
      - Ran "arc diff" in .arcconfig-less Mercurial, Git, and Subversion working copies.
      - Ran "arc upload" and "arc download" from my desktop.
      - Ran "arc paste" from somewhere random.
      - Cleared my config and hit the error, got useful instructions.
    
    Reviewers: btrahan, csilvers
    
    Reviewed By: csilvers
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2424
  2. @epriestley

    Provide more context in "arc diff" messages in Git

    epriestley authored
    Summary: These are the unambiguously-good changes from D2388. Show commits included in a revision in the editor in "arc diff".
    
    Test Plan: Ran "arc diff", saw which commits were being included.
    
    Reviewers: nh, jungejason, vrana, btrahan
    
    Reviewed By: btrahan
    
    CC: aran
    
    Maniphest Tasks: T1183
    
    Differential Revision: https://secure.phabricator.com/D2406
  3. Always use git.default-relative-commit, if present

    Nick Harper authored
    Summary:
    Always use the value in git.default-relative-commit when getting the relative
    commit in git.
    
    Test Plan:
    Ran arc diff in a repo with git.default-relative-commit set to HEAD^ on a
    branch tracking a remote (that is different from HEAD^), and checked that
    the diff against HEAD^, not the remote, was published.
    
    Reviewers: jungejason, epriestley, vrana
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2409
  4. @epriestley

    With `--verbatim`, update some fields automatically when updating rev…

    epriestley authored
    …isions
    
    Summary:
    Essentially D2391, but with, uh, more comments?
    
      - I forgot that we already implemented shouldOverwriteWhenCommitMessageIsEdited(). This patch already behaves nearly correctly.
      - Requires changes in D2412.
      - Use `'edit' => 'edit'`, which does the same thing as `'edit' => true`, but is more correct after the "edit" / "create" split.
      - Under "--verbatim", always get the message "from the user", which means "from the working copy" because verbtatim disables the editor part.
    
    Test Plan:
          - Created and updated revisions with `arc diff`.
          - Created and updated revisions with `arc diff --verbatim`.
          - Updated revisions with `arc diff --edit`.
    
    Reviewers: jungejason, btrahan
    
    Reviewed By: jungejason
    
    CC: vrana, aran
    
    Differential Revision: https://secure.phabricator.com/D2411
  5. @epriestley

    Finish "Committed" -> "Closed" changes

    epriestley authored
    Summary:
    Clean up the remaining odds-and-ends here -- move to "differential.close", get rid of the old constant, etc.
    
    I'll wait a week or two to land this since "differential.close" just landed and all the other stuff is trivial.
    
    Test Plan: Grepped for "committed".
    
    Reviewers: btrahan, vrana, Makinde
    
    Reviewed By: btrahan
    
    CC: aran
    
    Maniphest Tasks: T909, T1182
    
    Differential Revision: https://secure.phabricator.com/D2309
  6. @epriestley

    Add "arc get-config" and "arc set-config" for managing ~/.arcrc values

    epriestley authored
    Summary:
    The major thing I want to do here is allow you to set a default Phabricator URI, so we can make "arc paste", and "arc upload", "arc download" work anywhere.
    
    We can also relax the .arcconfig requirements (request from @csilvers).
    
    Test Plan:
    Get/set some values?
    
    iiam
    
    Reviewers: btrahan, vrana, jungejason
    
    Reviewed By: btrahan
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2400
  7. @epriestley

    Remove git commit template from Arcanist

    epriestley authored
    Summary: Out of date an no longer referenced.
    
    Test Plan: Grepped for references.
    
    Reviewers: btrahan, vrana
    
    Reviewed By: btrahan
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2387
  8. @epriestley

    Explicitly check for cURL in "arc"

    epriestley authored
    Summary: We do this in Windows, but it's not available by default in Ubuntu
    
    Test Plan: Ran "arc" after faking the function name, got a reasonable error.
    
    Reviewers: btrahan, jungejason
    
    Reviewed By: btrahan
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2404
Commits on May 4, 2012
  1. @epriestley

    Clean up some "arc" edge cases in Mercurial

    epriestley authored
    Summary:
      - We no longer need color options since we fake our way through parsing ANSI colorized diffs and use HGPLAIN (on Windows, too!). Drop 'em.
      - In the case where you have nothing outgoing, we don't cache the relative commit and thus run "hg outgoing" too many times, which is fairly slow (even if you have nothing outgoing). Cache it.
    
    Test Plan: Ran "arc diff --trace" in a mercurial working copy with nothing outgoing; verified we run "hg outgoing" only once.
    
    Reviewers: Makinde, csilvers, btrahan
    
    Reviewed By: Makinde
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2399
  2. Document first line behavior in update message

    vrana authored
    Test Plan: (This diff)
    
    Reviewers: epriestley, jungejason
    
    Reviewed By: jungejason
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2389
  3. Don't ask user to use saved message in `arc diff --verbatim`

    vrana authored
    Test Plan:
    Cancel `arc diff`.
    Verify that the message is created.
    Run `arc diff --verbatim` and see no reuse message question.
    
    Reviewers: epriestley
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2390
  4. Warn before parse errors in `arc diff --verbatim`

    vrana authored
    Summary: With `--verbatim` flag, notes created from parse errors are never displayed to user resulting in blank fields.
    
    Test Plan:
    - `arc diff --verbatim` with invalid Cc
    - `arc diff --verbatim` with all fields correct
    
    Reviewers: epriestley, jungejason
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2395
Commits on May 2, 2012
  1. @indiefan @epriestley

    Updated arc land workflow to use temp file approach for messages to a…

    indiefan authored epriestley committed
    …void newline escaping issue in windows.
    
    Test Plan: Ran arc land locally with both the mutable default option and with the --merge flag to ensure that messages are set properly.
    
    Reviewers: epriestley
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2372
  2. @epriestley

    Fix "HGPLAIN" environmental variable in Windows

    epriestley authored
    Summary: In Windows, you can't use `X=y cmd` syntax to set variables. Use "set X=y & cmd" instead.
    
    Test Plan:
      - Ran "arc diff" in a Mercurial repo in Windows, created D2367.
      - Verified this does //not// cause 'HGPLAIN' to be set in the outer shell (where you type "arc diff").
    
    Reviewers: Makinde, tido, indiefan, btrahan
    
    Reviewed By: tido
    
    CC: aran
    
    Maniphest Tasks: T1179
    
    Differential Revision: https://secure.phabricator.com/D2368
  3. @indiefan @epriestley

    Using a temp file for git commit --amend instead of passing multi-lin…

    indiefan authored epriestley committed
    …e message to the shell. Avoids escaping issue on windows.
    
    Test Plan: Run arc diff locally, verify via git log that the commit is amended afterwards (using the mutable history paradigm)
    
    Reviewers: epriestley
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2369
  4. @edwardspeyer

    Help the analyzer find phutil_is_hiphop_runtime

    edwardspeyer authored
    Summary:
    rPHUf9ba25d188c1dcf39e4454b2c6bb058e0beeaa3e adds
    global function phutil_is_hiphop_runtime() to
    __phutil_library_init__.php.  This diff helps lint files that call that
    function.
    
    Test Plan: Lint D2365.
    
    Reviewers: epriestley
    
    Reviewed By: epriestley
    
    CC: aran, Koolvin
    
    Differential Revision: https://secure.phabricator.com/D2366
Commits on May 1, 2012
  1. @epriestley

    Don't check working copy status for "arc commit --show"

    epriestley authored
    Summary:
    We do unnecessary working copy checks under "--show", even though the working copy isn't relevant.
    
    Also, 'sourcePath' may not be set (e.g., "arc commit --show --revision X" where X is some "--only" revision).
    
    Test Plan: Ran "arc commit --show --revision 1" against some test data, got clean output.
    
    Reviewers: vrana, btrahan, jungejason
    
    Reviewed By: vrana
    
    CC: aran
    
    Differential Revision: https://secure.phabricator.com/D2353
  2. @epriestley

    Remove "--no-decorate"

    epriestley authored
    Summary: Apparently my advice here was terrible, and `--no-decorate` and `--decorate=no` are both very recent additions to Git which a bunch of users don't have. Get rid of them since D2344 allows us to parse all decorate levels anyway.
    
    Test Plan: Tried to google "git changelog", got a bunch of pages about managing changelogs with git.
    
    Reviewers: zeeg, ehren
    
    Reviewed By: ehren
    
    CC: ehren, aran, NorthIsUp
    
    Differential Revision: https://secure.phabricator.com/D2354
Something went wrong with that request. Please try again.