<table><tr><td style="">simgunz added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10797">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>You're not really pushing a "branch" with <tt style="background: #ebebeb; font-size: 13px;">arc</tt>, you can think of it more as a diff describing a range of commits.</p></blockquote>

<p>This is indeed what I believed.</p>

<p>My doubt is: when you want to review a "review" what do you do exactly (I never did it). You download a patch and apply it manually to your code (probably arc does this with some command)?<br />
If I overwrite some commits, as I did today, does it create any problem to the reviewer that already "pulled" from arc the review once? (in git this will create diverging branches between local and remote) Or does arc redownload the diff completely each time, so avoiding problems?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>IOW, you should be able to do pretty much want you want on your local branch without having to fear to break something in the upstream Git repo (unless you <tt style="background: #ebebeb; font-size: 13px;">git push</tt> or <tt style="background: #ebebeb; font-size: 13px;">arc land</tt>).</p></blockquote>

<p>This was not my fear, but I feared to mess up things to the reviewer that already "pulled" the review before.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Also, in general I'd expect you could just <tt style="background: #ebebeb; font-size: 13px;">arc diff</tt> over a commit range containing a local revert. I cannot say too much without knowing details about "complaining and didn't let me update the revision", but given your accidental new Diff it seems to me that you got in a situation where <tt style="background: #ebebeb; font-size: 13px;">arc</tt> got the commit range wrong, i.e. it could not find a commit containing the "Differential Revision" line, so it could not attach to that.</p></blockquote>

<p>It was complaining about the fact that the commit with the revert did not contain a valid "Differential Revision" ID</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">a commit containing  the "Differential Revision" line,</pre></div></blockquote>

<p>Can you explain this better? I've never written any "Differential Revision" line in my commit messages explicitly. Does arc produces new commits with this line?</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Use <tt style="background: #ebebeb; font-size: 13px;">arc which</tt> before <tt style="background: #ebebeb; font-size: 13px;">arc diff</tt> to get more information about what Arcanist will do.</p></blockquote>

<p>Thanks. That is something I was looking for. I should re-read the phabricator guide more often.</p>

<p>I'll also have a look at  <a href="https://phabricator.kde.org/T7116" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T7116</a> and maybe I can give some feedback from a point of view of a newcomer.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10797">https://phabricator.kde.org/D10797</a></div></div><br /><div><strong>To: </strong>simgunz, Okular, aacid<br /><strong>Cc: </strong>rkflx, aacid, ngraham, michaelweghorn<br /></div>