GitHub PushEvent at 2014-09-13 05:49:28
Repository: git
Actor: peff
Payload: {"shas":[["aecf567cbfb6ab46e82f7f5df36fb6a2dd5bee69","a13b1135e6f2f8530e85bc2896cd5ed89a6ed863@twopensource.com","cache-tree: create/update cache-tree on checkout\n\nWhen git checkout checks out a branch, create or update the\ncache-tree so that subsequent operations are faster.\n\nupdate_main_cache_tree learned a new flag, WRITE_TREE_REPAIR.  When\nWRITE_TREE_REPAIR is set, portions of the cache-tree which do not\ncorrespond to existing tree objects are invalidated (and portions which\ndo are marked as valid).  No new tree objects are created.\n\nSigned-off-by: David Turner <dturner@twitter.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Turner",false],["969dd8c612b3b22c4080ffd4448cef4accbf428b","a13b1135e6f2f8530e85bc2896cd5ed89a6ed863@twopensource.com","test-dump-cache-tree: invalid trees are not errors\n\nDo not treat known-invalid trees as errors even when their subtree_nr is\nincorrect.  Because git already knows that these trees are invalid,\nan incorrect subtree_nr will not cause problems.\n\nAdd a couple of comments.\n\nSigned-off-by: David Turner <dturner@twitter.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Turner",false],["59a8adb6fb4ed27868e7144f4531b1bfd065a9c6","a13b1135e6f2f8530e85bc2896cd5ed89a6ed863@twopensource.com","cache-tree: subdirectory tests\n\nAdd tests to confirm that invalidation of subdirectories neither over-\nnor under-invalidates.\n\nSigned-off-by: David Turner <dturner@twitter.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Turner",false],["9c4d6c0297b7d6167bf330853139b0515b351b89","a13b1135e6f2f8530e85bc2896cd5ed89a6ed863@twopensource.com","cache-tree: Write updated cache-tree after commit\n\nDuring the commit process, update the cache-tree. Write this updated\ncache-tree so that it's ready for subsequent commands.\n\nAdd test code which demonstrates that git commit now writes the cache\ntree.  Make all tests test the entire cache-tree, not just the root\nlevel.\n\nSigned-off-by: David Turner <dturner@twitter.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","David Turner",false],["f57a8715bc5f50234dd1f756b3b26a0b81c3784c","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","test prerequisites: eradicate NOT_FOO\n\nSupport for Back when bdccd3c1 (test-lib: allow negation of\nprerequisites, 2012-11-14) introduced negated predicates\n(e.g. \"!MINGW,!CYGWIN\"), we already had 5 test files that use\nNOT_MINGW (and a few MINGW) as prerequisites.\n\nLet's not add NOT_FOO and rewrite existing ones as !FOO for both\nMINGW and CYGWIN.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["b0562224c93aca5addd994e37cd851b7a5de44b8","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","test prerequisites: enumerate with commas\n\ntest_have_prereq does understand multiple predicates given as\nseparate arguments, but that is by accident.  We should list the\nprerequisites just like we use them as the (first) optional\nparameter for test_expect_success, concatenated with commas, for\nconsistency.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["f07243fe1644552ef88a05cc8bb22c9757639065","348a3c4a81ea6081e50dd1888d577fda55d8e4f6@dotat.at","imap-send: clarify CRAM-MD5 vs LOGIN documentation\n\nExplicitly mention that leaving imap.authMethod unset makes\ngit imap-send use the basic IMAP plaintext LOGIN command.\n\nSigned-off-by: Tony Finch <dot@dotat.at>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tony Finch",false],["a789ca70e7a5b02973b116d21674acd795238f99","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","config: teach \"git -c\" to recognize an empty string\n\nIn a config file, you can do:\n\n  [foo]\n  bar\n\nto turn the \"foo.bar\" boolean flag on, and you can do:\n\n  [foo]\n  bar=\n\nto set \"foo.bar\" to the empty string. However, git's \"-c\"\nparameter treats both:\n\n  git -c foo.bar\n\nand\n\n  git -c foo.bar=\n\nas the boolean flag, and there is no way to set a variable\nto the empty string. This patch enables the latter form to\ndo that.\n\nSigned-off-by: Jeff King <peff@peff.net>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["a127b3f24d3a4f97fe53a3495d615a29b7ab8737","738bdd359be778fee9f0fc4e2934ad72f436ceda@crustytoothpaste.net","imap-send doc: omit confusing \"to use imap-send\" modifier\n\nIt wouldn't make sense for these configuration variables to be\nrequired for Git in general to function.  'Required' in this context\nmeans required for git imap-send to work.\n\nNoticed while trying to figure out what the sentence describing\nimap.tunnel meant.\n\n[jn: expanded to also simplify explanation of imap.folder and\n imap.host in the same way]\n\nSigned-off-by: brian m. carlson <sandals@crustytoothpaste.net>\nSigned-off-by: Jonathan Nieder <jrnieder@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","brian m. carlson",false],["8262aaa2835d71b0cdf44e68712703a452761328","d4d2cc2a6ec7b2aa587d4887cc5ed200f9b685a1@imag.fr","config.c: mark error and warnings strings for translation\n\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Matthieu Moy",false],["b3b3f60bb672d23b9db1582395a1d29561cb79ef","d4d2cc2a6ec7b2aa587d4887cc5ed200f9b685a1@imag.fr","config.c: fix accuracy of line number in errors\n\nIf a callback returns a negative value to `git_config*()` family,\nthey call `die()` while printing the line number and the file name.\nCurrently the printed line number is off by one, thus printing the\nwrong line number.\n\nMake `linenr` point to the line we just parsed during the call\nto callback to get accurate line number in error messages.\n\nCommit-message-by: Tanay Abhra <tanayabh@gmail.com>\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nSigned-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Matthieu Moy",false],["3df8fd625fba33a6525f61c85de39afb746db9bd","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","add line number and file name info to `config_set`\n\nStore file name and line number for each key-value pair in the cache\nduring parsing of the configuration files.\n\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["aace4385027b0366f43961a94c8ed95ac9b3bd53","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","change `git_config()` return value to void\n\nCurrently `git_config()` returns an integer signifying an error code.\nDuring rewrites of the function most of the code was shifted to\n`git_config_with_options()`. `git_config_with_options()` normally\nreturns positive values if its `config_source` parameter is set as NULL,\nas most errors are fatal, and non-fatal potential errors are guarded\nby \"if\" statements that are entered only when no error is possible.\n\nStill a negative value can be returned in case of race condition between\n`access_or_die()` & `git_config_from_file()`. Also, all callers of\n`git_config()` ignore the return value except for one case in branch.c.\n\nChange `git_config()` return value to void and make it die if it receives\na negative value from `git_config_with_options()`.\n\nOriginal-patch-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["5a80e97c827e9d73884dbe4119bf97f6dd84b237","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","config: add `git_die_config()` to the config-set API\n\nAdd `git_die_config` that dies printing the line number and the file name\nof the highest priority value for the configuration variable `key`. A custom\nerror message is also printed before dying, specified by the caller, which can\nbe skipped if `err` argument is set to NULL.\n\nIt has usage in non-callback based config value retrieval where we can\nraise an error and die if there is a semantic error.\nFor example,\n\n\tif (!git_config_get_value(key, &value)){\n\t\tif (!strcmp(value, \"foo\"))\n\t\t\tgit_config_die(key, \"value: `%s` is illegal\", value);\n\t\telse\n\t\t\t/* do work */\n\t}\n\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["155ef25f12329258c06b8875b5e372abec2d348d","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","rewrite git_config() to use the config-set API\n\nOf all the functions in `git_config*()` family, `git_config()` has the\nmost invocations in the whole code base. Each `git_config()` invocation\ncauses config file rereads which can be avoided using the config-set API.\n\nUse the config-set API to rewrite `git_config()` to use the config caching\nlayer to avoid config file rereads on each invocation during a git process\nlifetime. First invocation constructs the cache, and after that for each\nsuccessive invocation, `git_config()` feeds values from the config cache\ninstead of rereading the configuration files.\n\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["79e9ce21fa728edef5b7db12710ae24e091b8f9f","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","add a test for semantic errors in config files\n\nSemantic errors (for example, for alias.* variables NULL values are\nnot allowed) in configuration files cause a die printing the line\nnumber and file name of the offending value.\n\nAdd a test documenting that such errors cause a die printing the\naccurate line number and file name.\n\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["8a7b034d6d451491dbcfaebc3d4ed4f08c756822","43a0dd602247961f0b6e66a197d8f2daec3be19d@gmail.com","add tests for `git_config_get_string_const()`\n\nAdd tests for `git_config_get_string_const()`, check whether it\ndies printing the line number and the file name if a NULL\nvalue is retrieved for the given key.\n\nSigned-off-by: Tanay Abhra <tanayabh@gmail.com>\nReviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Tanay Abhra",false],["d487b0ba50884fb1fd1767ac8c9adc58066ad999","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","apply: use the right attribute for paths in non-Git patches\n\nWe parse each patchfile and find the name of the path the patch\napplies to, and then use that name to consult the attribute system\nto find the whitespace rules to be used, and also the target file\n(either in the working tree or in the index) to replay the changes\nagainst.\n\nUnlike a Git-generated patch, a non-Git patch is taken to have the\npathnames relative to the current working directory.  The names\nfound in such a patch are modified by prepending the prefix by the\nprefix_patches() helper function introduced in 56185f49 (git-apply:\nrequire -p<n> when working in a subdirectory., 2007-02-19).\n\nHowever, this prefixing is done after the patch is fully parsed and\naffects only what target files are patched.  Because the attributes\nare checked against the names found in the patch during the parsing,\nnot against the final pathname, the whitespace check that is done\nduring parsing ends up using attributes for a wrong path for non-Git\npatches.\n\nFix this by doing the prefix much earlier, immediately after the\nheader part of each patch is parsed and we learn the name of the\npath the patch affects.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["3ee2ad14c6a6823d16eeda2aa48496d4b6e16f70","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","apply: hoist use_patch() helper for path exclusion up\n\nWe will be adding a caller to the function a bit earlier in this\nfile in a later patch.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false],["477a08af04c227064860ce99197c501037f7f39c","a6723cc3f76163bf7adb636a73ac3b0ceb3e6b9b@pobox.com","apply: omit ws check for excluded paths\n\nWhitespace breakages are checked while the patch is being parsed.\nDisable them at the beginning of parse_chunk(), where each\nindividual patch is parsed, immediately after we learn the name of\nthe file the patch applies to and before we start parsing the diff\ncontained in the patch.\n\nOne may naively think that we should be able to not just skip the\nwhitespace checks but simply fast-forward to the next patch without\ndoing anything once use_patch() tells us that this patch is not\ngoing to be used.  But in reality we cannot really skip much of the\nparsing in order to do such a \"fast-forward\", primarily because\nparsing \"@@ -k,l +m,n @@\" lines and counting the input lines is how\nwe determine the boundaries of individual patches.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>","Junio C Hamano",false]],"size":172,"ref":"refs/heads/jk/rename-progress","head":"b171a1cecf1433e2fdcfe360add8232dce2ef0e7"}