Merge lp:~ace17/bzr/fixDiff into lp:bzr
Status: | Needs review |
---|---|
Proposed branch: | lp:~ace17/bzr/fixDiff |
Merge into: | lp:bzr |
Diff against target: |
31 lines (+10/-9) 1 file modified
bzrlib/diff.py (+10/-9) |
To merge this branch: | bzr merge lp:~ace17/bzr/fixDiff |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Richard Wilbur | 2014-06-11 | Needs Information on 2014-06-12 | |
Review via email:
|
Description of the change
Fixes a bug causing --diff-options being passed to the external GNU diff program when a custom diff tool has been specified.
ace@ANTEC bzr-test % bzr status -V
added:
File1
modified:
File2
ace@ANTEC bzr-test % bzr diff --using gvimdiff --diff-options "-f"
=== added file 'File1'
diff: conflicting output style options
diff: Try `diff --help' for more information.
bzr: ERROR: external diff failed with exit code 2; command: ['diff', '--label', u'File1\t1970-01-01 00:00:00 +0000', '/tmp/bzr-
Sebastien Alaiwan (ace17) wrote : | # |
Hi, thanks for your reply.
Here's how I did:
bzr branch --stacked lp:bzr
cd bzr
[edit the file(s) with your favourite editor]
bzr commit -m 'Fix external diff....'
bzr push lp:~ace17/bzr/fixDiff
I received a message from launchpad:
"Launchpad encountered an internal error during the following operation: generating the diff for a merge proposal. It was logged with id OOPS-8239c35b3d57eabeca064b77c3513d04. Sorry for the inconvenience."
I use "--stacked" because the whole bzr repository is very long to download. This might have cause the issue. I'm retrying to recreate the whole thing ('bzr reconfigure --unstacked' seems to have no effect)
Sebastien Alaiwan (ace17) wrote : | # |
OK, i've created a new branch (which seems to contain revisions this time!).
Richard, once you've read these two messages I'll delete this broken merge proposal. Let's continue the discussion here: https:/
Richard Wilbur (richard-wilbur) wrote : | # |
Sebastien, thanks for creating one that works. I'm sorry that the
"stacked" branch didn't work. I have never tried a "stacked" branch
so I hadn't previously encountered that problem. To be continued on
the new branch.
- 6599. By Sebastien Alaiwan on 2014-09-24
-
Merge
Unmerged revisions
- 6599. By Sebastien Alaiwan on 2014-09-24
-
Merge
- 6598. By Sebastien Alaiwan on 2014-06-11
-
Avoid calling 'external_diff' with options passed to --diff-options if --using is specified. Fixes the following from failing 'bzr add somefile.txt ; bzr diff --using gvimdiff --diff-options '-f -n -d'
Diff calculation failed
Calculating the branch diff failed. You can manually schedule an update if required.
I'd be happy to review this merge proposal but when I look at the web page it says:
Updating diff...
An updated diff will be available in a few minutes. Reload to see the changes.
I tried reloading the page ≥5 minutes later with identical results.
Next I checked the web page for the branch as I can generally look at the diff's for each revision and understand what changes are entailed. Surprisingly it said the branch is empty with no revisions!
I'm not sure what happened to cause this situation. My normal recipe for creating a branch is as follows:
bzr branch lp:bzr
cd bzr
[edit the file(s) with your favourite editor]
bzr commit -m 'Fix external diff....'
bzr push lp:~ace17/bzr/fixDiff
Then visit here: /code.launchpad .net/~ace17/ bzr/fixDiff/ +register- merge
https:/
Fill out the form and submit.