Heya, [picking up a topic on what to do if there's a import-marks from the stream and the commandline] On Thu, Aug 13, 2009 at 18:25, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote: > Sverre Rabbelier <srabbelier@xxxxxxxxx> wrote: >> On Thu, Aug 13, 2009 at 10:07, Johannes >> Schindelin<Johannes.Schindelin@xxxxxx> wrote: >> > ... and will import the marks twice? >> >> Ah, you're right :(. What's the best way to do this? Should we dump >> any previous marks when importing new ones? > > Uh, well, yes. We shouldn't define :5 if it was in the file that > appeared in the stream, but isn't in the file on the command line. > > Worse, what happens if we do this: > > echo "option import-marks=/not/found" \ > | git fast-import --import-marks=my.marks > > I want this to work, even though /not/found does not exist, but > my.marks does. So that does complicate things... According to the docs it is allowed to specify import-marks multiple times (the latest series does not honor that, and there's apparently no test to verify it). What should the behavior be wrt stream/commandline arguments? Load from stream if present and valid, and also load from commandline? In that case, how do you override what's in the stream? I think the simplest is to allow the stream to specify a marks file exactly once, and commandline arguments override what's in the stream. Listing import-marks on the commandline is still valid and keeps the old behavior. Sensible? -- Cheers, Sverre Rabbelier -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html