all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#5302: Diff-mode fails on Bzr diffs
@ 2010-01-04 17:29 Juri Linkov
  2017-01-31  8:05 ` bug#5302: Fixed Dima Kogan
  2017-01-31  9:02 ` bug#5302: Diff-mode fails on Bzr diffs Tino Calancha
  0 siblings, 2 replies; 6+ messages in thread
From: Juri Linkov @ 2010-01-04 17:29 UTC (permalink / raw)
  To: bug-gnu-emacs

With the Bzr output in Diff-mode, `C-c C-c' (`diff-goto-source') signals
the following error:

Debugger entered--Lisp error: (error "End of hunk ambiguously marked")
  signal(error ("End of hunk ambiguously marked"))
  error("End of hunk ambiguously marked")
  diff-sanity-check-hunk()
  diff-find-source-location(nil t)
  diff-goto-source(nil 3)
  call-interactively(diff-goto-source nil nil)

Steps to reproduce:

1. emacs -Q

2. In a Bzr controlled dir, run:

   M-! bzr log -p -r-2.. RET

3. Switch to the output buffer, and type:

   M-x diff-mode RET

4. Go to the last hunk of the first revision and type `C-c C-c'.

The problem is that Brz separates revisions with a line of dashes like

------------------------------------------------------------

that Diff-mode doesn't understand.

-- 
Juri Linkov
http://www.jurta.org/emacs/







^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#5302: Fixed
  2010-01-04 17:29 bug#5302: Diff-mode fails on Bzr diffs Juri Linkov
@ 2017-01-31  8:05 ` Dima Kogan
  2017-01-31  9:02 ` bug#5302: Diff-mode fails on Bzr diffs Tino Calancha
  1 sibling, 0 replies; 6+ messages in thread
From: Dima Kogan @ 2017-01-31  8:05 UTC (permalink / raw)
  To: 9597-done, 5302-done

I pushed the fix and a test in 47a1ebb8ab67c97904bf3211eb44ce2d7d40949d





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#5302: Diff-mode fails on Bzr diffs
  2010-01-04 17:29 bug#5302: Diff-mode fails on Bzr diffs Juri Linkov
  2017-01-31  8:05 ` bug#5302: Fixed Dima Kogan
@ 2017-01-31  9:02 ` Tino Calancha
  2017-01-31  9:35   ` bug#9597: " Tino Calancha
  1 sibling, 1 reply; 6+ messages in thread
From: Tino Calancha @ 2017-01-31  9:02 UTC (permalink / raw)
  To: Dima Kogan; +Cc: 5302, 9597, tino.calancha


Dima Kogan <dima@secretsauce.net> writes:

>I pushed the fix and a test in 47a1ebb8ab67c97904bf3211eb44ce2d7d40949d
Hi Dima,
i just pulled the master branch but i cannot find
such 47a1ebb commit.
$ grep diff-separator-re diff-mode.el
returns 1

Apparently the fix is not there.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#9597: bug#5302: Diff-mode fails on Bzr diffs
  2017-01-31  9:02 ` bug#5302: Diff-mode fails on Bzr diffs Tino Calancha
@ 2017-01-31  9:35   ` Tino Calancha
  2017-01-31 16:52     ` Dima Kogan
  0 siblings, 1 reply; 6+ messages in thread
From: Tino Calancha @ 2017-01-31  9:35 UTC (permalink / raw)
  To: Tino Calancha; +Cc: 5302, 9597, Dima Kogan



On Tue, 31 Jan 2017, Tino Calancha wrote:

>
> Dima Kogan <dima@secretsauce.net> writes:
>
>> I pushed the fix and a test in 47a1ebb8ab67c97904bf3211eb44ce2d7d40949d
> Hi Dima,
> i just pulled the master branch but i cannot find
> such 47a1ebb commit.
> $ grep diff-separator-re diff-mode.el
> returns 1
>
> Apparently the fix is not there.
Also your latest commit f3c77d11af65f3b319b1784b4c3cf08c51aa7997
looks wrong: it changes 5 files with >100 lines changed, 
but the commit message is just one word: stash





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#5302: Diff-mode fails on Bzr diffs
  2017-01-31  9:35   ` bug#9597: " Tino Calancha
@ 2017-01-31 16:52     ` Dima Kogan
  2017-01-31 16:55       ` bug#9597: " Tino Calancha
  0 siblings, 1 reply; 6+ messages in thread
From: Dima Kogan @ 2017-01-31 16:52 UTC (permalink / raw)
  To: Tino Calancha; +Cc: 5302, 9597

On January 31, 2017 1:35:34 AM PST, Tino Calancha <tino.calancha@gmail.com> wrote:
>
>
>On Tue, 31 Jan 2017, Tino Calancha wrote:
>
>>
>> Dima Kogan <dima@secretsauce.net> writes:
>>
>>> I pushed the fix and a test in
>47a1ebb8ab67c97904bf3211eb44ce2d7d40949d
>> Hi Dima,
>> i just pulled the master branch but i cannot find
>> such 47a1ebb commit.
>> $ grep diff-separator-re diff-mode.el
>> returns 1
>>
>> Apparently the fix is not there.
>Also your latest commit f3c77d11af65f3b319b1784b4c3cf08c51aa7997
>looks wrong: it changes 5 files with >100 lines changed, 
>but the commit message is just one word: stash

I made a mistake and ended up pushing the wrong few commits. The revert and the correct commits are pushed now. The fix lives in 189df8262e54c4. The test lives in 3506ec2b11.






^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#9597: bug#5302: Diff-mode fails on Bzr diffs
  2017-01-31 16:52     ` Dima Kogan
@ 2017-01-31 16:55       ` Tino Calancha
  0 siblings, 0 replies; 6+ messages in thread
From: Tino Calancha @ 2017-01-31 16:55 UTC (permalink / raw)
  To: Dima Kogan; +Cc: 5302, 9597



On 02/01/2017 01:52 AM, Dima Kogan wrote:
> On January 31, 2017 1:35:34 AM PST, Tino Calancha <tino.calancha@gmail.com> wrote:
>>
>> On Tue, 31 Jan 2017, Tino Calancha wrote:
>>
>>> Dima Kogan <dima@secretsauce.net> writes:
>>>
>>>> I pushed the fix and a test in
>> 47a1ebb8ab67c97904bf3211eb44ce2d7d40949d
>>> Hi Dima,
>>> i just pulled the master branch but i cannot find
>>> such 47a1ebb commit.
>>> $ grep diff-separator-re diff-mode.el
>>> returns 1
>>>
>>> Apparently the fix is not there.
>> Also your latest commit f3c77d11af65f3b319b1784b4c3cf08c51aa7997
>> looks wrong: it changes 5 files with >100 lines changed,
>> but the commit message is just one word: stash
> I made a mistake and ended up pushing the wrong few commits. The revert and the correct commits are pushed now. The fix lives in 189df8262e54c4. The test lives in 3506ec2b11.
>
I see your new commits now.  Thank you very much for fixing this bugs.






^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-01-31 16:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-04 17:29 bug#5302: Diff-mode fails on Bzr diffs Juri Linkov
2017-01-31  8:05 ` bug#5302: Fixed Dima Kogan
2017-01-31  9:02 ` bug#5302: Diff-mode fails on Bzr diffs Tino Calancha
2017-01-31  9:35   ` bug#9597: " Tino Calancha
2017-01-31 16:52     ` Dima Kogan
2017-01-31 16:55       ` bug#9597: " Tino Calancha

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.