{"thread":{"id":"16579","subject":"[PATCH 2/5] lost-found: use git rev-parse -q","startedAt":"2008-12-03T13:26:47Z","lastAt":"2008-12-03T13:26:52Z","messageCount":6,"participants":["Miklos Vajna"],"isPatch":true,"patchVersion":1,"patchTotal":5},"messages":[{"id":"97062","messageId":"cover.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":null,"subject":"[PATCH 0/5] More \"use rev-parse -q\" patches","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:47Z","receivedAt":"2008-12-03T13:26:47Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Hi,\n\nI'm sending these as separate patches as they touch completely unrelated\nparts of Git, but feel free to squash them together you think that's\nbetter.\n\nAll of them is about using rev-parse -q instead of rev-parse 2>/dev/null.\n\nMiklos Vajna (5):\n filter-branch: use git rev-parse -q\n lost-found: use git rev-parse -q\n pull: use git rev-parse -q\n rebase: use git rev-parse -q\n submodule: use git rev-parse -q\n\n git-filter-branch.sh | 2 +-\n git-lost-found.sh | 2 +-\n git-pull.sh | 8 ++++----\n git-rebase.sh | 4 ++--\n git-submodule.sh | 6 +++---\n 5 files changed, 11 insertions(+), 11 deletions(-)\n"},{"id":"97064","messageId":"4858dc4d5d92276fa96a8fb45b02adaae57dcf26.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":"cover.1228310570.git.vmiklos@frugalware.org","subject":"[PATCH 1/5] filter-branch: use git rev-parse -q","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:48Z","receivedAt":"2008-12-03T13:26:48Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Signed-off-by: Miklos Vajna \n---\n git-filter-branch.sh | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/git-filter-branch.sh b/git-filter-branch.sh\nindex 81392ad..c106f45 100755\n--- a/git-filter-branch.sh\n+++ b/git-filter-branch.sh\n@@ -256,7 +256,7 @@ while read commit parents; do\n \t*)\n \t\t# The commit may not have the subdirectory at all\n \t\terr=$(git read-tree -i -m $commit:\"$filter_subdir\" 2>&1) || {\n-\t\t\tif ! git rev-parse --verify $commit:\"$filter_subdir\" 2>/dev/null\n+\t\t\tif ! git rev-parse -q --verify $commit:\"$filter_subdir\"\n \t\t\tthen\n \t\t\t\trm -f \"$GIT_INDEX_FILE\"\n \t\t\telse\n-- \n1.6.0.4\n"},{"id":"97060","messageId":"73193271279140eda04ec46097936e64f3c526d6.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":"4858dc4d5d92276fa96a8fb45b02adaae57dcf26.1228310570.git.vmiklos@frugalware.org","subject":"[PATCH 2/5] lost-found: use git rev-parse -q","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:49Z","receivedAt":"2008-12-03T13:26:49Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Signed-off-by: Miklos Vajna \n---\n git-lost-found.sh | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/git-lost-found.sh b/git-lost-found.sh\nindex 9cedaf8..0b3e8c7 100755\n--- a/git-lost-found.sh\n+++ b/git-lost-found.sh\n@@ -20,7 +20,7 @@ while read dangling type sha1\n do\n \tcase \"$dangling\" in\n \tdangling)\n-\t\tif git rev-parse --verify \"$sha1^0\" >/dev/null 2>/dev/null\n+\t\tif git rev-parse -q --verify \"$sha1^0\" >/dev/null\n \t\tthen\n \t\t\tdir=\"$laf/commit\"\n \t\t\tgit show-branch \"$sha1\"\n-- \n1.6.0.4\n"},{"id":"97061","messageId":"8c4e8eda234e8e37d4a7a3ac43ffc7738e6fac4b.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":"73193271279140eda04ec46097936e64f3c526d6.1228310570.git.vmiklos@frugalware.org","subject":"[PATCH 3/5] pull: use git rev-parse -q","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:50Z","receivedAt":"2008-12-03T13:26:50Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Signed-off-by: Miklos Vajna \n---\n git-pull.sh | 8 ++++----\n 1 files changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/git-pull.sh b/git-pull.sh\nindex 1cac898..2c7f432 100755\n--- a/git-pull.sh\n+++ b/git-pull.sh\n@@ -121,13 +121,13 @@ test true = \"$rebase\" && {\n \ttest -z \"$origin\" && origin=$(get_default_remote)\n \treflist=\"$(get_remote_refs_for_fetch \"$@\" 2>/dev/null |\n \t\tsed \"s|refs/heads/\\(.*\\):|\\1|\")\" &&\n-\toldremoteref=\"$(git rev-parse --verify \\\n-\t\t\"refs/remotes/$origin/$reflist\" 2>/dev/null)\"\n+\toldremoteref=\"$(git rev-parse -q --verify \\\n+\t\t\"refs/remotes/$origin/$reflist\")\"\n }\n-orig_head=$(git rev-parse --verify HEAD 2>/dev/null)\n+orig_head=$(git rev-parse -q --verify HEAD)\n git fetch $verbosity --update-head-ok \"$@\" || exit 1\n \n-curr_head=$(git rev-parse --verify HEAD 2>/dev/null)\n+curr_head=$(git rev-parse -q --verify HEAD)\n if test -n \"$orig_head\" && test \"$curr_head\" != \"$orig_head\"\n then\n \t# The fetch involved updating the current branch.\n-- \n1.6.0.4\n"},{"id":"97063","messageId":"b67238b67501272485cc659c11a836f4f96e0d81.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":"8c4e8eda234e8e37d4a7a3ac43ffc7738e6fac4b.1228310570.git.vmiklos@frugalware.org","subject":"[PATCH 4/5] rebase: use git rev-parse -q","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:51Z","receivedAt":"2008-12-03T13:26:51Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Signed-off-by: Miklos Vajna \n---\n git-rebase.sh | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-rebase.sh b/git-rebase.sh\nindex 023a6dc..ea7720d 100755\n--- a/git-rebase.sh\n+++ b/git-rebase.sh\n@@ -365,10 +365,10 @@ case \"$#\" in\n \tswitch_to=\"$2\"\n \n \tif git show-ref --verify --quiet -- \"refs/heads/$2\" &&\n-\t branch=$(git rev-parse --verify \"refs/heads/$2\" 2>/dev/null)\n+\t branch=$(git rev-parse -q --verify \"refs/heads/$2\")\n \tthen\n \t\thead_name=\"refs/heads/$2\"\n-\telif branch=$(git rev-parse --verify \"$2\" 2>/dev/null)\n+\telif branch=$(git rev-parse -q --verify \"$2\")\n \tthen\n \t\thead_name=\"detached HEAD\"\n \telse\n-- \n1.6.0.4\n"},{"id":"97065","messageId":"c3a20382adecc78a12c9cf1bff9beb102d323b5e.1228310570.git.vmiklos@frugalware.org","threadId":"16579","inReplyTo":"b67238b67501272485cc659c11a836f4f96e0d81.1228310570.git.vmiklos@frugalware.org","subject":"[PATCH 5/5] submodule: use git rev-parse -q","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-03T13:26:52Z","receivedAt":"2008-12-03T13:26:52Z","isPatch":true,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"Signed-off-by: Miklos Vajna \n---\n git-submodule.sh | 6 +++---\n 1 files changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/git-submodule.sh b/git-submodule.sh\nindex 220d94e..2f47e06 100755\n--- a/git-submodule.sh\n+++ b/git-submodule.sh\n@@ -413,7 +413,7 @@ cmd_summary() {\n \n \ttest $summary_limit = 0 && return\n \n-\tif rev=$(git rev-parse --verify \"$1^0\" 2>/dev/null)\n+\tif rev=$(git rev-parse -q --verify \"$1^0\")\n \tthen\n \t\thead=$rev\n \t\tshift\n@@ -464,11 +464,11 @@ cmd_summary() {\n \t\tmissing_dst=\n \n \t\ttest $mod_src = 160000 &&\n-\t\t! GIT_DIR=\"$name/.git\" git-rev-parse --verify $sha1_src^0 >/dev/null 2>&1 &&\n+\t\t! GIT_DIR=\"$name/.git\" git-rev-parse -q --verify $sha1_src^0 >/dev/null &&\n \t\tmissing_src=t\n \n \t\ttest $mod_dst = 160000 &&\n-\t\t! GIT_DIR=\"$name/.git\" git-rev-parse --verify $sha1_dst^0 >/dev/null 2>&1 &&\n+\t\t! GIT_DIR=\"$name/.git\" git-rev-parse -q --verify $sha1_dst^0 >/dev/null &&\n \t\tmissing_dst=t\n \n \t\ttotal_commits=\n-- \n1.6.0.4\n"}]}