Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > Split out the discussion of why you should or should not create "open > bundles" into a new section, away from the discussion of our handling > of the rev-args syntax. I changed my mind about this part. * There is no option to create a bundle with one or more prerequisite commits with a non-thin packfile as its payload part. So "THIN BUNDLE" is an unnecessary new jargon and concept that does not help readers at all, because there is no "thick bundle" that has prerequisite commits. Let's not introduce such a new word. * We do not call DAG in our history with bottom end (i.e. a range with one or more uninteresting commits) an "open history". Let's not invent the words, "open" vs "close", to describe the history recorded in a bundle, either. We do already have an appropriate phrase we can use to explain what you wrote in open/close section: "the prerequisite commits" (find it in the description of "verify" subcommand) and that has been a concept well understood for a long time, ever since 2e0afafe (Add git-bundle: move objects and references by archive, 2007-02-22) introduced the bundle format. Let's use that phrase, instead of inventing a new word. In short, there are two kinds of bundles. - A bundle with one or more prerequisite commits can only be extracted in a repository that already has all these prerequisite commits and the history behind them, and "git bundle verify" is an easy way to check if your recipient repository has the prerequisites. The payload in such a bundle is always a thin packfile, but such a technical detail does not concern readers. - A bundle without any prerequisites can be extracted into anywhere, even into an empty repository, or cloned from it. And that should be sufficient to base your new paragraph(s). Sorry for having made a potentially confusing comment earlier. What I said about open vs thin was technically correct (i.e. there wasn't fundamental reason why a bundle with prerequisites MUST use a thin pack as its payload---we designed it that way because such a bundle MUST be extracted in a repository with these prerequisites present, hence it was OK to assume that a --fix-thin would work while extracting such a bundle), but it is irrelevant in practice because you cannot ask for a thick packfile to be created when creating such a bundle anyway. Thanks.
