{"thread":{"id":"24833","subject":"[PATCH 0/3] git-bundle.txt options clarification","startedAt":"2010-08-23T09:02:36Z","lastAt":"2010-08-23T12:20:13Z","messageCount":7,"participants":["Michael J Gruber","Thomas Rast"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"148736","messageId":"cover.1282553585.git.git@drmicha.warpmail.net","threadId":"24833","inReplyTo":null,"subject":"[PATCH 0/3] git-bundle.txt options clarification","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-08-23T09:02:36Z","receivedAt":"2010-08-23T09:02:36Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"This mini-series is the outcome of my attempt with the following command:\n\ngit bundle master~7..master~5\n\nI understand (now) why it fails and why it must fail, but this should be\neasier to grasp from the man page. Before amending it, the 1st two patches\ndo some systematic and coding resp. whitespace cleanup. Feel free to squash\nas you deem appropriate.\n\nMichael J Gruber (3):\n git-bundle.txt: Cleanup\n git-bundle.txt: whitespace cleanup\n git-bundle.txt: Clarify rev-list-args restrictions\n\n Documentation/git-bundle.txt | 73 +++++++++++++++++++++--------------------\n 1 files changed, 37 insertions(+), 36 deletions(-)\n\n-- \n1.7.2.1.52.g7f7860\n"},{"id":"148737","messageId":"4cc5ea19fda713050f3f247d32df44fcfcea3296.1282553586.git.git@drmicha.warpmail.net","threadId":"24833","inReplyTo":"cover.1282553585.git.git@drmicha.warpmail.net","subject":"[PATCH 1/3] git-bundle.txt: Cleanup","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-08-23T09:02:37Z","receivedAt":"2010-08-23T09:02:37Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Cleanup various spellings of the same argument, as well as the code\nfor the tilde: Since neither '~' nor '\\~' work consistently, use\n'{tilde}'.\n\nSigned-off-by: Michael J Gruber \n---\n Documentation/git-bundle.txt | 18 +++++++++---------\n 1 files changed, 9 insertions(+), 9 deletions(-)\n\ndiff --git a/Documentation/git-bundle.txt b/Documentation/git-bundle.txt\nindex a5ed8fb..2d92696 100644\n--- a/Documentation/git-bundle.txt\n+++ b/Documentation/git-bundle.txt\n@@ -9,7 +9,7 @@ git-bundle - Move objects and refs by archive\n SYNOPSIS\n --------\n [verse]\n-'git bundle' create \n+'git bundle' create \n 'git bundle' verify \n 'git bundle' list-heads [refname...]\n 'git bundle' unbundle [refname...]\n@@ -35,7 +35,7 @@ OPTIONS\n \n create ::\n Used to create a bundle named 'file'. This requires the\n- 'git rev-list' arguments to define the bundle contents.\n+ 'git-rev-list-args' arguments to define the bundle contents.\n \n verify ::\n Used to check that a bundle file is valid and will apply\n@@ -57,10 +57,10 @@ unbundle ::\n references matching those in the list are printed. This command is\n really plumbing, intended to be called only by 'git fetch'.\n \n-[git-rev-list-args...]::\n+::\n A list of arguments, acceptable to 'git rev-parse' and\n 'git rev-list', that specifies the specific objects and references\n- to transport. For example, `master\\~10..master` causes the\n+ to transport. For example, `master{tilde}10..master` causes the\n current master reference to be packaged along with all objects\n added since its 10th ancestor commit. There is no explicit\n limit to the number of references and objects that may be\n@@ -79,12 +79,12 @@ SPECIFYING REFERENCES\n \n 'git bundle' will only package references that are shown by\n 'git show-ref': this includes heads, tags, and remote heads. References\n-such as `master\\~1` cannot be packaged, but are perfectly suitable for\n+such as `master{tilde}1` cannot be packaged, but are perfectly suitable for\n defining the basis. More than one reference may be packaged, and more\n than one basis can be specified. The objects packaged are those not\n contained in the union of the given bases. Each basis can be\n-specified explicitly (e.g. `^master\\~10`), or implicitly (e.g.\n-`master\\~10..master`, `--since=10.days.ago master`).\n+specified explicitly (e.g. `^master{tilde}10`), or implicitly (e.g.\n+`master{tilde}10..master`, `--since=10.days.ago master`).\n \n It is very important that the basis used be held by the destination.\n It is okay to err on the side of caution, causing the bundle file\n@@ -154,7 +154,7 @@ machineB$ git pull\n If you know up to what commit the intended recipient repository should\n have the necessary objects, you can use that knowledge to specify the\n basis, giving a cut-off point to limit the revisions and objects that go\n-in the resulting bundle. The previous example used lastR2bundle tag\n+in the resulting bundle. The previous example used the lastR2bundle tag\n for this purpose, but you can use any other options that you would give to\n the linkgit:git-log[1] command. Here are more examples:\n \n@@ -194,7 +194,7 @@ references when fetching:\n $ git fetch mybundle master:localRef\n ----------------\n \n-You can also see what references it offers.\n+You can also see what references it offers:\n \n ----------------\n $ git ls-remote mybundle\n-- \n1.7.2.1.52.g7f7860\n"},{"id":"148738","messageId":"af750e09280f8d5118e1d854fa839cb1a7c32579.1282553586.git.git@drmicha.warpmail.net","threadId":"24833","inReplyTo":"cover.1282553585.git.git@drmicha.warpmail.net","subject":"[PATCH 2/3] git-bundle.txt: whitespace cleanup","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-08-23T09:02:38Z","receivedAt":"2010-08-23T09:02:38Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Use tabs, not 7 spaces.\n\nSigned-off-by: Michael J Gruber \n---\n Documentation/git-bundle.txt | 56 +++++++++++++++++++++---------------------\n 1 files changed, 28 insertions(+), 28 deletions(-)\n\ndiff --git a/Documentation/git-bundle.txt b/Documentation/git-bundle.txt\nindex 2d92696..3cdbc72 100644\n--- a/Documentation/git-bundle.txt\n+++ b/Documentation/git-bundle.txt\n@@ -34,45 +34,45 @@ OPTIONS\n -------\n \n create ::\n- Used to create a bundle named 'file'. This requires the\n- 'git-rev-list-args' arguments to define the bundle contents.\n+\tUsed to create a bundle named 'file'. This requires the\n+\t'git-rev-list-args' arguments to define the bundle contents.\n \n verify ::\n- Used to check that a bundle file is valid and will apply\n- cleanly to the current repository. This includes checks on the\n- bundle format itself as well as checking that the prerequisite\n- commits exist and are fully linked in the current repository.\n- 'git bundle' prints a list of missing commits, if any, and exits\n- with a non-zero status.\n+\tUsed to check that a bundle file is valid and will apply\n+\tcleanly to the current repository. This includes checks on the\n+\tbundle format itself as well as checking that the prerequisite\n+\tcommits exist and are fully linked in the current repository.\n+\t'git bundle' prints a list of missing commits, if any, and exits\n+\twith a non-zero status.\n \n list-heads ::\n- Lists the references defined in the bundle. If followed by a\n- list of references, only references matching those given are\n- printed out.\n+\tLists the references defined in the bundle. If followed by a\n+\tlist of references, only references matching those given are\n+\tprinted out.\n \n unbundle ::\n- Passes the objects in the bundle to 'git index-pack'\n- for storage in the repository, then prints the names of all\n- defined references. If a list of references is given, only\n- references matching those in the list are printed. This command is\n- really plumbing, intended to be called only by 'git fetch'.\n+\tPasses the objects in the bundle to 'git index-pack'\n+\tfor storage in the repository, then prints the names of all\n+\tdefined references. If a list of references is given, only\n+\treferences matching those in the list are printed. This command is\n+\treally plumbing, intended to be called only by 'git fetch'.\n \n ::\n- A list of arguments, acceptable to 'git rev-parse' and\n- 'git rev-list', that specifies the specific objects and references\n- to transport. For example, `master{tilde}10..master` causes the\n- current master reference to be packaged along with all objects\n- added since its 10th ancestor commit. There is no explicit\n- limit to the number of references and objects that may be\n- packaged.\n+\tA list of arguments, acceptable to 'git rev-parse' and\n+\t'git rev-list', that specifies the specific objects and references\n+\tto transport. For example, `master{tilde}10..master` causes the\n+\tcurrent master reference to be packaged along with all objects\n+\tadded since its 10th ancestor commit. There is no explicit\n+\tlimit to the number of references and objects that may be\n+\tpackaged.\n \n \n [refname...]::\n- A list of references used to limit the references reported as\n- available. This is principally of use to 'git fetch', which\n- expects to receive only those references asked for and not\n- necessarily everything in the pack (in this case, 'git bundle' acts\n- like 'git fetch-pack').\n+\tA list of references used to limit the references reported as\n+\tavailable. This is principally of use to 'git fetch', which\n+\texpects to receive only those references asked for and not\n+\tnecessarily everything in the pack (in this case, 'git bundle' acts\n+\tlike 'git fetch-pack').\n \n SPECIFYING REFERENCES\n ---------------------\n-- \n1.7.2.1.52.g7f7860\n"},{"id":"148739","messageId":"4a2d0ecb46d274f6143caab6982535bfb678d10e.1282553586.git.git@drmicha.warpmail.net","threadId":"24833","inReplyTo":"cover.1282553585.git.git@drmicha.warpmail.net","subject":"[PATCH 3/3] git-bundle.txt: Clarify rev-list-args restrictions","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-08-23T09:02:39Z","receivedAt":"2010-08-23T09:02:39Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Currently, one could think that 'git bundle create' groks\nany 'git rev-list' expression. But in fact it requires a named reference\nto be present. Try and make this clearer.\n\nSigned-off-by: Michael J Gruber \n---\n Documentation/git-bundle.txt | 3 ++-\n 1 files changed, 2 insertions(+), 1 deletions(-)\n\ndiff --git a/Documentation/git-bundle.txt b/Documentation/git-bundle.txt\nindex 3cdbc72..38e59af 100644\n--- a/Documentation/git-bundle.txt\n+++ b/Documentation/git-bundle.txt\n@@ -59,7 +59,8 @@ unbundle ::\n \n ::\n \tA list of arguments, acceptable to 'git rev-parse' and\n-\t'git rev-list', that specifies the specific objects and references\n+\t'git rev-list' (and containg a named ref, see SPECIFYING REFERENCES\n+\tbelow), that specifies the specific objects and references\n \tto transport. For example, `master{tilde}10..master` causes the\n \tcurrent master reference to be packaged along with all objects\n \tadded since its 10th ancestor commit. There is no explicit\n-- \n1.7.2.1.52.g7f7860\n"},{"id":"148745","messageId":"201008231359.00417.trast@student.ethz.ch","threadId":"24833","inReplyTo":"4cc5ea19fda713050f3f247d32df44fcfcea3296.1282553586.git.git@drmicha.warpmail.net","subject":"Re: [PATCH 1/3] git-bundle.txt: Cleanup","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2010-08-23T11:59:00Z","receivedAt":"2010-08-23T11:59:00Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Michael J Gruber wrote:\n> Cleanup various spellings of the same argument\n[...]\n> - 'git rev-list' arguments to define the bundle contents.\n> + 'git-rev-list-args' arguments to define the bundle contents.\n\nAs indicated by the Cc but not the text, this was fallout from my\nautomated replacements in 0b444cd (Documentation: spell 'git cmd'\nwithout dash throughout, 2010-01-10).\n\nAn analogous change to one of your {tilde} replacements is already in\nJonathan's series. The rest probably don't hurt, if only to drive\npeople towards a more consistent {tilde} spelling in the future.\n\nSo, Ack.\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"148746","messageId":"201008231401.26329.trast@student.ethz.ch","threadId":"24833","inReplyTo":"4a2d0ecb46d274f6143caab6982535bfb678d10e.1282553586.git.git@drmicha.warpmail.net","subject":"Re: [PATCH 3/3] git-bundle.txt: Clarify rev-list-args restrictions","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2010-08-23T12:01:25Z","receivedAt":"2010-08-23T12:01:25Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Michael J Gruber wrote:\n> diff --git a/Documentation/git-bundle.txt b/Documentation/git-bundle.txt\n> index 3cdbc72..38e59af 100644\n> --- a/Documentation/git-bundle.txt\n> +++ b/Documentation/git-bundle.txt\n> @@ -59,7 +59,8 @@ unbundle ::\n> \n> ::\n> \tA list of arguments, acceptable to 'git rev-parse' and\n> -\t'git rev-list', that specifies the specific objects and references\n> +\t'git rev-list' (and containg a named ref, see SPECIFYING REFERENCES\n> +\tbelow), that specifies the specific objects and references\n> \tto transport. For example, `master{tilde}10..master` causes the\n> \tcurrent master reference to be packaged along with all objects\n> \tadded since its 10th ancestor commit. There is no explicit\n\nAck on this, too; having it point at \"Specifying References\" is a good\nthing.\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"},{"id":"148747","messageId":"4C72677D.9030205@drmicha.warpmail.net","threadId":"24833","inReplyTo":"201008231359.00417.trast@student.ethz.ch","subject":"Re: [PATCH 1/3] git-bundle.txt: Cleanup","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-08-23T12:20:13Z","receivedAt":"2010-08-23T12:20:13Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Thomas Rast venit, vidit, dixit 23.08.2010 13:59:\n> Michael J Gruber wrote:\n>> Cleanup various spellings of the same argument\n> [...]\n>> - 'git rev-list' arguments to define the bundle contents.\n>> + 'git-rev-list-args' arguments to define the bundle contents.\n> \n> As indicated by the Cc but not the text, this was fallout from my\n> automated replacements in 0b444cd (Documentation: spell 'git cmd'\n> without dash throughout, 2010-01-10).\n\nNo blame intended (and no \"git blame\" run). I cc'ed you as the doc\nsheriff ;)\n\nThe change above was really about using the place holder name\nconsistently, which includes keeping the \"-args\".\n\n> An analogous change to one of your {tilde} replacements is already in\n> Jonathan's series. The rest probably don't hurt, if only to drive\n\nI was behind an idiotic firewall for the last 2 weeks which not only\nblocked vpn and nntp but made it impossible to get larger chunks over\ngit:, git+ssh: or even scp (bundle) even though bandwidth was no\napparent issue. One of those countries which are currently argueing with\nRIM. Makes you wonder how routinely they do content sniffing and how\nthey react to any (false) triggers. Might also have been an overly eager\ninstitution, of course.\n\nSo I didn't see Jonathan's series in time.\n\n> people towards a more consistent {tilde} spelling in the future.\n\nI did some experimenting after noticing that I had many broken ~ (either\ndisappearing or appearing with spurious \\ in the html). I might even\nhave come up with a rule, but I'm afraid these things change from\nasciidoc version to asciidoc version, so that I went the chatty {tilde}.\n\nCheers (back on home soil),\nMichael\n"}]}