KeyError: 20

Bug #598784 reported by Saša Janiška
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar Fast Import
Fix Released
Medium
Miklos Vajna

Bug Description

Hi!

Running: x2d -f git be (be is http://gitorious.org/be/be) gives:

Traceback (most recent call last):
  File "/home/gour/.bazaar/plugins/fastimport/exporters/darcs/darcs-fast-import", line 318, in <module>
    h.handle()
  File "/home/gour/.bazaar/plugins/fastimport/exporters/darcs/darcs-fast-import", line 300, in handle
    self.handle_commit()
  File "/home/gour/.bazaar/plugins/fastimport/exporters/darcs/darcs-fast-import", line 208, in handle_commit
    sock.write(self.marks[idx])
KeyError: 20

Sincerely,
Gour

Related branches

Miklos Vajna (vmiklos)
Changed in bzr-fastimport:
status: New → Confirmed
Revision history for this message
Miklos Vajna (vmiklos) wrote :

http://vmiklos.hu/gitweb/?p=bzr-fastimport.git;a=patch;h=7cc1706ec96298b4808e299f2560fb14095034f5

This should fix the reported issue.

Now it fails later with:

Traceback (most recent call last):
  File "/home/vmiklos/bin/darcs-fast-import", line 317, in <module>
    h.handle()
  File "/home/vmiklos/bin/darcs-fast-import", line 299, in handle
    self.handle_commit()
  File "/home/vmiklos/bin/darcs-fast-import", line 205, in handle_commit
    sock = open(path, "w")
IOError: [Errno 21] Is a directory: 'interfaces/email/interactive/becommands'

I don't have a patch for that issue yet. (But it's on my TODO.)

Changed in bzr-fastimport:
assignee: nobody → Miklos Vajna (vmiklos)
Revision history for this message
Miklos Vajna (vmiklos) wrote :

OK, I now see, it's because a directory is replaced by a symlink. Now I can add a better error message, but what should I do with symlinks? Darcs does not support them. :/

Revision history for this message
Miklos Vajna (vmiklos) wrote :
Revision history for this message
Miklos Vajna (vmiklos) wrote :

I worked a bit more on this, now it imports here fine.

Please test the code from git://vmiklos.hu/bzr-fastimport, using x2d -f git --symhack <repo>.

If it looks fine at you as well, then I'll push it to a bzr branch and propose for merging.

Thanks.

Revision history for this message
Saša Janiška (gour) wrote :

Thanks a lot for the fix.

It works here!

Sincerely,
Gour

Miklos Vajna (vmiklos)
Changed in bzr-fastimport:
status: Confirmed → Fix Committed
Jelmer Vernooij (jelmer)
Changed in bzr-fastimport:
importance: Undecided → Medium
Jelmer Vernooij (jelmer)
Changed in bzr-fastimport:
status: Fix Committed → Fix Released
milestone: none → 0.10.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.