<div dir="ltr">Thanks again, Austin, for all the help in this exchange and on #ghc, leading to <a href="https://phabricator.haskell.org/D2088">https://phabricator.haskell.org/D2088</a>. Looking forward to more GHC involvement.  - Conal</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 5:21 PM, Austin Seipp <span dir="ltr"><<a href="mailto:austin@well-typed.com" target="_blank">austin@well-typed.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Apr 6, 2016 at 7:11 PM, Conal Elliott <<a href="mailto:conal@conal.net">conal@conal.net</a>> wrote:<br>
> I'm trying to get through my first GHC patch submission, using git and<br>
> Phabricator, following directions at<br>
> <a href="https://ghc.haskell.org/trac/ghc/wiki/Phabricator" rel="noreferrer" target="_blank">https://ghc.haskell.org/trac/ghc/wiki/Phabricator</a>. "git status" shows two<br>
> unstaged changes: "deleted:    test   spaces" and "modified:<br>
> ../libraries/bytestring (modified content)" (in a submodule). I don't know<br>
> where either change came from. When I use "arc diff" to submit my intended<br>
> change to the compiler, I get asked about these to uncommitted changes. I<br>
> say to ignore the submodule change and do not amend my commit with the "test<br>
> spaces" deletion, I get "Usage Exception: You can not continue with<br>
> uncommitted changes. Commit or discard them before proceeding." I don't know<br>
> how to discard (nor whether safe), and I don't think I want to commit.<br>
<br>
</span>I'm not sure about the 'test spaces' change. That should be added to<br>
.gitignore I think. I'll try to see if I can get that done for you.<br>
<br>
It should be fine to discard the changes; the spaces thing is an<br>
artifact of the build system testing a binary distribution (I assume<br>
you did ./validate ?)<br>
<br>
The bytestring change might be because there were some left over files<br>
from the build system lingering, that might not be in .gitignore. Just<br>
try running 'git status' in libraries/bytestring and seeing what it<br>
shows.<br>
<span class=""><br>
> Note: the directions say to use "arc diff~" (rather than "arc diff"), to<br>
> which arc replied "(Assuming 'diff~' is the British spelling of 'diff'.)".<br>
<br>
</span>Ugh, sorry. That's a clerical error. I must have typo'd that...<br>
<br>
Assuming you want to submit the latest patch, on your current branch, you want:<br>
<br>
$ arc diff HEAD~<br>
<br>
The first argument to 'diff' is essentially the commit to start as the<br>
'base' commit. You then get a diff generated, from the 'base' commit<br>
against the head commit of your working copy. In other words, when<br>
you say "arc diff XXX", that's basically a way of saying "Send the<br>
diff between XXX..HEAD".<br>
<br>
In this case, you can see that if you say "git log HEAD~..HEAD", which<br>
would only contain one commit (presumably, what you want).<br>
<br>
Arcanist can always tell you exactly what it will do, in a detailed<br>
way, before doing it. Instead of running 'arc diff', run 'arc which'<br>
<br>
$ arc which HEAD~<br>
<br>
That will tell you exactly what arcanist will do, in a detailed<br>
manner. E.g. it will tell you if it will create a new diff, or update<br>
an old one, and exactly the commits it will use to create or update<br>
that diff. You can use further flags/arguments to 'arc diff' or 'arc<br>
which' to override the behavior as you want.<br>
<span class=""><br>
> Thanks for any help!<br>
><br>
> -- Conal<br>
><br>
</span>> _______________________________________________<br>
> ghc-devs mailing list<br>
> <a href="mailto:ghc-devs@haskell.org">ghc-devs@haskell.org</a><br>
> <a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" rel="noreferrer" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Regards,<br>
<br>
Austin Seipp, Haskell Consultant<br>
Well-Typed LLP, <a href="http://www.well-typed.com/" rel="noreferrer" target="_blank">http://www.well-typed.com/</a><br>
</font></span></blockquote></div><br></div>