GitHub PushEvent at 2014-10-08 23:45:28
Repository: git
Actor: peff
Payload: {"shas":[["1bb207e0fecb49e35b14b673987cce2884239e04","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","tests: drop GIT_*_TIMING_TESTS environment variable support\n\nTwo tests (t3302 and t3419) used to have their own environment\nvariable to trigger expensive tests without enabling expensive\ntests in other scripts; a user could set GIT_NOTES_TIMING_TESTS\nbut not GIT_TEST_LONG and run the whole test suite and trigger\nexpensive tests only in t3302 but not other tests.  The same for\nGIT_PATCHID_TIMING_TESTS in t3419.\n\nWhile this may have seemed a good flexibility, in reality if you are\nconcentrating on a single test (e.g. t3302), you can just run that\nsingle test with the GIT_TEST_LONG to trigger expensive tests.  It\ndoes not seem worth forcing other people who may want to come up\nwith their own expensive tests to invent new environment variables\nby keeping this convention.\n\nDrop them.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["981ff520b7487886c773910d6309ec327efba3ea","092ec58085ae5e368bc39d18752740ff81a2d09c@maxsi.org","git-compat-util.h: add missing semicolon after struct itimerval\n\nThis hasn't been a problem in practice as almost all systems have the\nsetitimer() API (or it is provided by git in the case of mingw). This code\nwasn't used in any default circumstances, as the build system never sets\nNO_STRUCT_ITIMERVAL - this breakage only occured if the user asked for it.\n\nWe repair this case so we can rely on it in the following commits.\n\nSigned-off-by: Jonas 'Sortie' Termansen <sortie@maxsi.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jonas 'Sortie' Termansen",false],["6441090cf234b88c2fdbb9cd754714108fd57755","092ec58085ae5e368bc39d18752740ff81a2d09c@maxsi.org","autoconf: check for struct itimerval\n\nThe Makefile has provisions for this case, so let's detect it in the\nconfigure script as well.\n\nSigned-off-by: Jonas 'Sortie' Termansen <sortie@maxsi.org>\nSigned-off-by: Jacob Keller <jacob.e.keller@intel.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jonas 'Sortie' Termansen",false],["a6fd4fb55d255ae14dc3ced5eae6247b4a9d86ea","092ec58085ae5e368bc39d18752740ff81a2d09c@maxsi.org","autoconf: check for setitimer()\n\nThe Makefile has provisions for this case, so let's detect it in the\nconfigure script as well.\n\nSigned-off-by: Jonas 'Sortie' Termansen <sortie@maxsi.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jonas 'Sortie' Termansen",false],["56625df74c969885c0c735203283bb1ebe1897ec","9de348c050f7cd1ca590883733c4e531ce610bf4@gmail.com","Documentation: a note about stdout for git rev-parse --verify --quiet\n\nSigned-off-by: David Aguilar <davvid@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Aguilar",false],["2da1f36671c3e3e28eba6a56d92249b6671c07d3","696e3fbcf235d40b6ea1ebd61c87cbea79d444e7@peff.net","mailinfo: make \">From\" in-body header check more robust\n\nSince commit 81c5cf7 (mailinfo: skip bogus UNIX From line inside\nbody, 2006-05-21), we have treated lines like \">From\" in the body as\nheaders. This makes \"git am\" work for people who erroneously paste\nthe whole output from format-patch:\n\n  From 12345abcd...fedcba543210 Mon Sep 17 00:00:00 2001\n  From: them\n  Subject: [PATCH] whatever\n\ninto their email body (assuming that an mbox writer then quotes\n\"From\" as \">From\", as otherwise we would actually mailsplit on the\nin-body line).\n\nHowever, this has false positives if somebody actually has a commit\nbody that starts with \"From \"; in this case we erroneously remove\nthe line entirely from the commit message. We can make this check\nmore robust by making sure the line actually looks like a real mbox\n\"From\" line.\n\nInspect the line that begins with \">From \" a more carefully to only\nskip lines that match the expected pattern (note that the datestamp\npart of the format-patch output is designed to be kept constant to\nhelp those who write magic(5) entries).\n\nSigned-off-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Jeff King",false],["2892dfeec3f98f7e65a2746d271471d2c3c4af57","9de348c050f7cd1ca590883733c4e531ce610bf4@gmail.com","t1503: use test_must_be_empty\n\nUse `test_must_be_be_empty <file>` instead of `test -z \"$(cat <file>)\"`.\n\nSuggested-by: Fabian Ruch <bafain@gmail.com>\nSigned-off-by: David Aguilar <davvid@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Aguilar",false],["7559a1be8a0afb10df41d25e4cf4c5285a5faef1","d189729ad9bf83f8833434c2a6377966e347e012@github.com","unblock and unignore SIGPIPE\n\nBlocked and ignored signals -- but not caught signals -- are inherited\nacross exec.  Some callers with sloppy signal-handling behavior can call\ngit with SIGPIPE blocked or ignored, even non-deterministically.  When\nSIGPIPE is blocked or ignored, several git commands can run indefinitely,\nignoring EPIPE returns from write() calls, even when the process that\ncalled them has gone away.  Our specific case involved a pipe of git\ndiff-tree output to a script that reads a limited amount of diff data.\n\nIn an ideal world, git would never be called with SIGPIPE blocked or\nignored.  But in the real world, several real potential callers, including\nPerl, Apache, and Unicorn, sometimes spawn subprocesses with SIGPIPE\nignored.  It is easier and more productive to harden git against this\nmistake than to clean it up in every potential parent process.\n\nSigned-off-by: Patrick Reynolds <patrick.reynolds@github.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Patrick Reynolds",false],["9271095cc5571e306d709ebf8eb7f0a388254d9d","23a0b5e4fb6c6e8280940920212ecd563859cb3c@exec64.co.uk","pretty: add %D format specifier\n\nAdd a new format specifier, '%D' that is identical in behaviour to '%d',\nexcept that it does not include the ' (' prefix or ')' suffix provided\nby '%d'.\n\nSigned-off-by: Harry Jeffery <harry@exec64.co.uk>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Harry Jeffery",false],["c41a87dd80cd32cfd6e2d670153a9b69dc627f71","9de348c050f7cd1ca590883733c4e531ce610bf4@gmail.com","refs: make rev-parse --quiet actually quiet\n\nWhen a reflog is deleted, e.g. when \"git stash\" clears its stashes,\n\"git rev-parse --verify --quiet\" dies:\n\n\tfatal: Log for refs/stash is empty.\n\nThe reason is that the get_sha1() code path does not allow us\nto suppress this message.\n\nPass the flags bitfield through get_sha1_with_context() so that\nread_ref_at() can suppress the message.\n\nUse get_sha1_with_context1() instead of get_sha1() in rev-parse\nso that the --quiet flag is honored.\n\nSigned-off-by: David Aguilar <davvid@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Aguilar",false],["1956dfa818e376c5d62adc16d77f5b5cacf923f8","9de348c050f7cd1ca590883733c4e531ce610bf4@gmail.com","stash: prefer --quiet over shell redirection of the standard error stream\n\nUse `git rev-parse --verify --quiet` instead of redirecting\nstderr to /dev/null.\n\nSigned-off-by: David Aguilar <davvid@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Aguilar",false],["0176e7a71f2c722effde0ed22db3682400010911","2da278a99b557fc93ddbed7517a413ad793e2e6a@web.de","graph: simplify graph_padding_line()\n\nDeduplicate code common to both branches of if statements.\n\nSigned-off-by: Rene Scharfe <l.s.r@web.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","René Scharfe",false],["07bfa575c1ce741d0e33580c336596d3407129b6","2da278a99b557fc93ddbed7517a413ad793e2e6a@web.de","remote: simplify match_name_with_pattern() using strbuf\n\nMake the code simpler and shorter by avoiding repetitive use of\nstring length variables and leaving memory allocation to strbuf\nfunctions.\n\nSigned-off-by: Rene Scharfe <l.s.r@web.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","René Scharfe",false],["634c42da2282cfc718c72da85efff870cfe5f253","42740f1721ea98c94a07c13b76c86023ff11da2d@gmail.com","t9300-fast-import: fix typo in test description\n\nSigned-off-by: Stefan Beller <stefanbeller@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Stefan Beller",false],["8c3419bdbd5b49892c28ef32af80e0d570e5d7a5","42740f1721ea98c94a07c13b76c86023ff11da2d@gmail.com","t6031-test-merge-recursive: do not forget to add file to be committed\n\nSigned-off-by: Stefan Beller <stefanbeller@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Stefan Beller",false],["4e6d207c45e6f5c13b38c4a200f0d3339f88ad34","fd432345609ae4d58c1aa629cb66ba96d1391622@kdbg.org","mingw.h: add dummy functions for sigset_t operations\n\nWindows does not have POSIX-like signals, and so we ignore all\noperations on the non-existent signal mask machinery.\n\nDo not turn sigemptyset into a function, but leave it a macro that\nerases the code in the argument because it is used to set sa_mask\nof a struct sigaction, but our dummy in mingw.h does not have that\nmember.\n\nSigned-off-by: Johannes Sixt <j6t@kdbg.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Johannes Sixt",false],["85de86a16bd853708658eb13689d913bc377ae3e","8d6e34f987851aa599257d3831a1af040886842f@sunshineco.com","mailinfo: work around -Wstring-plus-int warning\n\nThe just-released Apple Xcode 6.0.1 has -Wstring-plus-int enabled by\ndefault which complains about pointer arithmetic applied to a string\nliteral:\n\n    builtin/mailinfo.c:303:24: warning:\n        adding 'long' to a string does not append to the string\n            return !memcmp(SAMPLE + (cp - line), cp, strlen(SAMPLE) ...\n                           ~~~~~~~^~~~~~~~~~~~~\n\nSigned-off-by: Eric Sunshine <sunshine@sunshineco.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Eric Sunshine",false],["422af49c2fe707c5e7bff1e4a8250232401154b0","42740f1721ea98c94a07c13b76c86023ff11da2d@gmail.com","merge-recursive: fix copy-paste mistake\n\nThe following issue was found by scan.coverity.com (ID: 1049510),\nand claimed to be likely a copy-paste mistake.\n\nIntroduced in 331a1838b (2010-07-02, Try normalizing files\nto avoid delete/modify conflicts when merging), which is\nquite a long time ago, so I'm rather unsure if it's of any impact\nor just went unnoticed.\n\nThe line after the changed line has a comparison of 'o.len' to 'a.len',\nso we should assume the lengths may be different.\n\nI'd be happy to have a test for this bug(?) attached to\nt6031-merge-recursive.sh, but I did not manage to\ncome up with a test in a reasonable amount of time.\n\nSigned-off-by: Stefan Beller <stefanbeller@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Stefan Beller",false],["040b2ac978366215bfc27be5e110a2f0170e6864","42740f1721ea98c94a07c13b76c86023ff11da2d@gmail.com","merge-recursive: remove stale commented debugging code\n\nSigned-off-by: Stefan Beller <stefanbeller@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Stefan Beller",false],["b8e533f12acec63242c5405286fbbcfa66eecfdd","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","Merge branch 'hj/pretty-naked-decoration'\n\nThe pretty-format specifier \"%d\", which expanded to \" (tagname)\"\nfor a tagged commit, gained a cousin \"%D\" that just gives the\n\"tagname\" without frills.\n\n* hj/pretty-naked-decoration:\n  pretty: add %D format specifier","Junio C Hamano",false]],"size":44,"ref":"refs/heads/jk/archive-reachability","head":"4fef25af680db36299b90ad456b34c6d962e8115"}