{"thread":{"id":"16722","subject":"What's cooking in git.git (Dec 2008, #02; Sun, 14)","startedAt":"2008-12-14T08:24:54Z","lastAt":"2008-12-16T03:36:20Z","messageCount":12,"participants":["Junio C Hamano","Johannes Schindelin","Jakub Narebski","Jeff King","Miklos Vajna","Daniel Barkalow","Nguyen Thai Ngoc Duy"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"97846","messageId":"7v7i632mg9.fsf@gitster.siamese.dyndns.org","threadId":"16722","inReplyTo":null,"subject":"What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-12-14T08:24:54Z","receivedAt":"2008-12-14T08:24:54Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed\nwith '-' are only in 'pu' while commits prefixed with '+' are\nin 'next'.\n\nThe topics list the commits in reverse chronological order.  The topics\nmeant to be merged to the maintenance series have \"maint-\" in their names.\n\nAs we have already passed -rc2, things queued in 'next' let alone 'pu' are\nunlikely to be merged to 'master' by the end of year unless otherwise\nnoted.\n\n----------------------------------------------------------------\n[New Topics]\n\n* wp/add-p-goto (Thu Dec 4 10:22:40 2008 +0000) 2 commits\n + Add 'g' command to go to a hunk\n + Add subroutine to display one-line summary of hunks\n\n* jn/gitweb-blame (Thu Dec 11 01:33:29 2008 +0100) 3 commits\n - gitweb: cache $parent_commit info in git_blame()\n - gitweb: A bit of code cleanup in git_blame()\n - gitweb: Move 'lineno' id from link to row element in git_blame\n\nI've briefly looked at the resurrection of Ajaxy blame that comes on top\nof this series and it looked promising.\n\n* mv/um-pdf (Wed Dec 10 23:44:50 2008 +0100) 1 commit\n - Add support for a pdf version of the user manual\n\nI do not have a new enough combination of dblatex and asciidoc myself but\nthis would help interested people.\n\n* np/auto-thread (Sat Dec 13 15:06:40 2008 -0500) 2 commits\n + pack-objects: don't use too many threads with few objects\n + autodetect number of CPUs by default when using threads\n\n* sc/gitweb-category (Fri Dec 12 00:45:12 2008 +0100) 3 commits\n - gitweb: Optional grouping of projects by category\n - gitweb: Split git_project_list_body in two functions\n - gitweb: Modularized git_get_project_description to be more generic\n\n----------------------------------------------------------------\n[Post 1.6.1 items]\n\n* gb/gitweb-patch (Sat Dec 6 16:02:35 2008 +0100) 3 commits\n - gitweb: link to patch(es) view from commit and log views\n - gitweb: add patches view\n - gitweb: add patch view\n\nUpdated series.  Reviews and Acks?\n\n* lt/reset-merge (Wed Dec 3 18:00:12 2008 -0800) 2 commits\n + Document \"git-reset --merge\"\n + Add 'merge' mode to 'git reset'\n\nWith updated documents.\n\n* wp/add-patch-find (Thu Nov 27 04:08:03 2008 +0000) 3 commits\n . In add --patch, Handle K,k,J,j slightly more gracefully.\n . Add / command in add --patch\n . git-add -i/-p: Change prompt separater from slash to comma\n\nI am still holding onto this earlier topic to add '/' subcommand to allow\nfinding a hunk with given text, but I'd rather not merge/rebase it on top\nof wp/add-p-goto series myself.  Waiting for a reroll.\n\n* cb/mergetool (Fri Dec 12 21:48:41 2008 +0000) 4 commits\n - mergetool: Don't keep temporary merge files unless told to\n - mergetool: Add prompt to continue after failing to merge a file\n - Add -y/--no-prompt option to mergetool\n - Fix some tab/space inconsistencies in git-mergetool.sh\n\nUpdated series.  Waiting for comments from the original author (Ted) and\nother interested parties.\n\n----------------------------------------------------------------\n[Graduated to \"master\"]\n\n* jc/rm-i-t-a (Fri Nov 28 19:56:34 2008 -0800) 5 commits\n + git add --intent-to-add: do not let an empty blob be committed by\n   accident\n + git add --intent-to-add: fix removal of cached emptiness\n + builtin-rm.c: explain and clarify the \"local change\" logic\n + Merge branch 'nd/narrow' (early part) into jc/add-i-t-a\n + Extend index to save more flags\n\nAs intent-to-add index entry is a new feature for 1.6.1, it probably is a\ngood idea to merge this to 'master'.  Nitpicks and bugfixes are very much\nappreciated.\n\n* jn/gitweb-utf8 (Mon Dec 1 19:01:42 2008 +0100) 1 commit\n + gitweb: Fix handling of non-ASCII characters in inserted HTML\n   files.\n\nA bugfix.\n\n----------------------------------------------------------------\n[Will merge to \"master\" soon]\n\nWhat are you looking for?  We are in -rc ;-)\n\n----------------------------------------------------------------\n[On Hold]\n\n* jc/blame (Wed Jun 4 22:58:40 2008 -0700) 2 commits\n + blame: show \"previous\" information in --porcelain/--incremental\n   format\n + git-blame: refactor code to emit \"porcelain format\" output\n\n* ds/uintmax-config (Mon Nov 3 09:14:28 2008 -0900) 1 commit\n - autoconf: Enable threaded delta search when pthreads are supported\n\nRebased to 'master', that introduced NO_PTHREADS.\n\n* cc/bisect-replace (Mon Nov 24 22:20:30 2008 +0100) 9 commits\n - bisect: add \"--no-replace\" option to bisect without using replace\n   refs\n - rev-list: make it possible to disable replacing using \"--no-\n   bisect-replace\"\n - bisect: use \"--bisect-replace\" options when checking merge bases\n - merge-base: add \"--bisect-replace\" option to use fixed up revs\n - commit: add \"bisect_replace_all\" prototype to \"commit.h\"\n - rev-list: add \"--bisect-replace\" to list revisions with fixed up\n   history\n - Documentation: add \"git bisect replace\" documentation\n - bisect: add test cases for \"git bisect replace\"\n - bisect: add \"git bisect replace\" subcommand\n\nI really hate the idea of introducing a potentially much more useful\nreplacement of the existing graft mechanism and tie it very tightly to\nbisect, making it unusable from outside.\n\n (1) I do not think \"bisect replace\" workflow is a practical and usable\n     one;\n\n (2) The underlying mechanism to express \"this object replaces that other\n     object\" is much easier to work with than what the graft does which is\n     \"the parents of this commit are these\", and idea to use the normal\n     ref to point at them means this can potentially be used for\n     transferring the graft information across repositories, which the\n     current graft mechanism cannot do.\n\n (3) Because I like the aspect (2) of this series so much, it deeply\n     disappoints and troubles me that this is implemented minimally near\n     the surface, and that it is controlled by the \"bisect\" Porcelain\n     alone, by explicitly passing command line arguments.\n\nI think a mechanism like this should be added to replace grafts, but it\nshould always be enabled for normal revision traversal operation, while\nalways disabled for object enumeration and transfer operation (iow, fsck,\nfetch and push should use the real ancestry information recorded in the\nunderlying objects, while rev-list, log, etc. should always use the\nreplaced objects).  I have a suspicion that even cat-file could honor it.\n\n* nd/narrow (Sun Nov 30 17:54:38 2008 +0700) 17 commits\n - wt-status: show sparse checkout info\n - Introduce default sparse patterns (core.defaultsparse)\n - checkout: add new options to support sparse checkout\n - clone: support sparse checkout with --sparse-checkout option\n - unpack_trees(): add support for sparse checkout\n - unpack_trees(): keep track of unmerged entries\n - Introduce \"sparse patterns\"\n - Merge branch 'master' into nd/narrow\n + t2104: touch portability fix\n + grep: skip files outside sparse checkout area\n + checkout_entry(): CE_NO_CHECKOUT on checked out entries.\n + Prevent diff machinery from examining worktree outside sparse\n   checkout\n + ls-files: Add tests for --sparse and friends\n + update-index: add --checkout/--no-checkout to update\n   CE_NO_CHECKOUT bit\n + update-index: refactor mark_valid() in preparation for new options\n + ls-files: add options to support sparse checkout\n + Introduce CE_NO_CHECKOUT bit\n\nKicked back to 'on hold' until 1.6.1 final by popular demand; will be\ndropped from 'next' (see recent discussion on the interaction between the\ncheckout area and commands such as \"grep\").\n\n* jc/clone-symref-2 (Sat Nov 29 23:38:21 2008 -0800) 7 commits\n - clone: test the new HEAD detection logic\n - Merge commit 'HEAD@{2}' into HEAD\n - upload-pack: send the HEAD information\n - clone: find the current branch more explicitly\n - connect.c::read_extra_info(): find where HEAD points at\n - connect.c::read_extra_info(): prepare to receive more than server\n   capabilities\n - get_remote_heads(): refactor code to read \"server capabilities\"\n\nThis is no way meant for 1.6.1, let alone next, yet.\n\n* jc/send-pack-tell-me-more (Thu Mar 20 00:44:11 2008 -0700) 1 commit\n - \"git push\": tellme-more protocol extension\n\nThis seems to have a deadlock during communication between the peers.\nSomeone needs to pick up this topic and resolve the deadlock before it can\ncontinue.\n\n* jk/renamelimit (Sat May 3 13:58:42 2008 -0700) 1 commit\n - diff: enable \"too large a rename\" warning when -M/-C is explicitly\n   asked for\n\nThis would be the right thing to do for command line use,\nbut gitk will be hit due to tcl/tk's limitation, so I am holding\nthis back for now.\n\n* jc/stripspace (Sun Mar 9 00:30:35 2008 -0800) 6 commits\n - git-am --forge: add Signed-off-by: line for the author\n - git-am: clean-up Signed-off-by: lines\n - stripspace: add --log-clean option to clean up signed-off-by:\n   lines\n - stripspace: use parse_options()\n - Add \"git am -s\" test\n - git-am: refactor code to add signed-off-by line for the committer\n"},{"id":"97854","messageId":"alpine.DEB.1.00.0812141142300.2014@eeepc-johanness","threadId":"16722","inReplyTo":"7v7i632mg9.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-12-14T10:44:23Z","receivedAt":"2008-12-14T10:44:23Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Sun, 14 Dec 2008, Junio C Hamano wrote:\n\n> * mv/um-pdf (Wed Dec 10 23:44:50 2008 +0100) 1 commit\n>  - Add support for a pdf version of the user manual\n> \n> I do not have a new enough combination of dblatex and asciidoc myself \n> but this would help interested people.\n\nI haven't had a look at the patches, but isn't \"pdflatex\" supposed to be \nthe common way to get .pdf files from LaTeX sources?\n\nCiao,\nDscho\n"},{"id":"97859","messageId":"m3prjvdngm.fsf@localhost.localdomain","threadId":"16722","inReplyTo":"7v7i632mg9.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-12-14T11:08:24Z","receivedAt":"2008-12-14T11:08:24Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> ----------------------------------------------------------------\n> [New Topics]\n> \n> * jn/gitweb-blame (Thu Dec 11 01:33:29 2008 +0100) 3 commits\n>  - gitweb: cache $parent_commit info in git_blame()\n>  - gitweb: A bit of code cleanup in git_blame()\n>  - gitweb: Move 'lineno' id from link to row element in git_blame\n> \n> I've briefly looked at the resurrection of Ajaxy blame that comes on top\n> of this series and it looked promising.\n\nAJAXy blame in my opinion is not yet ready.  Close, but not yet.\nAt minimal I'd have to remove or separate the 'time to generate'\npatch.\n\nI wonder if it should made into 1.6.1 for the performance improvements\nin 'blame' view...\n \n> * sc/gitweb-category (Fri Dec 12 00:45:12 2008 +0100) 3 commits\n>  - gitweb: Optional grouping of projects by category\n>  - gitweb: Split git_project_list_body in two functions\n>  - gitweb: Modularized git_get_project_description to be more generic\n\nIt shapes nicely, and I think with one more commit on top (or with\nextending topmost commit) we can have categories sorted in given\norder, and not hardcoded to always be in alphabetic order.\n\n> ----------------------------------------------------------------\n> [Post 1.6.1 items]\n> \n> * gb/gitweb-patch (Sat Dec 6 16:02:35 2008 +0100) 3 commits\n>  - gitweb: link to patch(es) view from commit and log views\n>  - gitweb: add patches view\n>  - gitweb: add patch view\n> \n> Updated series.  Reviews and Acks?\n\nI'll try to find time to review (and most probably Ack) later today.\n \n> ----------------------------------------------------------------\n> [Graduated to \"master\"]\n>\n> * jn/gitweb-utf8 (Mon Dec 1 19:01:42 2008 +0100) 1 commit\n>  + gitweb: Fix handling of non-ASCII characters in inserted HTML\n>    files.\n> \n> A bugfix.\n\nWhat about\n\n   + gitweb: Fix bug in insert_file() subroutine\n\nIsn't it in the same topic branch?\n\nI am very sorry for buginess of this patch series...\n\n[...] \n> ----------------------------------------------------------------\n> [On Hold]\n> \n> * jc/stripspace (Sun Mar 9 00:30:35 2008 -0800) 6 commits\n>  - git-am --forge: add Signed-off-by: line for the author\n>  - git-am: clean-up Signed-off-by: lines\n>  - stripspace: add --log-clean option to clean up signed-off-by:\n>    lines\n>  - stripspace: use parse_options()\n>  - Add \"git am -s\" test\n>  - git-am: refactor code to add signed-off-by line for the committer\n\nWhat happened to this series to be so stale^Wstalled?\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"97862","messageId":"20081214111548.GB6499@coredump.intra.peff.net","threadId":"16722","inReplyTo":"alpine.DEB.1.00.0812141142300.2014@eeepc-johanness","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2008-12-14T11:15:48Z","receivedAt":"2008-12-14T11:15:48Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Dec 14, 2008 at 11:44:23AM +0100, Johannes Schindelin wrote:\n\n> > * mv/um-pdf (Wed Dec 10 23:44:50 2008 +0100) 1 commit\n> >  - Add support for a pdf version of the user manual\n> > \n> > I do not have a new enough combination of dblatex and asciidoc myself \n> > but this would help interested people.\n> \n> I haven't had a look at the patches, but isn't \"pdflatex\" supposed to be \n> the common way to get .pdf files from LaTeX sources?\n\nBut we don't have latex sources; we have docbook sources. Hence dblatex\n(which does the conversion to PDF internally, so it is a bit of a\nmisnomer).\n\n-Peff\n"},{"id":"97863","messageId":"m3ljujdn2k.fsf@localhost.localdomain","threadId":"16722","inReplyTo":"alpine.DEB.1.00.0812141142300.2014@eeepc-johanness","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2008-12-14T11:17:05Z","receivedAt":"2008-12-14T11:17:05Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n> On Sun, 14 Dec 2008, Junio C Hamano wrote:\n> \n> > * mv/um-pdf (Wed Dec 10 23:44:50 2008 +0100) 1 commit\n> >  - Add support for a pdf version of the user manual\n> > \n> > I do not have a new enough combination of dblatex and asciidoc myself \n> > but this would help interested people.\n> \n> I haven't had a look at the patches, but isn't \"pdflatex\" supposed to be \n> the common way to get .pdf files from LaTeX sources?\n\nThe problem is that AsciiDoc, from what I understand, by itself\ngenerates only XML-ish formats, namely HTML, XHTML and DocBook, and\nthe rest of formats are generated in postprocessing step by extra\ntools from DocBook format:\n * manpages require xmlto\n * info requires docbook2X and makeinfo\n * PDF requires dblatex (db = DocBook), or FOP and xmlto\n\nSo the answer is that AsciiDoc does not generate LaTeX, so pdflatex\nwould be not enough.\n\nBy the way, from the AsciiDoc page:\n\n NOTE: Owning to to other commitments, Benjamin is unable to maintain\n       this backend. I don't have the expertise or time to take this\n       on consequently the LaTeX backend has not been tested or\n       updated since AsciiDoc version 8.2.7 and is currently\n       unsupported.\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"97865","messageId":"alpine.DEB.1.00.0812141224030.2014@eeepc-johanness","threadId":"16722","inReplyTo":"20081214111548.GB6499@coredump.intra.peff.net","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2008-12-14T11:24:17Z","receivedAt":"2008-12-14T11:24:17Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Sun, 14 Dec 2008, Jeff King wrote:\n\n> On Sun, Dec 14, 2008 at 11:44:23AM +0100, Johannes Schindelin wrote:\n> \n> > > * mv/um-pdf (Wed Dec 10 23:44:50 2008 +0100) 1 commit\n> > >  - Add support for a pdf version of the user manual\n> > > \n> > > I do not have a new enough combination of dblatex and asciidoc myself \n> > > but this would help interested people.\n> > \n> > I haven't had a look at the patches, but isn't \"pdflatex\" supposed to be \n> > the common way to get .pdf files from LaTeX sources?\n> \n> But we don't have latex sources; we have docbook sources. Hence dblatex\n> (which does the conversion to PDF internally, so it is a bit of a\n> misnomer).\n\nIndeed, I was fooled.\n\nThanks for the clarification,\nDscho\n"},{"id":"97874","messageId":"20081214131508.GA5691@genesis.frugalware.org","threadId":"16722","inReplyTo":"m3ljujdn2k.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Miklos Vajna","fromEmail":"vmiklos@frugalware.org","sentAt":"2008-12-14T13:15:08Z","receivedAt":"2008-12-14T13:15:08Z","isPatch":false,"sender":{"key":"vmiklos@frugalware.org","avatar":"https://gravatar.com/avatar/401c1cbbb3a5d13e650c691a2c71d6fd0b80df1a01bc74d9f1972675dd58f2bd?d=mp&s=160"},"body":"On Sun, Dec 14, 2008 at 03:17:05AM -0800, Jakub Narebski <jnareb@gmail.com> wrote:\n> So the answer is that AsciiDoc does not generate LaTeX, so pdflatex\n> would be not enough.\n> \n> By the way, from the AsciiDoc page:\n> \n>  NOTE: Owning to to other commitments, Benjamin is unable to maintain\n>        this backend. I don't have the expertise or time to take this\n>        on consequently the LaTeX backend has not been tested or\n>        updated since AsciiDoc version 8.2.7 and is currently\n>        unsupported.\n\nYes, this is the LaTeX backend, which would allow not to use dblatex,\nbut generate tex directly from asciidoc. That's why my patch uses\ndblatex instead.\n"},{"id":"97926","messageId":"7vabayynda.fsf@gitster.siamese.dyndns.org","threadId":"16722","inReplyTo":"m3prjvdngm.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-12-15T06:16:17Z","receivedAt":"2008-12-15T06:16:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jakub Narebski <jnareb@gmail.com> writes:\n\n>> [New Topics]\n>> \n>> * jn/gitweb-blame (Thu Dec 11 01:33:29 2008 +0100) 3 commits\n>>  - gitweb: cache $parent_commit info in git_blame()\n>>  - gitweb: A bit of code cleanup in git_blame()\n>>  - gitweb: Move 'lineno' id from link to row element in git_blame\n>\n> I wonder if it should made into 1.6.1 for the performance improvements\n> in 'blame' view...\n\nDoes is addresses a grave bug?  I thought not.\n\n>> [Post 1.6.1 items]\n>> \n>> * gb/gitweb-patch (Sat Dec 6 16:02:35 2008 +0100) 3 commits\n>> Updated series.  Reviews and Acks?\n>\n> I'll try to find time to review (and most probably Ack) later today.\n\nThanks.\n\n>> * jn/gitweb-utf8 (Mon Dec 1 19:01:42 2008 +0100) 1 commit\n>>  + gitweb: Fix handling of non-ASCII characters in inserted HTML\n>>    files.\n>> \n>> A bugfix.\n>\n> What about\n>\n>    + gitweb: Fix bug in insert_file() subroutine\n\nIt already is part of 'master' for some time, isn't it?.\n\n>> * jc/stripspace (Sun Mar 9 00:30:35 2008 -0800) 6 commits\n> ...\n> What happened to this series to be so stale^Wstalled?\n\nMostly lack of positive feedbacks which is the same thing as \"nobody but\nthe author is interested'.\n"},{"id":"97997","messageId":"alpine.LNX.1.00.0812151423390.19665@iabervon.org","threadId":"16722","inReplyTo":"7v7i632mg9.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-12-15T19:59:19Z","receivedAt":"2008-12-15T19:59:19Z","isPatch":false,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"On Sun, 14 Dec 2008, Junio C Hamano wrote:\n\n> [On Hold]\n> \n> * nd/narrow (Sun Nov 30 17:54:38 2008 +0700) 17 commits\n>  - wt-status: show sparse checkout info\n>  - Introduce default sparse patterns (core.defaultsparse)\n>  - checkout: add new options to support sparse checkout\n>  - clone: support sparse checkout with --sparse-checkout option\n>  - unpack_trees(): add support for sparse checkout\n>  - unpack_trees(): keep track of unmerged entries\n>  - Introduce \"sparse patterns\"\n>  - Merge branch 'master' into nd/narrow\n>  + t2104: touch portability fix\n>  + grep: skip files outside sparse checkout area\n>  + checkout_entry(): CE_NO_CHECKOUT on checked out entries.\n>  + Prevent diff machinery from examining worktree outside sparse\n>    checkout\n\nI think this patch (slightly reframed) would be good to have independantly \nof the series; it also means that we won't do weird things if someone \nchanges a file in the worktree which is marked as CE_VALID, and may well \nmay CE_VALID sufficient for the index representation of paths outside the \ncheckout.\n\n\t-Daniel\n*This .sig left intentionally blank*\n"},{"id":"98024","messageId":"fcaeb9bf0812151735g37f14022o8176a342f9b0d9f1@mail.gmail.com","threadId":"16722","inReplyTo":"alpine.LNX.1.00.0812151423390.19665@iabervon.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Nguyen Thai Ngoc Duy","fromEmail":"pclouds@gmail.com","sentAt":"2008-12-16T01:35:02Z","receivedAt":"2008-12-16T01:35:02Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On 12/16/08, Daniel Barkalow <barkalow@iabervon.org> wrote:\n>  >  + grep: skip files outside sparse checkout area\n>  >  + Prevent diff machinery from examining worktree outside sparse\n>  >    checkout\n>\n>\n> I think this patch (slightly reframed) would be good to have independantly\n>  of the series; it also means that we won't do weird things if someone\n>  changes a file in the worktree which is marked as CE_VALID, and may well\n>  may CE_VALID sufficient for the index representation of paths outside the\n>  checkout.\n\nI have made a fixup patch for CE_VALID. Unfortunately I have not had\ntime to write tests and finish it before submitting. I may have free\ntime this weekend to complete.\n-- \nDuy\n"},{"id":"98026","messageId":"7vr648uc88.fsf@gitster.siamese.dyndns.org","threadId":"16722","inReplyTo":"alpine.LNX.1.00.0812151423390.19665@iabervon.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-12-16T01:42:47Z","receivedAt":"2008-12-16T01:42:47Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Daniel Barkalow <barkalow@iabervon.org> writes:\n\n> On Sun, 14 Dec 2008, Junio C Hamano wrote:\n>\n>> [On Hold]\n>> \n>> * nd/narrow (Sun Nov 30 17:54:38 2008 +0700) 17 commits\n>>  - wt-status: show sparse checkout info\n>>  - Introduce default sparse patterns (core.defaultsparse)\n>>  - checkout: add new options to support sparse checkout\n>>  - clone: support sparse checkout with --sparse-checkout option\n>>  - unpack_trees(): add support for sparse checkout\n>>  - unpack_trees(): keep track of unmerged entries\n>>  - Introduce \"sparse patterns\"\n>>  - Merge branch 'master' into nd/narrow\n>>  + t2104: touch portability fix\n>>  + grep: skip files outside sparse checkout area\n>>  + checkout_entry(): CE_NO_CHECKOUT on checked out entries.\n>>  + Prevent diff machinery from examining worktree outside sparse\n>>    checkout\n>\n> I think this patch (slightly reframed) would be good to have independantly \n> of the series; it also means that we won't do weird things if someone \n> changes a file in the worktree which is marked as CE_VALID, and may well \n> may CE_VALID sufficient for the index representation of paths outside the \n> checkout.\n\nIt is a bit unclear which one you meant by \"this patch\", sorry.\n"},{"id":"98033","messageId":"alpine.LNX.1.00.0812152211310.19665@iabervon.org","threadId":"16722","inReplyTo":"7vr648uc88.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Dec 2008, #02; Sun, 14)","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-12-16T03:36:20Z","receivedAt":"2008-12-16T03:36:20Z","isPatch":false,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"On Mon, 15 Dec 2008, Junio C Hamano wrote:\n\n> Daniel Barkalow <barkalow@iabervon.org> writes:\n> \n> > On Sun, 14 Dec 2008, Junio C Hamano wrote:\n> >\n> >> [On Hold]\n> >> \n> >> * nd/narrow (Sun Nov 30 17:54:38 2008 +0700) 17 commits\n> >>  - wt-status: show sparse checkout info\n> >>  - Introduce default sparse patterns (core.defaultsparse)\n> >>  - checkout: add new options to support sparse checkout\n> >>  - clone: support sparse checkout with --sparse-checkout option\n> >>  - unpack_trees(): add support for sparse checkout\n> >>  - unpack_trees(): keep track of unmerged entries\n> >>  - Introduce \"sparse patterns\"\n> >>  - Merge branch 'master' into nd/narrow\n> >>  + t2104: touch portability fix\n> >>  + grep: skip files outside sparse checkout area\n> >>  + checkout_entry(): CE_NO_CHECKOUT on checked out entries.\n> >>  + Prevent diff machinery from examining worktree outside sparse\n> >>    checkout\n> >\n> > I think this patch (slightly reframed) would be good to have independantly \n> > of the series; it also means that we won't do weird things if someone \n> > changes a file in the worktree which is marked as CE_VALID, and may well \n> > may CE_VALID sufficient for the index representation of paths outside the \n> > checkout.\n> \n> It is a bit unclear which one you meant by \"this patch\", sorry.\n\n\"Prevent diff machinery from examining worktree outside sparse checkout\", \nwhich (possibly in the next version) will prevent the diff machinery from \nexamining worktree files for paths that are CE_VALID in the index.\n\n\t-Daniel\n*This .sig left intentionally blank*\n"}]}