On Fri, May 3, 2013 at 5:08 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Thomas Rast <trast@xxxxxxxxxxx> writes: > >> IIUC, you are unconditionally storing only marks to commit objects. >> >> Are you allowed to do that at this point? I notice that >> git-fast-export(1) says >> >> --export-marks=<file> >> Dumps the internal marks table to <file> when complete. Marks are >> written one per line as :markid SHA-1. Only marks for revisions >> are dumped[...] >> >> But git-fast-import(1) says nothing of the sort; I would even claim that >> >> --export-marks=<file> >> Dumps the internal marks table to <file> when complete. >> >> means that the *full* marks table is dumped. >> >> How do we know that this doesn't break any users of fast-import? Your >> comment isn't very reassuring: >> >>> the vast majority of them will never be used again >> >> So what's with the minority? >> >> In any case, if this does go in, please update the documentation to >> match, probably by copying the sentence from git-fast-export(1). > > A safe and sane approach may be to teach these an option to tell > them to omit non-commits or to emit all kinds, and make remote-bzr > use that to exclude non-commits. This has nothing to do with remote-bzr, or any remote helper. These objects are not useful, not even to 'git fast-export'. > If the defaults is matched to the > current behaviour, nobody will get hurt Changing nothing always ensures that nobody will get hurt, but that doesn't improve anything either. -- Felipe Contreras -- 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