{"thread":{"id":"1088","subject":"[PATCH 2/2] Change to gitk to make it use --topo-order rather than --merge-order","startedAt":"2005-07-02T06:09:38Z","lastAt":"2005-07-02T06:09:38Z","messageCount":1,"participants":["Jon Seymour"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"5561","messageId":"20050702060938.13030.qmail@blackcubes.dyndns.org","threadId":"1088","inReplyTo":null,"subject":"[PATCH 2/2] Change to gitk to make it use --topo-order rather than --merge-order","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2005-07-02T06:09:38Z","receivedAt":"2005-07-02T06:09:38Z","isPatch":true,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"\nThis patch make gitk use --topo-order rather than --merge-order.\n\nThere is no actual advantage in doing this as the git-rev-list \nexecution times are dwarfed by tcl/tk overheads so the 3-4 time\ngit-rev-list speedup is effectively invisible. Still, it\ndoes demonstrate the use of the --topo-order flag.\n\nOne advantage of this patch would be to allow gitk to be used\nin environments that don't have the ssl bignum library available\nfor some reason.\n\nSigned-off-by: Jon Seymour <jon.seymour@gmail.com>\n---\nThis patch assumes that git has been patched to include support\nfor --topo-order.\n\nHere's a chance to test merging a gitk patch from git back into\ngitk :-)\n---\n\n gitk |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\n4deb99b73fd6fc8e4995b55e9e23988c8766f509\ndiff --git a/gitk b/gitk\n--- a/gitk\n+++ b/gitk\n@@ -37,7 +37,7 @@ proc getcommits {rargs} {\n \tset parsed_args $rargs\n     }\n     if [catch {\n-\tset commfd [open \"|git-rev-list --header --merge-order $parsed_args\" r]\n+\tset commfd [open \"|git-rev-list --header --topo-order $parsed_args\" r]\n     } err] {\n \tputs stderr \"Error executing git-rev-list: $err\"\n \texit 1\n------------\n"}]}