GitHub PushEvent at 2014-11-06 08:22:06
Repository: git
Actor: peff
Payload: {"shas":[["f1dd90bd193637eeef772890c37afe3529a665d0","696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","t5304: use test_path_is_* instead of \"test -f\"\n\nThis is slightly more robust (checking \"! test -f\" would not\nnotice a directory of the same name, though that is not\nlikely to happen here). It also makes debugging easier, as\nthe test script will output a message on failure.\n\nSigned-off-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jeff King",false],["8ad16524183baf196d1db82b99ef52d05ca438e9","696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","t5304: use helper to report failure of \"test foo = bar\"\n\nFor small outputs, we sometimes use:\n\n  test \"$(some_cmd)\" = \"something we expect\"\n\ninstead of a full test_cmp. The downside of this is that\nwhen it fails, there is no output at all from the script.\nLet's introduce a small helper to make tests easier to\ndebug.\n\nSigned-off-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jeff King",false],["64a7e92f28e9bc053cf68d9b2e9ce4fb40e60e98","2da278a99b557fc93ddbed7517a413ad793e2e6a@web.de","receive-pack: plug minor memory leak in unpack()\n\nThe argv_array used in unpack() is never freed.  Instead of adding\nexplicit calls to argv_array_clear() use the args member of struct\nchild_process and let run_command() and friends clean up for us.\n\nSigned-off-by: Rene Scharfe <l.s.r@web.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","René Scharfe",false],["e0e21283b60da5d5d9d5f95f9f2a73189fa978f1","711813140c2a065f41125e654e0855dc11e31f6a@gmail.com","index-pack: fix compilation with NO_PTHREADS\n\ntype_cas_lock/unlock() should be defined as no-op for NO_PTHREADS\nbuild, just like all the other locking primitives.\n\nSigned-off-by: Etienne Buira <etienne.buira@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Etienne Buira",false],["0c45d258ec35c1ef51523dd45e4518bd8a09258c","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","pack-objects: set number of threads before checking and warning\n\nUnder NO_PTHREADS build, we warn when delta_search_threads is not\nset to 1, because that is the only sensible value on a single\nthreaded build.\n\nHowever, the auto detection that kicks in when that variable is set\nto 0 (e.g. there is no configuration variable or command line option,\nor an explicit --threads=0 is given from the command line to override\nthe pack.threads configuration to force auto-detection) was not done\nbefore the condition to issue this warning was tested.\n\nMove the auto-detection code and place it at an appropriate spot.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["c9a5172fdf073e090111950d3d42c4f9ef144ffd","738bdd359be778fee9f0fc4e2934ad72f436ceda@crustytoothpaste.net","Documentation: adjust document title underlining\n\nAsciiDoc specification states that in two-line titles, the title\nunderline has to be the same length as the title text, plus or minus two\ncharacters.  Asciidoctor, however, requires that this must be plus or\nminus one character.  Adjust the underlines to be the same length as the\ntitle text to improve compatibility.\n\nSigned-off-by: brian m. carlson <sandals@crustytoothpaste.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","brian m. carlson",false],["f8a48affbb8829190fe3aa8a1092d5762fd921b1","738bdd359be778fee9f0fc4e2934ad72f436ceda@crustytoothpaste.net","Documentation: fix mismatched delimiters in git-imap-send\n\nThe documentation for git-imap-send uses block delimiters with\nmismatched lengths, which Asciidoctor doesn't support.  As a result, the\npage is misrendered.  Adjust the delimiters so that they are of the same\nlength.\n\nSigned-off-by: brian m. carlson <sandals@crustytoothpaste.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","brian m. carlson",false],["9385b5d7065351104c7bfd3d1e53a28207b8b7ed","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: add data structures and basic functions\n\nWe will use a doubly linked list to store all information\nabout trailers and their configuration.\n\nThis way we can easily remove or add trailers to or from\ntrailer lists while traversing the lists in either direction.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["4103818d20a4aa8f2473bff3cc8e2f9048d9c310","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: process trailers from input message and arguments\n\nImplement the logic to process trailers from the input message\nand from arguments.\n\nAt the beginning trailers from the input message are in their\nown \"in_tok\" doubly linked list, and trailers from arguments\nare in their own \"arg_tok\" doubly linked list.\n\nThe lists are traversed and when an \"arg_tok\" should be \"applied\",\nit is removed from its list and inserted into the \"in_tok\" list.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["46a0613f00f2cf6a61d04fa49131fcbe3afaad07","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: read and process config information\n\nRead the configuration to get trailer information, and then process\nit and store it in a doubly linked list.\n\nThe config information is stored in the list whose first item is\npointed to by:\n\nstatic struct trailer_item *first_conf_item;\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["f0a90b4edff44af9a3cb3c30ff8e656a59581a2f","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: process command line trailer arguments\n\nParse the trailer command line arguments and put\nthe result into an arg_tok doubly linked list.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["2013d8505da95d355e610c08473f2e0b45645bf9","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: parse trailers from file or stdin\n\nRead trailers from a file or from stdin, parse the trailers and then\nput the result into a doubly linked list.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["b1d78d77bfbf026e8222c02f1009767a466de0b1","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: put all the processing together and print\n\nThis patch adds the process_trailers() function that\ncalls all the previously added processing functions\nand then prints the results on the standard output.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["6634f05454662b62e1f34766d3684b99fb53ad66","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: add interpret-trailers command\n\nThis patch adds the \"git interpret-trailers\" command.\nThis command uses the previously added process_trailers()\nfunction in trailer.c.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["76bed78a5829392d4e4dcf21b899d372dd71f408","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: add tests for \"git interpret-trailers\"\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["85039fb6e4e7123958a40a22ef40e25bcb29f9c0","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: execute command from 'trailer.<name>.command'\n\nLet the user specify a command that will give on its standard output\nthe value to use for the specified trailer.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["b9384ff34e18890c6b5aab91669094f0a3b411ae","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","trailer: add tests for commands in config file\n\nAnd add a few other tests for some special cases.\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["dfd66ddf5aa9aacc06ed38a9ee2cc15a4f2d4896","ddbaab5867ed34fd53535b58c9cdd59abd2e2aff@tuxfamily.org","Documentation: add documentation for 'git interpret-trailers'\n\nWhile at it add git-interpret-trailers to \"command-list.txt\".\n\nSigned-off-by: Christian Couder <chriscool@tuxfamily.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Christian Couder",false],["dc05179b5a8a6dcc9a25de461d43fb5edfc93c46","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","t1308: fix broken here document in test script\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["a136f6d8ff290ab486c85cd12d1a9a47d53b432d","696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","test-lib.sh: support -x option for shell-tracing\n\nUsually running a test under \"-v\" makes it clear which\ncommand is failing. However, sometimes it can be useful to\nalso see a complete trace of the shell commands being run in\nthe test. You can do so without any support from the test\nsuite by running \"sh -x tXXXX-foo.sh\". However, this\nproduces quite a large bit of output, as we see a trace of\nthe entire test suite.\n\nThis patch instead introduces a \"-x\" option to the test\nscripts (i.e., \"./tXXXX-foo.sh -x\"). When enabled, this\nturns on \"set -x\" only for the tests themselves. This can\nstill be a bit verbose, but should keep things to a more\nmanageable level. You can even use \"--verbose-only\" to see\nthe trace only for a specific test.\n\nThe implementation is a little invasive. We turn on the \"set\n-x\" inside the \"eval\" of the test code. This lets the eval\nitself avoid being reported in the trace (which would be\nlong, and redundant with the verbose listing we already\nshowed). And then after the eval runs, we do some trickery\nwith stderr to avoid showing the \"set +x\" to the user.\n\nWe also show traces for test_cleanup functions (since they\ncan impact the test outcome, too). However, we do avoid\nrunning the noop \":\" cleanup (the default if the test does\nnot use test_cleanup at all), as it creates unnecessary\nnoise in the \"set -x\" output.\n\nSigned-off-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jeff King",false]],"size":189,"ref":"refs/heads/jk/negative-refspecs-wip","head":"12ccf5e5dce517e98330450da5e6170f37897030"}