{"push_id":1211507479,"size":92,"distinct_size":0,"ref":"refs/heads/jk/child-cleanup","head":"5dd8b0ae2b7f366e6ece5ef3f9eafe1588beb299","before":"87730e53fdb7da38e671914a969513da2824e47a","commits":[{"sha":"17a07e2ae2de36cadfac2349189e0c65e1c08d07","author":{"name":"David Kastrup","email":"da88ec47658348441bddc1d7d5ea2cbbfff2e571@gnu.org"},"message":"blame: require 0 context lines while finding moved lines with -M\n\nThe core part of git blame -M required 1 context line, but\nthere is no rationale to be found in the code; it causes artifacts\nlike discussed in the thread:\n\n  <http://thread.gmane.org/gmane.comp.version-control.git/255289/>.\n\n<http://permalink.gmane.org/gmane.comp.version-control.git/295795>\nsheds some more light on the history of the previous choice.\n\nSigned-off-by: David Kastrup <dak@gnu.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/17a07e2ae2de36cadfac2349189e0c65e1c08d07"},{"sha":"6ea57703f660afd00159b67bf8749d90881df3b4","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"log: prepare log/log-tree to reuse the diffopt.close_file attribute\n\nWe are about to teach the log-tree machinery to reuse the diffopt.file\nfield to output to a file stream other than stdout, in line with the\ndiff machinery already writing to diffopt.file.\n\nHowever, we might want to write something after the diff in\nlog_tree_commit() (e.g. with the --show-linear-break option), therefore\nwe must not let the diff machinery close the file (as per\ndiffopt.close_file.\n\nThis means that log_tree_commit() itself must override the\ndiffopt.close_file flag and close the file, and if log_tree_commit() is\ncalled in a loop, the caller is responsible to do the same.\n\nNote: format-patch has an `--output-directory` option. Due to the fact\nthat format-patch's options are parsed first, and that the parse-options\nmachinery accepts uniquely abbreviated options, the diff options\n`--output` (and `-o`) are shadowed. Therefore close_file is not set to 1\nso that cmd_format_patch() does *not* need to handle the close_file flag\ndifferently, even if it calls log_tree_commit() in a loop.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/6ea57703f660afd00159b67bf8749d90881df3b4"},{"sha":"4d7b0efc5e2eea1923803dd9c5d5d1288c99cd00","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"log-tree: respect diffopt's configured output file stream\n\nThe diff options already know how to print the output anywhere else\nthan stdout. The same is needed for log output in general, e.g.\nwhen writing patches to files in `git format-patch`. Let's allow\nusers to use log_tree_commit() *without* changing global state via\nfreopen().\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/4d7b0efc5e2eea1923803dd9c5d5d1288c99cd00"},{"sha":"179795e51171ebf10932b553d7df72819fa4a7b5","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"line-log: respect diffopt's configured output file stream\n\nThe diff machinery can optionally output to a file stream other than\nstdout, by overriding diffopt.file. In such a case, the rest of the\nlog tree machinery should also write to that stream.\n\nCurrently, there is no user of the line level log that wants to\nredirect output to a file. Therefore, one might argue that it is\nsuperfluous to support that now. However, it is better to be\nconsistent now, rather than to face hard-to-debug problems later.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/179795e51171ebf10932b553d7df72819fa4a7b5"},{"sha":"c61008fdfb59aff00ec546be6bc6cf3bd8869165","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"graph: respect the diffopt.file setting\n\nWhen the caller overrides diffopt.file (which defaults to stdout),\nthe diff machinery already redirects its output, and the graph display\nshould also write to that file.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/c61008fdfb59aff00ec546be6bc6cf3bd8869165"},{"sha":"0a7b357737e3352ce1b0385313c1efc1b2564dbb","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"shortlog: support outputting to streams other than stdout\n\nThis will be needed to avoid freopen() in `git format-patch`.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/0a7b357737e3352ce1b0385313c1efc1b2564dbb"},{"sha":"11f4eb19843c28b1c50d3c1bef1183e6313b87b8","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"format-patch: explicitly switch off color when writing to files\n\nThe --color=auto handling is done by seeing if file descriptor 1\n(the standard output) is connected to a terminal.  format-patch\nused freopen() to reuse the standard output stream even when sending\nits output to an on-disk file, and this check is appropriate.\n\nIn the next step, however, we will stop reusing \"FILE *stdout\", and\ninstead start using arbitrary file descriptor obtained by doing an\nfopen(3) ourselves.  The check --color=auto does will become useless,\nas we no longer are writing to the standard output stream.\n\nBut then, we do not need to guess to begin with. As argued in the commit\nmessage of 7787570c (format-patch: ignore ui.color, 2011-09-13), we do not\nallow the ui.color setting to affect format-patch's output. The only time,\ntherefore, that we allow color sequences to be written to the output files\nis when the user specified the --color=always command-line option explicitly.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/11f4eb19843c28b1c50d3c1bef1183e6313b87b8"},{"sha":"95235f5ba1a9398c8503b809370ece18bba72ba2","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"format-patch: avoid freopen()\n\nWe just taught the relevant functions to respect the diffopt.file field,\nto allow writing somewhere else than stdout. Let's make use of it.\n\nTechnically, we do not need to avoid that call in a builtin: we assume\nthat builtins (as opposed to library functions) are stand-alone programs\nthat may do with their (global) state. Yet, we want to be able to reuse\nthat code in properly lib-ified code, e.g. when converting scripts into\nbuiltins.\n\nFurther, while we did not *have* to touch the cmd_show() and cmd_cherry()\ncode paths (because they do not want to write anywhere but stdout as of\nyet), it just makes sense to be consistent, making it easier and safer to\nmove the code later.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/95235f5ba1a9398c8503b809370ece18bba72ba2"},{"sha":"36a4d905c36a598d17274765b57690ecfb4a23f8","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"format-patch: use stdout directly\n\nEarlier, we freopen()ed stdout in order to write patches to files.\nThat forced us to duplicate stdout (naming it \"realstdout\") because we\n*still* wanted to be able to report the file names.\n\nAs we do not abuse stdout that way anymore, we no longer need to\nduplicate stdout, either.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/36a4d905c36a598d17274765b57690ecfb4a23f8"},{"sha":"7f7d712bcfde8afe0a007042d5cb4b809617fb96","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"shortlog: respect the --output=<file> setting\n\nThanks to the diff option parsing, we already know about this option.\nWe just have to make use of it.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/7f7d712bcfde8afe0a007042d5cb4b809617fb96"},{"sha":"c1496934cfd6603ac3fbf74cc3a16fcd584d5484","author":{"name":"Johannes Schindelin","email":"c714eff280a95c3e5bbcc9303c4499eebb0d49aa@gmx.de"},"message":"t4211: ensure that log respects --output=<file>\n\nThe test script t4202-log.sh is already pretty long, and it is a good\nidea to test --output with a more obscure option, anyway. So let's\ntest it in conjunction with line-log.\n\nThe most important part of this test, of course, is to ensure that the\nfile is not closed after writing the diff, but only at the very end\nof the log output. That is the entire reason why the test tries to\ngenerate a log that covers more than one commit.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/c1496934cfd6603ac3fbf74cc3a16fcd584d5484"},{"sha":"60452a30f5971c54ecdcd1bd389c1cabbbc844f5","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"grep: break down an \"if\" stmt in preparation for next changes\n\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/60452a30f5971c54ecdcd1bd389c1cabbbc844f5"},{"sha":"949782d860889cbd50edfdf8c00e91d7a4f6cb05","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"test-regex: isolate the bug test code\n\nThis is in preparation to turn test-regex into some generic regex\ntesting command.\n\nHelped-by: Eric Sunshine <sunshine@sunshineco.com>\nHelped-by: Ramsay Jones <ramsay@ramsayjones.plus.com>\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/949782d860889cbd50edfdf8c00e91d7a4f6cb05"},{"sha":"d8acfe1eaf8621025c4095d3fbb88b2703b3fa54","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"test-regex: expose full regcomp() to the command line\n\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/d8acfe1eaf8621025c4095d3fbb88b2703b3fa54"},{"sha":"5c1ebcca4d1df3b2a84d01b3f32ec13bf8f760f8","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"grep/icase: avoid kwsset on literal non-ascii strings\n\nWhen we detect the pattern is just a literal string, we avoid heavy\nregex engine and use fast substring search implemented in kwsset.c.\nBut kws uses git-ctype which is locale-independent so it does not know\nhow to fold case properly outside ascii range. Let regcomp or pcre\ntake care of this case instead. Slower, but accurate.\n\nNoticed-by: Plamen Totev <plamen.totev@abv.bg>\nHelped-by: René Scharfe <l.s.r@web.de>\nHelped-by: Eric Sunshine <sunshine@sunshineco.com>\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/5c1ebcca4d1df3b2a84d01b3f32ec13bf8f760f8"},{"sha":"c22f6202052ca84c68df4fbb16e42c826d429558","author":{"name":"Eric Wong","email":"47aede7528d154728613163cc3055c3c2c82ea2f@80x24.org"},"message":"xread: retry after poll on EAGAIN/EWOULDBLOCK\n\nWe should continue to loop after EAGAIN/EWOULDBLOCK as the\nintent of xread is to try until there is available data,\nEOF, or an unrecoverable error.\n\nFixes: 1079c4be0b720 (\"xread: poll on non blocking fds\")\n\nSigned-off-by: Eric Wong <e@80x24.org>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/c22f6202052ca84c68df4fbb16e42c826d429558"},{"sha":"ef1cf0167a362cd504afdcc0eef4c3200ea6dfbb","author":{"name":"Eric Wong","email":"47aede7528d154728613163cc3055c3c2c82ea2f@80x24.org"},"message":"xwrite: poll on non-blocking FDs\n\nwrite(2) can hit the same EAGAIN/EWOULDBLOCK errors as read(2),\nso busy-looping on a non-blocking FD is a waste of resources.\n\nCurrently, I do not know of a way for this happen:\n\n* the NonBlocking directive in systemd does not apply to stdin,\n  stdout, or stderr.\n\n* xinetd provides no way to set the non-blocking flag at all\n\nBut theoretically, it's possible a careless C10K HTTP server\ncould use pipe2(..., O_NONBLOCK) to setup a pipe for\ngit-http-backend with only the intent to use non-blocking reads;\nbut accidentally leave non-blocking set on the write end passed\nas stdout to git-upload-pack.\n\nFollowup-to: 1079c4be0b720 (\"xread: poll on non blocking fds\")\n\nSigned-off-by: Eric Wong <e@80x24.org>\nReviewed-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/ef1cf0167a362cd504afdcc0eef4c3200ea6dfbb"},{"sha":"a52397cce6c1049c3c9507d734243ee125d11f0c","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"git-fetch.txt: document fetch output\n\nThis documents the ref update status of fetch. The structure of this\noutput is defined in [1]. The ouput content is refined a bit in [2]\n[3] [4].\n\nThis patch is a copy from git-push.txt, modified a bit because the\nflag '-' means different things in push (delete) and fetch (tag\nupdate).\n\nPS. For code archaeologists, the discussion mentioned in [1] is\nprobably [5].\n\n[1] 165f390 (git-fetch: more terse fetch output - 2007-11-03)\n[2] 6315472 (fetch: report local storage errors ... - 2008-06-26)\n[3] f360d84 (builtin-fetch: add --prune option - 2009-11-10)\n[4] 0997ada (fetch: describe new refs based on where... - 2012-04-16)\n[5] http://thread.gmane.org/gmane.comp.version-control.git/61657\n\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/a52397cce6c1049c3c9507d734243ee125d11f0c"},{"sha":"d0b39a03cd5d6d5f9e2eac4907d9f47ec3d821da","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"fetch: refactor ref update status formatting code\n\nThis makes it easier to change the formatting later. And it makes sure\ntranslators cannot mess up format specifiers and break Git.\n\nThere are a couple call sites where the length of the second column is\nTRANSPORT_SUMMARY_WIDTH instead of calculated by TRANSPORT_SUMMARY(),\nwhich is enforced now. The result should be the same because these call\nsites do not contain characters outside ASCII range.\n\nThe two strbuf_addf() calls instead of one is mostly to reduce\ndiff-noise in a future patch where \"ref -> ref\" is reformatted\ndifferently.\n\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/d0b39a03cd5d6d5f9e2eac4907d9f47ec3d821da"},{"sha":"2cb040baa6fcb8a6314a54933cbcb4d3fcb60401","author":{"name":"Nguyễn Thái Ngọc Duy","email":"d0e3b15a74ff6c70e81434a55b9c843230dc3715@gmail.com"},"message":"fetch: change flag code for displaying tag update and deleted ref\n\nThis makes the fetch flag code consistent with push, where '-' means\ndeleted ref.\n\nSigned-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","distinct":false,"url":"https://api.github.com/repos/peff/git/commits/2cb040baa6fcb8a6314a54933cbcb4d3fcb60401"}]}
{"actor":{"display_login":"peff"}}