Re: [PATCH/RFC 2/2] git-p4: Add complex test case for branch import

On Wed, Aug 24, 2011 at 11:46 AM, Vitor Antunes [off-list ref] wrote:

1. When doing a "checkpoint" it would make sense to have some kind of
  feedback loop to know when fast-import completes it. In fact, I've
  just looked at fast-import man page and the "progress" command seems
  to do exactly that! Myabe we should crease a specific function for
  "checkpoint" that would call also "progress" instead of calling
  os.sleep().

    # Force a checkpoint in fast-import and wait for it to finish
    def checkpoint(self):
        self.gitStream.write("checkpoint\n\n")
        self.gitStream.write("progress checkpoint\n\n")
        out = self.gitOutput.readline()
        if self.verbose:
            print "checkpoint finished: " + out

This seems to work! :)

2. In order to avoid needing to use "--force" it would be nice to have
  some form of "drop" command in fast-import that would allows us to
  actively drop an older commit after calling "reset" (this could even
  be an option of "reset"). This way fast-import would not find
  dangling commits in the end of the import.  Maybe there's already
  some sort of command to achieve this in fast-import...? We could
  probably ask to one of its maintainers.

Just missing point 2 now.

-- 
Vitor Antunes