Mercurial > hg
annotate tests/test-phabricator.t @ 47072:4c041c71ec01
revlog: introduce an explicit tracking of what the revlog is about
Since the dawn of time, people have been forced to rely to lossy introspection
of the index filename to determine what the purpose and role of the revlog they
encounter is. This is hacky, error prone, inflexible, abstraction-leaky,
<insert-your-own-complaints-here>.
In f63299ee7e4d Raphaël introduced a new attribute to track this information:
`revlog_kind`. However it is initialized in an odd place and various instances
end up not having it set. In addition is only tracking some of the information
we end up having to introspect in various pieces of code.
So we add a new attribute that holds more data and is more strictly enforced.
This work is done in collaboration with Raphaël.
The `revlog_kind` one will be removed/adapted in the next changeset. We expect
to be able to clean up various existing piece of code and to simplify coming
work around the newer revlog format.
Differential Revision: https://phab.mercurial-scm.org/D10352
author | Pierre-Yves David <pierre-yves.david@octobus.net> |
---|---|
date | Tue, 06 Apr 2021 05:20:24 +0200 |
parents | 3e381eb557f3 |
children | bf1d26bd5b6a |
rev | line source |
---|---|
39651
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
1 #require vcr |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
2 $ cat >> $HGRCPATH <<EOF |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
3 > [extensions] |
39652
d2c81e83de2a
phabricator: move extension from contrib to hgext
Augie Fackler <raf@durin42.com>
parents:
39651
diff
changeset
|
4 > phabricator = |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
5 > |
44716
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
6 > [auth] |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
7 > hgphab.schemes = https |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
8 > hgphab.prefix = phab.mercurial-scm.org |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
9 > # When working on the extension and making phabricator interaction |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
10 > # changes, edit this to be a real phabricator token. When done, edit |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
11 > # it back. The VCR transcripts will be auto-sanitised to replace your real |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
12 > # token with this value. |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
13 > hgphab.phabtoken = cli-hahayouwish |
ed81fa859426
tests: move the phabricator auth token to the global config file
Matt Harbison <matt_harbison@yahoo.com>
parents:
44715
diff
changeset
|
14 > |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
15 > [phabricator] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
16 > debug = True |
39651
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
17 > EOF |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
18 $ hg init repo |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
19 $ cd repo |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
20 $ cat >> .hg/hgrc <<EOF |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
21 > [phabricator] |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
22 > url = https://phab.mercurial-scm.org/ |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
23 > callsign = HG |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
24 > EOF |
a641fd1a1196
tests: add some basic tests of phabricator interactions
Augie Fackler <raf@durin42.com>
parents:
diff
changeset
|
25 $ VCR="$TESTDIR/phabricator" |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
26 |
45246
b1f2659c1c34
phabricator: unconditionally pop `test_vcr` to fix debugcallconduit
Ian Moody <moz-ian@perix.co.uk>
parents:
45245
diff
changeset
|
27 debugcallconduit doesn't claim invalid arguments without --test-vcr: |
45245
b4887d0680a4
phabricator: demonstrate debugcallconduit being broken without --test-vcr
Ian Moody <moz-ian@perix.co.uk>
parents:
45152
diff
changeset
|
28 $ echo '{}' | HGRCSKIPREPO= hg debugcallconduit 'conduit.ping' |
45246
b1f2659c1c34
phabricator: unconditionally pop `test_vcr` to fix debugcallconduit
Ian Moody <moz-ian@perix.co.uk>
parents:
45245
diff
changeset
|
29 abort: config phabricator.url is required |
45245
b4887d0680a4
phabricator: demonstrate debugcallconduit being broken without --test-vcr
Ian Moody <moz-ian@perix.co.uk>
parents:
45152
diff
changeset
|
30 [255] |
b4887d0680a4
phabricator: demonstrate debugcallconduit being broken without --test-vcr
Ian Moody <moz-ian@perix.co.uk>
parents:
45152
diff
changeset
|
31 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
32 Error is handled reasonably. We override the phabtoken here so that |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
33 when you're developing changes to phabricator.py you can edit the |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
34 above config and have a real token in the test but not have to edit |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
35 this test. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
36 $ hg phabread --config auth.hgphab.phabtoken=cli-notavalidtoken \ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
37 > --test-vcr "$VCR/phabread-conduit-error.json" D4480 | head |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
38 abort: Conduit Error (ERR-INVALID-AUTH): API token "cli-notavalidtoken" has the wrong length. API tokens should be 32 characters long. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
39 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
40 Missing arguments don't crash, and may print the command help |
44436
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
41 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
42 $ hg debugcallconduit |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
43 hg debugcallconduit: invalid arguments |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
44 hg debugcallconduit METHOD |
44436
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
45 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
46 call Conduit API |
44436
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
47 |
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
48 options: |
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
49 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
50 (use 'hg debugcallconduit -h' to show more help) |
46865
466236e99eac
tests: update the detailed exit codes in test-phabricator.t
Matt Harbison <matt_harbison@yahoo.com>
parents:
46037
diff
changeset
|
51 [10] |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
52 $ hg phabread |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
53 abort: empty DREVSPEC set |
44436
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
54 [255] |
09f3e003fc2a
phabricator: avoid a stacktrace when command arguments are missing
Matt Harbison <matt_harbison@yahoo.com>
parents:
44329
diff
changeset
|
55 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
56 Basic phabread: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
57 $ hg phabread --test-vcr "$VCR/phabread-4480.json" D4480 | head |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
58 # HG changeset patch |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
59 # Date 1536771503 0 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
60 # Parent a5de21c9e3703f8e8eb064bd7d893ff2f703c66a |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
61 exchangev2: start to implement pull with wire protocol v2 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
62 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
63 Wire protocol version 2 will take a substantially different |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
64 approach to exchange than version 1 (at least as far as pulling |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
65 is concerned). |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
66 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
67 This commit establishes a new exchangev2 module for holding |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
68 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
69 Phabread with multiple DREVSPEC |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
70 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
71 TODO: attempt to order related revisions like --stack? |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
72 $ hg phabread --test-vcr "$VCR/phabread-multi-drev.json" D8205 8206 D8207 \ |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
73 > | grep '^Differential Revision' |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
74 Differential Revision: https://phab.mercurial-scm.org/D8205 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
75 Differential Revision: https://phab.mercurial-scm.org/D8206 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
76 Differential Revision: https://phab.mercurial-scm.org/D8207 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
77 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
78 Empty DREVSPECs don't crash |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
79 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
80 $ hg phabread --test-vcr "$VCR/phabread-empty-drev.json" D7917-D7917 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
81 abort: empty DREVSPEC set |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
82 [255] |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
83 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
84 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
85 phabupdate with an accept: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
86 $ hg phabupdate --accept D4564 \ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
87 > -m 'I think I like where this is headed. Will read rest of series later.'\ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
88 > --test-vcr "$VCR/accept-4564.json" |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
89 abort: Conduit Error (ERR-CONDUIT-CORE): Validation errors: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
90 - You can not accept this revision because it has already been closed. Only open revisions can be accepted. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
91 [255] |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
92 $ hg phabupdate --accept D7913 -m 'LGTM' --test-vcr "$VCR/accept-7913.json" |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
93 |
45135
225588c4c255
phabupdate: allow revisions to be marked with "plan changes"
Matt Harbison <matt_harbison@yahoo.com>
parents:
44945
diff
changeset
|
94 phabupdate with --plan-changes: |
225588c4c255
phabupdate: allow revisions to be marked with "plan changes"
Matt Harbison <matt_harbison@yahoo.com>
parents:
44945
diff
changeset
|
95 |
225588c4c255
phabupdate: allow revisions to be marked with "plan changes"
Matt Harbison <matt_harbison@yahoo.com>
parents:
44945
diff
changeset
|
96 $ hg phabupdate --plan-changes D6876 --test-vcr "$VCR/phabupdate-change-6876.json" |
225588c4c255
phabupdate: allow revisions to be marked with "plan changes"
Matt Harbison <matt_harbison@yahoo.com>
parents:
44945
diff
changeset
|
97 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
98 Create a differential diff: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
99 $ HGENCODING=utf-8; export HGENCODING |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
100 $ echo alpha > alpha |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
101 $ hg ci --addremove -m 'create alpha for phabricator test €' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
102 adding alpha |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
103 $ hg phabsend -r . --test-vcr "$VCR/phabsend-create-alpha.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
104 D7915 - created - 0:d386117f30e6 tip "create alpha for phabricator test \xe2\x82\xac" (esc) |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
105 new commits: ['347bf67801e5'] |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
106 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/d386117f30e6-24ffe649-phabsend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
107 $ echo more >> alpha |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
108 $ HGEDITOR=true hg ci --amend |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
109 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/347bf67801e5-3bf313e4-amend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
110 $ echo beta > beta |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
111 $ hg ci --addremove -m 'create beta for phabricator test' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
112 adding beta |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
113 $ hg phabsend -r ".^::" --test-vcr "$VCR/phabsend-update-alpha-create-beta.json" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
114 c44b38f24a45 mapped to old nodes [] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
115 D7915 - updated - 0:c44b38f24a45 "create alpha for phabricator test \xe2\x82\xac" (esc) |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
116 D7916 - created - 1:9e6901f21d5b tip "create beta for phabricator test" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
117 new commits: ['a692622e6937'] |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
118 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/9e6901f21d5b-1fcd4f0e-phabsend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
119 $ unset HGENCODING |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
120 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
121 The amend won't explode after posting a public commit. The local tag is left |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
122 behind to identify it. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
123 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
124 $ echo 'public change' > beta |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
125 $ hg ci -m 'create public change for phabricator testing' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
126 $ hg phase --public . |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
127 $ echo 'draft change' > alpha |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
128 $ hg ci -m 'create draft change for phabricator testing' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
129 $ hg phabsend --amend -r '.^::' --test-vcr "$VCR/phabsend-create-public.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
130 D7917 - created - 2:7b4185ab5d16 "create public change for phabricator testing" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
131 D7918 - created - 3:251c1c333fc6 tip "create draft change for phabricator testing" |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
132 warning: not updating public commit 2:7b4185ab5d16 |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
133 new commits: ['3244dc4a3334'] |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
134 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/251c1c333fc6-41cb7c3b-phabsend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
135 $ hg tags -v |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
136 tip 3:3244dc4a3334 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
137 D7917 2:7b4185ab5d16 local |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
138 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
139 $ hg debugcallconduit user.search --test-vcr "$VCR/phab-conduit.json" <<EOF |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
140 > { |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
141 > "constraints": { |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
142 > "isBot": true |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
143 > } |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
144 > } |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
145 > EOF |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
146 { |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
147 "cursor": { |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
148 "after": null, |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
149 "before": null, |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
150 "limit": 100, |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
151 "order": null |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
152 }, |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
153 "data": [], |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
154 "maps": {}, |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
155 "query": { |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
156 "queryKey": null |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
157 } |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
158 } |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
159 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
160 Template keywords |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
161 $ hg log -T'{rev} {phabreview|json}\n' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
162 3 {"id": "D7918", "url": "https://phab.mercurial-scm.org/D7918"} |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
163 2 {"id": "D7917", "url": "https://phab.mercurial-scm.org/D7917"} |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
164 1 {"id": "D7916", "url": "https://phab.mercurial-scm.org/D7916"} |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
165 0 {"id": "D7915", "url": "https://phab.mercurial-scm.org/D7915"} |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
166 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
167 $ hg log -T'{rev} {if(phabreview, "{phabreview.url} {phabreview.id}")}\n' |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
168 3 https://phab.mercurial-scm.org/D7918 D7918 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
169 2 https://phab.mercurial-scm.org/D7917 D7917 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
170 1 https://phab.mercurial-scm.org/D7916 D7916 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
171 0 https://phab.mercurial-scm.org/D7915 D7915 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
172 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
173 Commenting when phabsending: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
174 $ echo comment > comment |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
175 $ hg ci --addremove -m "create comment for phabricator test" |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
176 adding comment |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
177 $ hg phabsend -r . -m "For default branch" --test-vcr "$VCR/phabsend-comment-created.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
178 D7919 - created - 4:d5dddca9023d tip "create comment for phabricator test" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
179 new commits: ['f7db812bbe1d'] |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
180 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/d5dddca9023d-adf673ba-phabsend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
181 $ echo comment2 >> comment |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
182 $ hg ci --amend |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
183 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/f7db812bbe1d-8fcded77-amend.hg |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
184 $ hg phabsend -r . -m "Address review comments" --test-vcr "$VCR/phabsend-comment-updated.json" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
185 1849d7828727 mapped to old nodes [] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
186 D7919 - updated - 4:1849d7828727 tip "create comment for phabricator test" |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
187 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
188 Phabsending a skipped commit: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
189 $ hg phabsend --no-amend -r . --test-vcr "$VCR/phabsend-skipped.json" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
190 1849d7828727 mapped to old nodes ['1849d7828727'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
191 D7919 - skipped - 4:1849d7828727 tip "create comment for phabricator test" |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
192 |
44775
cf3e07d7648a
tests: clarify a comment describing a phabricator test scenario
Matt Harbison <matt_harbison@yahoo.com>
parents:
44774
diff
changeset
|
193 Phabsend doesn't create an instability when restacking existing revisions on top |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
194 of new revisions. |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
195 |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
196 $ hg init reorder |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
197 $ cd reorder |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
198 $ cat >> .hg/hgrc <<EOF |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
199 > [phabricator] |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
200 > url = https://phab.mercurial-scm.org/ |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
201 > callsign = HG |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
202 > [experimental] |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
203 > evolution = all |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
204 > EOF |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
205 |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
206 $ echo "add" > file1.txt |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
207 $ hg ci -Aqm 'added' |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
208 $ echo "mod1" > file1.txt |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
209 $ hg ci -m 'modified 1' |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
210 $ echo "mod2" > file1.txt |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
211 $ hg ci -m 'modified 2' |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
212 $ hg phabsend -r . --test-vcr "$VCR/phabsend-add-parent-setup.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
213 D8433 - created - 2:5d3959e20d1d tip "modified 2" |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
214 new commits: ['2b4aa8a88d61'] |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
215 $ hg log -G -T compact |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
216 @ 3[tip]:1 2b4aa8a88d61 1970-01-01 00:00 +0000 test |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
217 | modified 2 |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
218 | |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
219 o 1 d549263bcb2d 1970-01-01 00:00 +0000 test |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
220 | modified 1 |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
221 | |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
222 o 0 5cbade24e0fa 1970-01-01 00:00 +0000 test |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
223 added |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
224 |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
225 Also check that it doesn't create more orphans outside of the stack |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
226 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
227 $ hg up -q 1 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
228 $ echo "mod3" > file1.txt |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
229 $ hg ci -m 'modified 3' |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
230 created new head |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
231 $ hg up -q 3 |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
232 $ hg phabsend -r ".^ + ." --test-vcr "$VCR/phabsend-add-parent.json" |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
233 2b4aa8a88d61 mapped to old nodes ['2b4aa8a88d61'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
234 D8434 - created - 1:d549263bcb2d "modified 1" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
235 D8433 - updated - 3:2b4aa8a88d61 "modified 2" |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
236 new commits: ['876a60d024de'] |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
237 new commits: ['0c6523cb1d0f'] |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
238 restabilizing 1eda4bf55021 as d2c78c3a3e01 |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
239 $ hg log -G -T compact |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
240 o 7[tip]:5 d2c78c3a3e01 1970-01-01 00:00 +0000 test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
241 | modified 3 |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
242 | |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
243 | @ 6 0c6523cb1d0f 1970-01-01 00:00 +0000 test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
244 |/ modified 2 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
245 | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
246 o 5:0 876a60d024de 1970-01-01 00:00 +0000 test |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
247 | modified 1 |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
248 | |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
249 o 0 5cbade24e0fa 1970-01-01 00:00 +0000 test |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
250 added |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
251 |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
252 Posting obsolete commits is disallowed |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
253 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
254 $ echo "mod3" > file1.txt |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
255 $ hg ci -m 'modified A' |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
256 $ echo "mod4" > file1.txt |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
257 $ hg ci -m 'modified B' |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
258 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
259 $ hg up '.^' |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
260 1 files updated, 0 files merged, 0 files removed, 0 files unresolved |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
261 $ echo 'obsolete' > file1.txt |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
262 $ hg amend --config extensions.amend= |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
263 1 new orphan changesets |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
264 $ hg log -G |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
265 @ changeset: 10:082be6c94150 |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
266 | tag: tip |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
267 | parent: 6:0c6523cb1d0f |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
268 | user: test |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
269 | date: Thu Jan 01 00:00:00 1970 +0000 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
270 | summary: modified A |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
271 | |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
272 | * changeset: 9:a67643f48146 |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
273 | | user: test |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
274 | | date: Thu Jan 01 00:00:00 1970 +0000 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
275 | | instability: orphan |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
276 | | summary: modified B |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
277 | | |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
278 | x changeset: 8:db79727cb2f7 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
279 |/ parent: 6:0c6523cb1d0f |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
280 | user: test |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
281 | date: Thu Jan 01 00:00:00 1970 +0000 |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
282 | obsolete: rewritten using amend as 10:082be6c94150 |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
283 | summary: modified A |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
284 | |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
285 | o changeset: 7:d2c78c3a3e01 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
286 | | parent: 5:876a60d024de |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
287 | | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
288 | | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
289 | | summary: modified 3 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
290 | | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
291 o | changeset: 6:0c6523cb1d0f |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
292 |/ user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
293 | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
294 | summary: modified 2 |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
295 | |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
296 o changeset: 5:876a60d024de |
44719
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
297 | parent: 0:5cbade24e0fa |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
298 | user: test |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
299 | date: Thu Jan 01 00:00:00 1970 +0000 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
300 | summary: modified 1 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
301 | |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
302 o changeset: 0:5cbade24e0fa |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
303 user: test |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
304 date: Thu Jan 01 00:00:00 1970 +0000 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
305 summary: added |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
306 |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
307 $ hg phabsend -r 5:: |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
308 abort: obsolete commits cannot be posted for review |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
309 [255] |
c482e2fe444c
phabricator: prevent posting obsolete commits
Matt Harbison <matt_harbison@yahoo.com>
parents:
44718
diff
changeset
|
310 |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
311 Don't restack existing orphans |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
312 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
313 $ hg phabsend -r 5::tip --test-vcr "$VCR/phabsend-no-restack-orphan.json" |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
314 876a60d024de mapped to old nodes ['876a60d024de'] |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
315 0c6523cb1d0f mapped to old nodes ['0c6523cb1d0f'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
316 D8434 - updated - 5:876a60d024de "modified 1" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
317 D8433 - updated - 6:0c6523cb1d0f "modified 2" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
318 D8435 - created - 10:082be6c94150 tip "modified A" |
44720
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
319 new commits: ['b5913193c805'] |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
320 not restabilizing unchanged d2c78c3a3e01 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
321 $ hg log -G |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
322 @ changeset: 11:b5913193c805 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
323 | tag: tip |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
324 | parent: 6:0c6523cb1d0f |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
325 | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
326 | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
327 | summary: modified A |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
328 | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
329 | * changeset: 9:a67643f48146 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
330 | | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
331 | | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
332 | | instability: orphan |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
333 | | summary: modified B |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
334 | | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
335 | x changeset: 8:db79727cb2f7 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
336 |/ parent: 6:0c6523cb1d0f |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
337 | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
338 | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
339 | obsolete: rewritten using amend, phabsend as 11:b5913193c805 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
340 | summary: modified A |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
341 | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
342 | o changeset: 7:d2c78c3a3e01 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
343 | | parent: 5:876a60d024de |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
344 | | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
345 | | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
346 | | summary: modified 3 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
347 | | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
348 o | changeset: 6:0c6523cb1d0f |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
349 |/ user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
350 | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
351 | summary: modified 2 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
352 | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
353 o changeset: 5:876a60d024de |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
354 | parent: 0:5cbade24e0fa |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
355 | user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
356 | date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
357 | summary: modified 1 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
358 | |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
359 o changeset: 0:5cbade24e0fa |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
360 user: test |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
361 date: Thu Jan 01 00:00:00 1970 +0000 |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
362 summary: added |
601ce5392cb0
phabricator: restack any new orphans created by phabsend (issue6045)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44719
diff
changeset
|
363 |
44718
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
364 $ cd .. |
0680b8a1992a
phabricator: avoid creating unstable children within the review stack
Matt Harbison <matt_harbison@yahoo.com>
parents:
44717
diff
changeset
|
365 |
44169
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
366 Phabesending a new binary, a modified binary, and a removed binary |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
367 |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
368 >>> open('bin', 'wb').write(b'\0a') and None |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
369 $ hg ci -Am 'add binary' |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
370 adding bin |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
371 >>> open('bin', 'wb').write(b'\0b') and None |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
372 $ hg ci -m 'modify binary' |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
373 $ hg rm bin |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
374 $ hg ci -m 'remove binary' |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
375 $ hg phabsend -r .~2:: --test-vcr "$VCR/phabsend-binary.json" |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
376 uploading bin@aa24a81f55de |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
377 D8007 - created - 5:aa24a81f55de "add binary" |
44169
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
378 uploading bin@d8d62a881b54 |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
379 D8008 - created - 6:d8d62a881b54 "modify binary" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
380 D8009 - created - 7:af55645b2e29 tip "remove binary" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
381 new commits: ['b8139fbb4a57'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
382 new commits: ['c88ce4c2d2ad'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
383 new commits: ['75dbbc901145'] |
44169
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
384 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/aa24a81f55de-a3a0cf24-phabsend.hg |
5d85e9ddc7b9
phabricator: fix a crash when submitting binaries (issue6260)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44128
diff
changeset
|
385 |
44329
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
386 Phabsend a renamed binary and a copied binary, with and without content changes |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
387 to src and dest |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
388 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
389 >>> open('bin2', 'wb').write(b'\0c') and None |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
390 $ hg ci -Am 'add another binary' |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
391 adding bin2 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
392 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
393 TODO: "bin2" can't be viewed in this commit (left or right side), and the URL |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
394 looks much different than when viewing "bin2_moved". No idea if this is a phab |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
395 bug, or phabsend bug. The patch (as printed by phabread) look reasonable |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
396 though. |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
397 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
398 $ hg mv bin2 bin2_moved |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
399 $ hg ci -m "moved binary" |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
400 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
401 Note: "bin2_moved" is also not viewable in phabricator with this review |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
402 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
403 $ hg cp bin2_moved bin2_copied |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
404 $ hg ci -m "copied binary" |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
405 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
406 Note: "bin2_moved_again" is marked binary in phabricator, and both sides of it |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
407 are viewable in their proper state. "bin2_copied" is not viewable, and not |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
408 listed as binary in phabricator. |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
409 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
410 >>> open('bin2_copied', 'wb').write(b'\0move+mod') and None |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
411 $ hg mv bin2_copied bin2_moved_again |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
412 $ hg ci -m "move+mod copied binary" |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
413 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
414 Note: "bin2_moved" and "bin2_moved_copy" are both marked binary, and both |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
415 viewable on each side. |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
416 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
417 >>> open('bin2_moved', 'wb').write(b'\0precopy mod') and None |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
418 $ hg cp bin2_moved bin2_moved_copied |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
419 >>> open('bin2_moved', 'wb').write(b'\0copy src+mod') and None |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
420 $ hg ci -m "copy+mod moved binary" |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
421 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
422 $ hg phabsend -r .~4:: --test-vcr "$VCR/phabsend-binary-renames.json" |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
423 uploading bin2@f42f9195e00c |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
424 D8128 - created - 8:f42f9195e00c "add another binary" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
425 D8129 - created - 9:834ab31d80ae "moved binary" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
426 D8130 - created - 10:494b750e5194 "copied binary" |
44329
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
427 uploading bin2_moved_again@25f766b50cc2 |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
428 D8131 - created - 11:25f766b50cc2 "move+mod copied binary" |
44329
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
429 uploading bin2_moved_copied@1b87b363a5e4 |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
430 uploading bin2_moved@1b87b363a5e4 |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
431 D8132 - created - 12:1b87b363a5e4 tip "copy+mod moved binary" |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
432 new commits: ['90437c20312a'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
433 new commits: ['f391f4da4c61'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
434 new commits: ['da86a9f3268c'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
435 new commits: ['003ffc16ba66'] |
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
436 new commits: ['13bd750c36fa'] |
44329
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
437 saved backup bundle to $TESTTMP/repo/.hg/strip-backup/f42f9195e00c-e82a0769-phabsend.hg |
d1177d39012e
phabricator: fix a phabsend crash when processing a renamed binary
Matt Harbison <matt_harbison@yahoo.com>
parents:
44169
diff
changeset
|
438 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
439 Phabreading a DREV with a local:commits time as a string: |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
440 $ hg phabread --test-vcr "$VCR/phabread-str-time.json" D1285 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
441 # HG changeset patch |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
442 # User Pulkit Goyal <7895pulkit@gmail.com> |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
443 # Date 1509404054 -19800 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
444 # Node ID 44fc1c1f1774a76423b9c732af6938435099bcc5 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
445 # Parent 8feef8ef8389a3b544e0a74624f1efc3a8d85d35 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
446 repoview: add a new attribute _visibilityexceptions and related API |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
447 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
448 Currently we don't have a defined way in core to make some hidden revisions |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
449 visible in filtered repo. Extensions to achieve the purpose of unhiding some |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
450 hidden commits, wrap repoview.pinnedrevs() function. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
451 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
452 To make the above task simple and have well defined API, this patch adds a new |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
453 attribute '_visibilityexceptions' to repoview class which will contains |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
454 the hidden revs which should be exception. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
455 This will allow to set different exceptions for different repoview objects |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
456 backed by the same unfiltered repo. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
457 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
458 This patch also adds API to add revs to the attribute set and get them. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
459 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
460 Thanks to Jun for suggesting the use of repoview class instead of localrepo. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
461 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
462 Differential Revision: https://phab.mercurial-scm.org/D1285 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
463 diff --git a/mercurial/repoview.py b/mercurial/repoview.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
464 --- a/mercurial/repoview.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
465 +++ b/mercurial/repoview.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
466 @@ * @@ (glob) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
467 subclasses of `localrepo`. Eg: `bundlerepo` or `statichttprepo`. |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
468 """ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
469 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
470 + # hidden revs which should be visible |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
471 + _visibilityexceptions = set() |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
472 + |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
473 def __init__(self, repo, filtername): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
474 object.__setattr__(self, r'_unfilteredrepo', repo) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
475 object.__setattr__(self, r'filtername', filtername) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
476 @@ -231,6 +234,14 @@ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
477 return self |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
478 return self.unfiltered().filtered(name) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
479 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
480 + def addvisibilityexceptions(self, revs): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
481 + """adds hidden revs which should be visible to set of exceptions""" |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
482 + self._visibilityexceptions.update(revs) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
483 + |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
484 + def getvisibilityexceptions(self): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
485 + """returns the set of hidden revs which should be visible""" |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
486 + return self._visibilityexceptions |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
487 + |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
488 # everything access are forwarded to the proxied repo |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
489 def __getattr__(self, attr): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
490 return getattr(self._unfilteredrepo, attr) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
491 diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
492 --- a/mercurial/localrepo.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
493 +++ b/mercurial/localrepo.py |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
494 @@ -570,6 +570,14 @@ |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
495 def close(self): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
496 self._writecaches() |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
497 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
498 + def addvisibilityexceptions(self, exceptions): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
499 + # should be called on a filtered repository |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
500 + pass |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
501 + |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
502 + def getvisibilityexceptions(self): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
503 + # should be called on a filtered repository |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
504 + return set() |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
505 + |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
506 def _loadextensions(self): |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
507 extensions.loadall(self.ui) |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
508 |
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
509 |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
510 A bad .arcconfig doesn't error out |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
511 $ echo 'garbage' > .arcconfig |
47071
3e381eb557f3
config: add --source option to include source of value
Martin von Zweigbergk <martinvonz@google.com>
parents:
46865
diff
changeset
|
512 $ hg config phabricator --source |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
513 invalid JSON in $TESTTMP/repo/.arcconfig |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
514 */.hgrc:*: phabricator.debug=True (glob) |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
515 $TESTTMP/repo/.hg/hgrc:*: phabricator.url=https://phab.mercurial-scm.org/ (glob) |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
516 $TESTTMP/repo/.hg/hgrc:*: phabricator.callsign=HG (glob) |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
517 |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
518 The .arcconfig content overrides global config |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
519 $ cat >> $HGRCPATH << EOF |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
520 > [phabricator] |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
521 > url = global |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
522 > callsign = global |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
523 > EOF |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
524 $ cp $TESTDIR/../.arcconfig . |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
525 $ mv .hg/hgrc .hg/hgrc.bak |
47071
3e381eb557f3
config: add --source option to include source of value
Martin von Zweigbergk <martinvonz@google.com>
parents:
46865
diff
changeset
|
526 $ hg config phabricator --source |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
527 */.hgrc:*: phabricator.debug=True (glob) |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
528 $TESTTMP/repo/.arcconfig: phabricator.callsign=HG |
44128
ff396501e841
phabricator: use .arcconfig for `phabricator.url` if not set locally
Matt Harbison <matt_harbison@yahoo.com>
parents:
44127
diff
changeset
|
529 $TESTTMP/repo/.arcconfig: phabricator.url=https://phab.mercurial-scm.org/ |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
530 |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
531 But it doesn't override local config |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
532 $ cat >> .hg/hgrc << EOF |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
533 > [phabricator] |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
534 > url = local |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
535 > callsign = local |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
536 > EOF |
47071
3e381eb557f3
config: add --source option to include source of value
Martin von Zweigbergk <martinvonz@google.com>
parents:
46865
diff
changeset
|
537 $ hg config phabricator --source |
44715
38f7b2f02f6d
phabricator: add debug logging to show previous node values in `phabsend`
Matt Harbison <matt_harbison@yahoo.com>
parents:
44579
diff
changeset
|
538 */.hgrc:*: phabricator.debug=True (glob) |
44127
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
539 $TESTTMP/repo/.hg/hgrc:*: phabricator.url=local (glob) |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
540 $TESTTMP/repo/.hg/hgrc:*: phabricator.callsign=local (glob) |
59b3fe1e2021
phabricator: use .arcconfig for the callsign if not set locally (issue6243)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44125
diff
changeset
|
541 $ mv .hg/hgrc.bak .hg/hgrc |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
542 |
44550
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
543 Phabimport works with a stack |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
544 |
44125
a5e3f38407cb
tests: restore phabricator tests and regenerate the recordings
Matt Harbison <matt_harbison@yahoo.com>
parents:
43260
diff
changeset
|
545 $ cd .. |
44550
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
546 $ hg clone repo repo2 -qr 1 |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
547 $ cp repo/.hg/hgrc repo2/.hg/ |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
548 $ cd repo2 |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
549 $ hg phabimport --stack 'D7918' --test-vcr "$VCR/phabimport-stack.json" |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
550 applying patch from D7917 |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
551 applying patch from D7918 |
44576
2ec6160449aa
tests: avoid logging a commit with a Unicode character in test-phabricator.t
Matt Harbison <matt_harbison@yahoo.com>
parents:
44551
diff
changeset
|
552 $ hg log -r .: -G -Tcompact |
44550
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
553 o 3[tip] aaef04066140 1970-01-01 00:00 +0000 test |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
554 | create draft change for phabricator testing |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
555 | |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
556 o 2 8de3712202d1 1970-01-01 00:00 +0000 test |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
557 | create public change for phabricator testing |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
558 | |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
559 @ 1 a692622e6937 1970-01-01 00:00 +0000 test |
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
560 | create beta for phabricator test |
44576
2ec6160449aa
tests: avoid logging a commit with a Unicode character in test-phabricator.t
Matt Harbison <matt_harbison@yahoo.com>
parents:
44551
diff
changeset
|
561 ~ |
44551
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
562 Phabimport can create secret commits |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
563 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
564 $ hg rollback --config ui.rollback=True |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
565 repository tip rolled back to revision 1 (undo phabimport) |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
566 $ hg phabimport --stack 'D7918' --test-vcr "$VCR/phabimport-stack.json" \ |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
567 > --config phabimport.secret=True |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
568 applying patch from D7917 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
569 applying patch from D7918 |
44576
2ec6160449aa
tests: avoid logging a commit with a Unicode character in test-phabricator.t
Matt Harbison <matt_harbison@yahoo.com>
parents:
44551
diff
changeset
|
570 $ hg log -r 'reverse(.:)' -T phases |
44551
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
571 changeset: 3:aaef04066140 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
572 tag: tip |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
573 phase: secret |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
574 user: test |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
575 date: Thu Jan 01 00:00:00 1970 +0000 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
576 summary: create draft change for phabricator testing |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
577 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
578 changeset: 2:8de3712202d1 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
579 phase: secret |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
580 user: test |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
581 date: Thu Jan 01 00:00:00 1970 +0000 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
582 summary: create public change for phabricator testing |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
583 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
584 changeset: 1:a692622e6937 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
585 phase: public |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
586 user: test |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
587 date: Thu Jan 01 00:00:00 1970 +0000 |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
588 summary: create beta for phabricator test |
9bae1d1a0f4c
phabricator: add a config knob to import in the secret phase
Matt Harbison <matt_harbison@yahoo.com>
parents:
44550
diff
changeset
|
589 |
46037
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
590 phabupdate can convert from local revisions |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
591 |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
592 $ hg phabupdate --reclaim D7917 -r '.: and not public()' |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
593 abort: cannot specify both DREVSPEC and --rev |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
594 [10] |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
595 |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
596 $ hg phabupdate --reclaim -r '.: and not public()' --test-vcr "$VCR/phabupdate-revs.json" |
9624bf057c2a
phabricator: allow local revisions to be specified with `phabupdate`
Matt Harbison <matt_harbison@yahoo.com>
parents:
45772
diff
changeset
|
597 |
44579
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
598 Phabimport accepts multiple DREVSPECs |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
599 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
600 $ hg rollback --config ui.rollback=True |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
601 repository tip rolled back to revision 1 (undo phabimport) |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
602 $ hg phabimport --no-stack D7917 D7918 --test-vcr "$VCR/phabimport-multi-drev.json" |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
603 applying patch from D7917 |
a7f8c657a3f0
phabricator: allow multiple DREVSPEC args to phabread|phabimport|phabupdate
Matt Harbison <matt_harbison@yahoo.com>
parents:
44576
diff
changeset
|
604 applying patch from D7918 |
44550
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
605 |
44774
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
606 Phabsend requires a linear range of commits |
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
607 |
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
608 $ hg phabsend -r 0+2+3 |
45152
b3b0cd8b9366
phabricator: clarify the multiple heads/roots error message for phabsend
Matt Harbison <matt_harbison@yahoo.com>
parents:
45135
diff
changeset
|
609 abort: cannot phabsend multiple head revisions: c44b38f24a45 aaef04066140 |
44774
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
610 (the revisions must form a linear chain) |
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
611 [255] |
c1c922391314
phabricator: ensure that `phabsend` is given a contiguous, linear commit range
Matt Harbison <matt_harbison@yahoo.com>
parents:
44720
diff
changeset
|
612 |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
613 Validate arguments with --fold |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
614 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
615 $ hg phabsend --fold -r 1 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
616 abort: cannot fold a single revision |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
617 [255] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
618 $ hg phabsend --fold --no-amend -r 1:: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
619 abort: cannot fold with --no-amend |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
620 [255] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
621 $ hg phabsend --fold -r 1:: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
622 abort: cannot fold revisions with different DREV values |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
623 [255] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
624 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
625 Setup a series of commits to be folded, and include the Test Plan field multiple |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
626 times to test the concatenation logic. No Test Plan field in the last one to |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
627 ensure missing fields are skipped. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
628 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
629 $ hg init ../folded |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
630 $ cd ../folded |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
631 $ cat >> .hg/hgrc <<EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
632 > [phabricator] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
633 > url = https://phab.mercurial-scm.org/ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
634 > callsign = HG |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
635 > EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
636 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
637 $ echo 'added' > file.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
638 $ hg ci -Aqm 'added file' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
639 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
640 $ cat > log.txt <<EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
641 > one: first commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
642 > |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
643 > This file was modified with 'mod1' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
644 > |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
645 > Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
646 > LOL! What testing?! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
647 > EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
648 $ echo mod1 > file.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
649 $ hg ci -l log.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
650 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
651 $ cat > log.txt <<EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
652 > two: second commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
653 > |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
654 > This file was modified with 'mod2' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
655 > |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
656 > Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
657 > Haha! yeah, right. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
658 > |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
659 > EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
660 $ echo mod2 > file.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
661 $ hg ci -l log.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
662 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
663 $ echo mod3 > file.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
664 $ hg ci -m '3: a commit with no detailed message' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
665 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
666 The folding of immutable commits works... |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
667 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
668 $ hg phase -r tip --public |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
669 $ hg phabsend --fold -r 1:: --test-vcr "$VCR/phabsend-fold-immutable.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
670 D8386 - created - 1:a959a3f69d8d "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
671 D8386 - created - 2:24a4438154ba "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
672 D8386 - created - 3:d235829e802c tip "3: a commit with no detailed message" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
673 warning: not updating public commit 1:a959a3f69d8d |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
674 warning: not updating public commit 2:24a4438154ba |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
675 warning: not updating public commit 3:d235829e802c |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
676 no newnodes to update |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
677 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
678 $ hg phase -r 0 --draft --force |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
679 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
680 ... as does the initial mutable fold... |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
681 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
682 $ echo y | hg phabsend --fold --confirm -r 1:: \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
683 > --test-vcr "$VCR/phabsend-fold-initial.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
684 NEW - 1:a959a3f69d8d "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
685 NEW - 2:24a4438154ba "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
686 NEW - 3:d235829e802c tip "3: a commit with no detailed message" |
44945
7a0a1be721a3
phabricator: make it clear what happen when no response
Sushil khanchi <sushilkhanchi97@gmail.com>
parents:
44775
diff
changeset
|
687 Send the above changes to https://phab.mercurial-scm.org/ (Y/n)? y |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
688 D8387 - created - 1:a959a3f69d8d "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
689 D8387 - created - 2:24a4438154ba "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
690 D8387 - created - 3:d235829e802c tip "3: a commit with no detailed message" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
691 updating local commit list for D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
692 new commits: ['602c4e738243', '832553266fe8', '921f8265efbd'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
693 saved backup bundle to $TESTTMP/folded/.hg/strip-backup/a959a3f69d8d-a4a24136-phabsend.hg |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
694 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
695 ... and doesn't mangle the local commits. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
696 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
697 $ hg log -T '{rev}:{node|short}\n{indent(desc, " ")}\n' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
698 3:921f8265efbd |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
699 3: a commit with no detailed message |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
700 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
701 Differential Revision: https://phab.mercurial-scm.org/D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
702 2:832553266fe8 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
703 two: second commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
704 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
705 This file was modified with 'mod2' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
706 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
707 Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
708 Haha! yeah, right. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
709 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
710 Differential Revision: https://phab.mercurial-scm.org/D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
711 1:602c4e738243 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
712 one: first commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
713 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
714 This file was modified with 'mod1' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
715 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
716 Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
717 LOL! What testing?! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
718 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
719 Differential Revision: https://phab.mercurial-scm.org/D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
720 0:98d480e0d494 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
721 added file |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
722 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
723 Setup some obsmarkers by adding a file to the middle commit. This stress tests |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
724 getoldnodedrevmap() in later phabsends. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
725 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
726 $ hg up '.^' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
727 1 files updated, 0 files merged, 0 files removed, 0 files unresolved |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
728 $ echo 'modified' > file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
729 $ hg add file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
730 $ hg amend --config experimental.evolution=all --config extensions.amend= |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
731 1 new orphan changesets |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
732 $ hg up 3 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
733 obsolete feature not enabled but 1 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
734 1 files updated, 0 files merged, 1 files removed, 0 files unresolved |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
735 $ hg rebase --config experimental.evolution=all --config extensions.rebase= |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
736 note: not rebasing 2:832553266fe8 "two: second commit to review", already in destination as 4:0124e5474c88 tip "two: second commit to review" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
737 rebasing 3:921f8265efbd "3: a commit with no detailed message" |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
738 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
739 When commits have changed locally, the local commit list on Phabricator is |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
740 updated. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
741 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
742 $ echo y | hg phabsend --fold --confirm -r 1:: \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
743 > --test-vcr "$VCR/phabsend-fold-updated.json" |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
744 obsolete feature not enabled but 2 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
745 602c4e738243 mapped to old nodes ['602c4e738243'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
746 0124e5474c88 mapped to old nodes ['832553266fe8'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
747 e4edb1fe3565 mapped to old nodes ['921f8265efbd'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
748 D8387 - 1:602c4e738243 "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
749 D8387 - 4:0124e5474c88 "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
750 D8387 - 5:e4edb1fe3565 tip "3: a commit with no detailed message" |
44945
7a0a1be721a3
phabricator: make it clear what happen when no response
Sushil khanchi <sushilkhanchi97@gmail.com>
parents:
44775
diff
changeset
|
751 Send the above changes to https://phab.mercurial-scm.org/ (Y/n)? y |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
752 D8387 - updated - 1:602c4e738243 "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
753 D8387 - updated - 4:0124e5474c88 "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
754 D8387 - updated - 5:e4edb1fe3565 tip "3: a commit with no detailed message" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
755 obsolete feature not enabled but 2 markers found! (?) |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
756 updating local commit list for D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
757 new commits: ['602c4e738243', '0124e5474c88', 'e4edb1fe3565'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
758 $ hg log -Tcompact |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
759 obsolete feature not enabled but 2 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
760 5[tip] e4edb1fe3565 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
761 3: a commit with no detailed message |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
762 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
763 4:1 0124e5474c88 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
764 two: second commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
765 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
766 1 602c4e738243 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
767 one: first commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
768 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
769 0 98d480e0d494 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
770 added file |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
771 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
772 When nothing has changed locally since the last phabsend, the commit list isn't |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
773 updated, and nothing is changed locally afterward. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
774 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
775 $ hg phabsend --fold -r 1:: --test-vcr "$VCR/phabsend-fold-no-changes.json" |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
776 obsolete feature not enabled but 2 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
777 602c4e738243 mapped to old nodes ['602c4e738243'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
778 0124e5474c88 mapped to old nodes ['0124e5474c88'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
779 e4edb1fe3565 mapped to old nodes ['e4edb1fe3565'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
780 D8387 - updated - 1:602c4e738243 "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
781 D8387 - updated - 4:0124e5474c88 "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
782 D8387 - updated - 5:e4edb1fe3565 tip "3: a commit with no detailed message" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
783 obsolete feature not enabled but 2 markers found! (?) |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
784 local commit list for D8387 is already up-to-date |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
785 $ hg log -Tcompact |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
786 obsolete feature not enabled but 2 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
787 5[tip] e4edb1fe3565 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
788 3: a commit with no detailed message |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
789 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
790 4:1 0124e5474c88 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
791 two: second commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
792 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
793 1 602c4e738243 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
794 one: first commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
795 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
796 0 98d480e0d494 1970-01-01 00:00 +0000 test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
797 added file |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
798 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
799 Fold will accept new revisions at the end... |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
800 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
801 $ echo 'another mod' > file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
802 $ hg ci -m 'four: extend the fold range' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
803 obsolete feature not enabled but 2 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
804 $ hg phabsend --fold -r 1:: --test-vcr "$VCR/phabsend-fold-extend-end.json" \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
805 > --config experimental.evolution=all |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
806 602c4e738243 mapped to old nodes ['602c4e738243'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
807 0124e5474c88 mapped to old nodes ['0124e5474c88'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
808 e4edb1fe3565 mapped to old nodes ['e4edb1fe3565'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
809 D8387 - updated - 1:602c4e738243 "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
810 D8387 - updated - 4:0124e5474c88 "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
811 D8387 - updated - 5:e4edb1fe3565 "3: a commit with no detailed message" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
812 D8387 - created - 6:94aaae213b23 tip "four: extend the fold range" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
813 updating local commit list for D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
814 new commits: ['602c4e738243', '0124e5474c88', 'e4edb1fe3565', '51a04fea8707'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
815 $ hg log -r . -T '{desc}\n' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
816 four: extend the fold range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
817 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
818 Differential Revision: https://phab.mercurial-scm.org/D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
819 $ hg log -T'{rev} {if(phabreview, "{phabreview.url} {phabreview.id}")}\n' -r 1:: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
820 obsolete feature not enabled but 3 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
821 1 https://phab.mercurial-scm.org/D8387 D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
822 4 https://phab.mercurial-scm.org/D8387 D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
823 5 https://phab.mercurial-scm.org/D8387 D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
824 7 https://phab.mercurial-scm.org/D8387 D8387 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
825 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
826 ... and also accepts new revisions at the beginning of the range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
827 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
828 It's a bit unfortunate that not having a Differential URL on the first commit |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
829 causes a new Differential Revision to be created, though it isn't *entirely* |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
830 unreasonable. At least this updates the subsequent commits. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
831 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
832 TODO: See if it can reuse the existing Differential. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
833 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
834 $ hg phabsend --fold -r 0:: --test-vcr "$VCR/phabsend-fold-extend-front.json" \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
835 > --config experimental.evolution=all |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
836 602c4e738243 mapped to old nodes ['602c4e738243'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
837 0124e5474c88 mapped to old nodes ['0124e5474c88'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
838 e4edb1fe3565 mapped to old nodes ['e4edb1fe3565'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
839 51a04fea8707 mapped to old nodes ['51a04fea8707'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
840 D8388 - created - 0:98d480e0d494 "added file" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
841 D8388 - updated - 1:602c4e738243 "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
842 D8388 - updated - 4:0124e5474c88 "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
843 D8388 - updated - 5:e4edb1fe3565 "3: a commit with no detailed message" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
844 D8388 - updated - 7:51a04fea8707 tip "four: extend the fold range" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
845 updating local commit list for D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
846 new commits: ['15e9b14b4b4c', '6320b7d714cf', '3ee132d41dbc', '30682b960804', 'ac7db67f0991'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
847 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
848 $ hg log -T '{rev}:{node|short}\n{indent(desc, " ")}\n' |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
849 obsolete feature not enabled but 8 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
850 12:ac7db67f0991 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
851 four: extend the fold range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
852 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
853 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
854 11:30682b960804 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
855 3: a commit with no detailed message |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
856 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
857 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
858 10:3ee132d41dbc |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
859 two: second commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
860 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
861 This file was modified with 'mod2' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
862 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
863 Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
864 Haha! yeah, right. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
865 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
866 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
867 9:6320b7d714cf |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
868 one: first commit to review |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
869 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
870 This file was modified with 'mod1' as its contents. |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
871 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
872 Test Plan: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
873 LOL! What testing?! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
874 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
875 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
876 8:15e9b14b4b4c |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
877 added file |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
878 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
879 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
880 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
881 Test phabsend --fold with an `hg split` at the end of the range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
882 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
883 $ echo foo > file3.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
884 $ hg add file3.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
885 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
886 $ hg log -r . -T '{desc}' > log.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
887 $ echo 'amended mod' > file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
888 $ hg ci --amend -l log.txt --config experimental.evolution=all |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
889 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
890 $ cat <<EOF | hg --config extensions.split= --config ui.interactive=True \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
891 > --config experimental.evolution=all split -r . |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
892 > n |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
893 > y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
894 > y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
895 > y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
896 > y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
897 > EOF |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
898 diff --git a/file2.txt b/file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
899 1 hunks, 1 lines changed |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
900 examine changes to 'file2.txt'? |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
901 (enter ? for help) [Ynesfdaq?] n |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
902 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
903 diff --git a/file3.txt b/file3.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
904 new file mode 100644 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
905 examine changes to 'file3.txt'? |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
906 (enter ? for help) [Ynesfdaq?] y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
907 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
908 @@ -0,0 +1,1 @@ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
909 +foo |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
910 record change 2/2 to 'file3.txt'? |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
911 (enter ? for help) [Ynesfdaq?] y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
912 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
913 created new head |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
914 diff --git a/file2.txt b/file2.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
915 1 hunks, 1 lines changed |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
916 examine changes to 'file2.txt'? |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
917 (enter ? for help) [Ynesfdaq?] y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
918 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
919 @@ -1,1 +1,1 @@ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
920 -modified |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
921 +amended mod |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
922 record this change to 'file2.txt'? |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
923 (enter ? for help) [Ynesfdaq?] y |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
924 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
925 $ hg phabsend --fold -r 8:: --test-vcr "$VCR/phabsend-fold-split-end.json" \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
926 > --config experimental.evolution=all |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
927 15e9b14b4b4c mapped to old nodes ['15e9b14b4b4c'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
928 6320b7d714cf mapped to old nodes ['6320b7d714cf'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
929 3ee132d41dbc mapped to old nodes ['3ee132d41dbc'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
930 30682b960804 mapped to old nodes ['30682b960804'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
931 6bc15dc99efd mapped to old nodes ['ac7db67f0991'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
932 b50946d5e490 mapped to old nodes ['ac7db67f0991'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
933 D8388 - updated - 8:15e9b14b4b4c "added file" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
934 D8388 - updated - 9:6320b7d714cf "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
935 D8388 - updated - 10:3ee132d41dbc "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
936 D8388 - updated - 11:30682b960804 "3: a commit with no detailed message" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
937 D8388 - updated - 14:6bc15dc99efd "four: extend the fold range" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
938 D8388 - updated - 15:b50946d5e490 tip "four: extend the fold range" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
939 updating local commit list for D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
940 new commits: ['15e9b14b4b4c', '6320b7d714cf', '3ee132d41dbc', '30682b960804', '6bc15dc99efd', 'b50946d5e490'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
941 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
942 Test phabsend --fold with an `hg fold` at the end of the range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
943 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
944 $ hg --config experimental.evolution=all --config extensions.rebase= \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
945 > rebase -r '.^' -r . -d '.^^' --collapse -l log.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
946 rebasing 14:6bc15dc99efd "four: extend the fold range" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
947 rebasing 15:b50946d5e490 tip "four: extend the fold range" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
948 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
949 $ hg phabsend --fold -r 8:: --test-vcr "$VCR/phabsend-fold-fold-end.json" \ |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
950 > --config experimental.evolution=all |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
951 15e9b14b4b4c mapped to old nodes ['15e9b14b4b4c'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
952 6320b7d714cf mapped to old nodes ['6320b7d714cf'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
953 3ee132d41dbc mapped to old nodes ['3ee132d41dbc'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
954 30682b960804 mapped to old nodes ['30682b960804'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
955 e919cdf3d4fe mapped to old nodes ['6bc15dc99efd', 'b50946d5e490'] |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
956 D8388 - updated - 8:15e9b14b4b4c "added file" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
957 D8388 - updated - 9:6320b7d714cf "one: first commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
958 D8388 - updated - 10:3ee132d41dbc "two: second commit to review" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
959 D8388 - updated - 11:30682b960804 "3: a commit with no detailed message" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
960 D8388 - updated - 16:e919cdf3d4fe tip "four: extend the fold range" |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
961 updating local commit list for D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
962 new commits: ['15e9b14b4b4c', '6320b7d714cf', '3ee132d41dbc', '30682b960804', 'e919cdf3d4fe'] |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
963 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
964 $ hg log -r tip -v |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
965 obsolete feature not enabled but 12 markers found! |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
966 changeset: 16:e919cdf3d4fe |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
967 tag: tip |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
968 parent: 11:30682b960804 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
969 user: test |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
970 date: Thu Jan 01 00:00:00 1970 +0000 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
971 files: file2.txt file3.txt |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
972 description: |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
973 four: extend the fold range |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
974 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
975 Differential Revision: https://phab.mercurial-scm.org/D8388 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
976 |
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
977 |
45426
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
978 Hashes in the messages are updated automatically as phabsend amends and restacks |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
979 them. This covers both commits that are posted and descendants that are |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
980 restacked. |
44717
3dc6a70779f2
phabricator: add an option to fold several commits into one review (issue6244)
Matt Harbison <matt_harbison@yahoo.com>
parents:
44716
diff
changeset
|
981 |
45426
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
982 $ cat >> .hg/hgrc << EOF |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
983 > [experimental] |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
984 > evolution = all |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
985 > EOF |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
986 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
987 $ echo content > file.txt |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
988 $ hg ci -m 'base review (generate test for phabsend)' |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
989 $ echo 'more content' > file.txt |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
990 $ hg ci -m '133c1c6c6449 is my parent (generate test for phabsend)' |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
991 $ echo 'even more content' > file.txt |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
992 $ hg ci -m 'c2874a398f7e is my parent (generate test for phabsend)' |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
993 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
994 $ hg phabsend -r 17::18 --test-vcr "$VCR/phabsend-hash-fixes.json" |
45772
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
995 D8945 - created - 17:133c1c6c6449 "base review (generate test for phabsend)" |
294aab57a5f3
phabsend: use default one-line summary
Martin von Zweigbergk <martinvonz@google.com>
parents:
45426
diff
changeset
|
996 D8946 - created - 18:c2874a398f7e "133c1c6c6449 is my parent (generate test for phabsend)" |
45426
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
997 new commits: ['f444f060f4d6'] |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
998 new commits: ['9c9290f945b1'] |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
999 restabilizing 1528c12fa2e4 as b28b20212bd4 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1000 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1001 $ hg log -l 3 -Tcompact |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1002 22[tip] b28b20212bd4 1970-01-01 00:00 +0000 test |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1003 9c9290f945b1 is my parent (generate test for phabsend) |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1004 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1005 21 9c9290f945b1 1970-01-01 00:00 +0000 test |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1006 f444f060f4d6 is my parent (generate test for phabsend) |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1007 |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1008 20:16 f444f060f4d6 1970-01-01 00:00 +0000 test |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1009 base review (generate test for phabsend) |
1a5d3e555c70
phabricator: update the hashes in commit messages as they get submitted
Matt Harbison <matt_harbison@yahoo.com>
parents:
45246
diff
changeset
|
1010 |
44550
bbb170f9396d
phabricator: add a `phabimport` command
Matt Harbison <matt_harbison@yahoo.com>
parents:
44436
diff
changeset
|
1011 $ cd .. |