{"thread":{"id":"17084","subject":"What's cooking in git.git (Jan 2009, #02; Sun, 11)","startedAt":"2009-01-11T09:51:53Z","lastAt":"2009-01-23T12:04:22Z","messageCount":11,"participants":["Junio C Hamano","Alexander Potashev","Jakub Narebski","Marcel Koeppen","Sebastien Cevey"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"99936","messageId":"7v63kmtbk6.fsf@gitster.siamese.dyndns.org","threadId":"17084","inReplyTo":null,"subject":"What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-11T09:51:53Z","receivedAt":"2009-01-11T09:51:53Z","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'.  The ones marked with '.' do not appear in any of the branches,\nbut I am still holding onto them.\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\n----------------------------------------------------------------\n[New Topics]\n\n* rs/fgrep (Sat Jan 10 00:18:34 2009 +0100) 2 commits\n + grep: don't call regexec() for fixed strings\n + grep -w: forward to next possible position after rejected match\n\n* lt/zlib-wrap-xprm (Wed Jan 7 19:54:47 2009 -0800) 1 commit\n - Wrap inflateInit to retry allocation after releasing pack memory\n\nNeed to clean up the log message, perhaps rebase it to maint-1.6.0 and\nstart cooking in 'next'.\n\n* jc/maint-format-patch (Sat Jan 10 12:41:33 2009 -0800) 1 commit\n + format-patch: show patch text for the root commit\n\n* tr/maint-no-index-fixes (Wed Jan 7 12:15:30 2009 +0100) 3 commits\n + diff --no-index -q: fix endless loop\n + diff --no-index: test for pager after option parsing\n + diff: accept -- when using --no-index\n\n* gb/gitweb-opml (Fri Jan 2 13:49:30 2009 +0100) 2 commits\n - gitweb: suggest name for OPML view\n - gitweb: don't use pathinfo for global actions\n\n* mh/maint-commit-color-status (Thu Jan 8 19:53:05 2009 +0100) 2 commits\n - git-status -v: color diff output when color.ui is set\n - git-commit: color status output when color.ui is set\n\n* ks/maint-mailinfo-folded (Thu Jan 8 01:43:42 2009 +0300) 1 commit\n - mailinfo: correctly handle multiline 'Subject:' header\n\n* js/patience-diff (Thu Jan 1 17:39:37 2009 +0100) 3 commits\n - bash completions: Add the --patience option\n - Introduce the diff option '--patience'\n - Implement the patience diff algorithm\n\nAll of the above 'pu' topics are ready for 'next'.\n\n* ap/clone-into-empty (Fri Jan 9 02:24:23 2009 +0300) 2 commits\n - Use is_pseudo_dir_name everywhere\n - Allow cloning to an existing empty directory\n\nThere is an updated patch that only refactors the repeated code to check\nif a dirent is dot or dot-dot posted, which I should have picked up to\nreplace these but I haven't yet (the \"clone into empty\" can and should\nbuild on top of it).\n\n----------------------------------------------------------------\n[Stalled and may need help and prodding to go forward]\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\nThis automatically enables threaded delta search code when autoconf\ndetects pthreads are usable.  I haven't heard neither positive nor\nnegative comments from minority platforms that might be harmed, but\nthis feels like the right thing to do, so perhaps the best course of\naction is to merge this down to 'master' and see if anybody screams.\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\nThis gives Porcelains (like gitweb) the information on the commit _before_\nthe one that the final blame is laid on, which should save them one\nrev-parse to dig further.  The line number in the \"previous\" information\nmay need refining, and sanity checking code for reference counting may\nneed to be resurrected before this can move forward.\n\n----------------------------------------------------------------\n[Actively cooking]\n\n* mv/apply-parse-opt (Fri Jan 9 22:21:36 2009 -0800) 2 commits\n + Resurrect \"git apply --flags -\" to read from the standard input\n + parse-opt: migrate builtin-apply.\n\n* rs/maint-shortlog-foldline (Tue Jan 6 21:41:06 2009 +0100) 1 commit\n + shortlog: handle multi-line subjects like log --pretty=oneline et.\n   al. do\n\n* tr/rebase-root (Fri Jan 2 23:28:29 2009 +0100) 4 commits\n - rebase: update documentation for --root\n - rebase -i: learn to rebase root commit\n - rebase: learn to rebase root commit\n - rebase -i: execute hook only after argument checking\n\nI should be able to find time to read this over again and merge to\n'next' sometime this week.\n\n* as/autocorrect-alias (Sun Jan 4 18:16:01 2009 +0100) 1 commit\n + git.c: make autocorrected aliases work\n\n* js/notes (Sat Dec 20 13:06:03 2008 +0100) 4 commits\n - Add an expensive test for git-notes\n - Speed up git notes lookup\n - Add a script to edit/inspect notes\n - Introduce commit notes\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* gb/gitweb-patch (Thu Dec 18 08:13:19 2008 +0100) 4 commits\n - gitweb: link to patch(es) view in commit(diff) and (short)log view\n - gitweb: add patches view\n - gitweb: change call pattern for git_commitdiff\n - gitweb: add patch view\n\n----------------------------------------------------------------\n[Graduated to \"master\"]\n\n* mh/maint-sendmail-cc-doc (Mon Dec 29 00:37:25 2008 +0100) 1 commit\n + doc/git-send-email: mention sendemail.cc config variable\n\n* rs/diff-ihc (Sun Dec 28 19:45:32 2008 +0100) 1 commit\n + diff: add option to show context between close hunks\n\n* js/maint-merge-recursive-r-d-conflict (Mon Dec 22 23:10:20 2008 +0100) 1 commit\n + merge-recursive: mark rename/delete conflict as unmerged\n\n* mk/gitweb-feature (Mon Dec 15 22:16:19 2008 -0800) 1 commit\n + gitweb: unify boolean feature subroutines\n\n* cb/merge-recursive-fix (Mon Dec 15 02:41:24 2008 -0800) 3 commits\n + Merge branch 'cb/maint-merge-recursive-fix' into cb/merge-\n   recursive-fix\n + merge-recursive: do not clobber untracked working tree garbage\n + modify/delete conflict resolution overwrites untracked file\n\n* cb/maint-merge-recursive-fix (Sun Dec 14 19:40:09 2008 -0800) 2 commits\n + merge-recursive: do not clobber untracked working tree garbage\n + modify/delete conflict resolution overwrites untracked file\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\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* kk/maint-http-push (Tue Dec 23 11:31:15 2008 +0300) 1 commit\n + http-push: support full URI in handle_remote_ls_ctx()\n\n----------------------------------------------------------------\n[Will merge to \"master\" soon]\n\n* nd/grep-assume-unchanged (Sat Dec 27 15:21:03 2008 +0700) 2 commits\n + grep: grep cache entries if they are \"assume unchanged\"\n + grep: support --no-ext-grep to test builtin grep\n\n* as/maint-shortlog-cleanup (Tue Dec 30 22:01:44 2008 +0100) 1 commit\n + builtin-shortlog.c: use string_list_append(), and don't strdup\n   unnecessarily\n\n* jc/maint-ls-tree (Wed Dec 31 19:00:50 2008 +0900) 2 commits\n + Document git-ls-tree --full-tree\n + ls-tree: add --full-tree option\n\n* js/bundle-tags (Fri Jan 2 19:08:46 2009 +0100) 1 commit\n + bundle: allow rev-list options to exclude annotated tags\n\n* js/add-not-submodule (Fri Jan 2 19:08:40 2009 +0100) 1 commit\n + git add: do not add files from a submodule\n\n* pb/maint-git-pm-false-dir (Mon Dec 29 01:25:00 2008 +0100) 1 commit\n + Git.pm: correctly handle directory name that evaluates to \"false\"\n\n* pj/maint-ldflags (Sun Jan 4 21:27:41 2009 -0500) 1 commit\n + configure clobbers LDFLAGS\n\n* fe/cvsserver (Fri Jan 2 16:40:14 2009 +0100) 2 commits\n + cvsserver: change generation of CVS author names\n + cvsserver: add option to configure commit message\n\n* js/maint-bisect-gitk (Fri Jan 2 19:08:00 2009 +0100) 1 commit\n + bisect view: call gitk if Cygwin's SESSIONNAME variable is set\n\n* np/no-loosen-prune-expire-now (Tue Dec 30 14:45:11 2008 -0500) 1 commit\n + objects to be pruned immediately don't have to be loosened\n\n* cb/maint-unpack-trees-absense (Thu Jan 1 21:54:33 2009 +0100) 3 commits\n + unpack-trees: remove redundant path search in verify_absent\n + unpack-trees: fix path search bug in verify_absent\n + unpack-trees: handle failure in verify_absent\n\n* mc/cd-p-pwd (Tue Dec 30 07:10:24 2008 -0800) 1 commit\n + git-sh-setup: Fix scripts whose PWD is a symlink to a work-dir on\n   OS X\n\n* mh/cherry-default (Thu Jan 1 22:56:29 2009 +0100) 2 commits\n + Documentation: clarify which parameters are optional to git-cherry\n + git-cherry: make <upstream> parameter optional\n\n----------------------------------------------------------------\n[Will drop]\n\n* as/commit-signoff (Mon Dec 29 12:16:45 2008 +0100) 1 commit\n - [WIP] Add a commit.signoff configuration option to always use --\n   signoff in commit\n\nThe semantics when \"git commit\" was used as a backend for other actions\nsuch as rebase and cherry-pick was unclear.\n\n----------------------------------------------------------------\n[On Hold]\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":"99952","messageId":"20090111122128.GA16108@myhost","threadId":"17084","inReplyTo":"7v63kmtbk6.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Alexander Potashev","fromEmail":"aspotashev@gmail.com","sentAt":"2009-01-11T12:21:28Z","receivedAt":"2009-01-11T12:21:28Z","isPatch":false,"sender":{"key":"aspotashev@gmail.com","avatar":null},"body":"On 01:51 Sun 11 Jan     , Junio C Hamano wrote:\n> [New Topics]\n> \n> Need to clean up the log message, perhaps rebase it to maint-1.6.0 and\n> start cooking in 'next'.\n> \n> * jc/maint-format-patch (Sat Jan 10 12:41:33 2009 -0800) 1 commit\n>  + format-patch: show patch text for the root commit\n\nMy testcases ([PATCH] Add new testcases for format-patch root commits)\nfor this don't satisfy the target behaviour.\n\n> \n> All of the above 'pu' topics are ready for 'next'.\n> \n> * ap/clone-into-empty (Fri Jan 9 02:24:23 2009 +0300) 2 commits\n>  - Use is_pseudo_dir_name everywhere\n>  - Allow cloning to an existing empty directory\n\nAs far as I understood from your message, you don't think that cloning\ninto empty directories is necessary. So, I thought, the best solution for\nyesterday was \"[PATCH] add is_dot_or_dotdot inline function\" (to make you\nhappy ;)).\n\nBut the workarounds like this:\n\n|    $ git clone -n $there it.git\n|    $ mv it.git/.git . && rmdir it.git && git checkout -f\n\nare painful, especially for newbies who have no idea about anything but\n'git clone'.\n\n> \n> There is an updated patch that only refactors the repeated code to check\n> if a dirent is dot or dot-dot posted, which I should have picked up to\n> replace these but I haven't yet (the \"clone into empty\" can and should\n> build on top of it).\n> \n\n\nBtw, I've sent some worthwhile patches, I but haven't got any reply from you:\n\t[PATCH] use || instead of | in logical expressions\n\t[PATCH] Replace deprecated dashed git commands in usage\n\t[PATCH] remove unnecessary 'if'\nIt's better if you say \"No\" than nothing.\n"},{"id":"99962","messageId":"m3ljthzzdq.fsf@localhost.localdomain","threadId":"17084","inReplyTo":"7v63kmtbk6.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-01-11T14:33:33Z","receivedAt":"2009-01-11T14:33:33Z","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> [Actively cooking]\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\nThis I think needs some further cooking.  I guess with addition of one\nmore patch to series categories could be sorted together with projects\nthey contain, and not always have to be in fixed ordering.\n \n> * gb/gitweb-patch (Thu Dec 18 08:13:19 2008 +0100) 4 commits\n>  - gitweb: link to patch(es) view in commit(diff) and (short)log view\n>  - gitweb: add patches view\n>  - gitweb: change call pattern for git_commitdiff\n>  - gitweb: add patch view\n\nIf I remember correctly the only point of discussion is calling\nconvention for git_commitdiff, and whether 'patches' view should\n(re)use git_commitdiff or use its own subroutine.\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"99999","messageId":"7veiz9siag.fsf@gitster.siamese.dyndns.org","threadId":"17084","inReplyTo":"20090111122128.GA16108@myhost","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-11T20:24:07Z","receivedAt":"2009-01-11T20:24:07Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alexander Potashev <aspotashev@gmail.com> writes:\n\n>> * jc/maint-format-patch (Sat Jan 10 12:41:33 2009 -0800) 1 commit\n>>  + format-patch: show patch text for the root commit\n>\n> My testcases ([PATCH] Add new testcases for format-patch root commits)\n> for this don't satisfy the target behaviour.\n\nI thought I squashed the test case from your original to it and they seem\nto pass for me, but maybe you are talking about some other tests?  If you\nknow of breakages please send in incremental updates.\n\n>> * ap/clone-into-empty (Fri Jan 9 02:24:23 2009 +0300) 2 commits\n>>  - Use is_pseudo_dir_name everywhere\n>>  - Allow cloning to an existing empty directory\n>\n> As far as I understood from your message, you don't think that cloning\n> into empty directories is necessary. So, I thought, the best solution for\n> yesterday was \"[PATCH] add is_dot_or_dotdot inline function\" (to make you\n> happy ;)).\n\nI merely said \"I am not particularly interested in it.\"  That's quite\ndifferent from \"I oppose and reject\".\n\nAs long as the new feature is maintainability-wise low-impact and does not\nhurt users who do _not_ use it, I am not opposed to have a new feature\neven when I see it is only narrowly useful.\n\nIf a topic brings in a large change that helps to support only one\nparticular workflow better, while making it cumbersome to update the\nresulting code to support some other workflow later, even if the change is\nuseful for users of that one particular workflow, I may oppose it.  It\nwould be high-impact from the maintainability point of view [*1*].\n\nBut I do not think your \"clone here\" falls into that category.\n\nIt is really up to you to follow through with it, and people with similar\nneeds to cheer you on.  I thought you took a good strategy to first get\ndot-or-dotdot in (which is generally useful), hoping to bring up the\n\"clone here\" topic again by building on top of it later.\n\n> Btw, I've sent some worthwhile patches, I but haven't got any reply from you:\n> \t[PATCH] use || instead of | in logical expressions\n> \t[PATCH] Replace deprecated dashed git commands in usage\n> \t[PATCH] remove unnecessary 'if'\n> It's better if you say \"No\" than nothing.\n\nI do not recall the last one.\n\nThe first one I thought was a trivial janitor patch that (1) didn't matter\nvery deeply but made things somewhat easier to read, and more importantly\n(2) you had \"oops\" reply to yourself.\n\nI often clean up trivial \"oops\" in a patch that fixes bugs or adds\nfeatures to avoid extra round trip with the contributor, but that is only\nwhen bugfix and enhancements are worthwhile by itself.\n\nThe purpose of a clean-up patch is to clean things up.  If it itself has\n\"oops\" in it, that fails its own criteria of goodness.  Please don't\nexpect/force me to spend time cleaning up \"oops\" in a clean-up patch, but\nsubmit a replacement I can apply straight out of my mailbox.\n\nThe second one I was expecting to hear from people who were involved in\nthe discussion back when we standardized on dashless form to show hands as\nI recall these messages were deliberately left with dashed form for some\nreason (perhaps to help avoiding \"man git foo\" vs \"man git-foo\"\nconfusion).\n\n[Footnote]\n\n*1* Such a change probably needs to be justified either by showing any\nother workflow does not make sense (so supporting that one true workflow\nwell is sufficient) or by demonstrating that support for some other\nequally valid workflows can be included trivially, or both.\n"},{"id":"100014","messageId":"7vwsd1pjst.fsf@gitster.siamese.dyndns.org","threadId":"17084","inReplyTo":"m3ljthzzdq.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-11T22:19:46Z","receivedAt":"2009-01-11T22:19:46Z","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>> ----------------------------------------------------------------\n>> [Actively cooking]\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> This I think needs some further cooking.  I guess with addition of one\n> more patch to series categories could be sorted together with projects\n> they contain, and not always have to be in fixed ordering.\n\nThese should be moved to the Stalled category; nobody seems to be\ndiscussing improvements and sending updates to the series as far as I\nrecall.\n\n>> * gb/gitweb-patch (Thu Dec 18 08:13:19 2008 +0100) 4 commits\n>>  - gitweb: link to patch(es) view in commit(diff) and (short)log view\n>>  - gitweb: add patches view\n>>  - gitweb: change call pattern for git_commitdiff\n>>  - gitweb: add patch view\n>\n> If I remember correctly the only point of discussion is calling\n> convention for git_commitdiff, and whether 'patches' view should\n> (re)use git_commitdiff or use its own subroutine.\n\nThanks; I take it that it is basically usable, useful and can be\nincrementally improved in 'next'?\n"},{"id":"100045","messageId":"200901120225.30175.jnareb@gmail.com","threadId":"17084","inReplyTo":"7vwsd1pjst.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-01-12T01:25:29Z","receivedAt":"2009-01-12T01:25:29Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"On Sun, 11 Jan 2009, Junio C Hamano wrote:\n> Jakub Narebski <jnareb@gmail.com> writes:\n> \n>>> ----------------------------------------------------------------\n>>> [Actively cooking]\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>> This I think needs some further cooking.  I guess with addition of one\n>> more patch to series categories could be sorted together with projects\n>> they contain, and not always have to be in fixed ordering.\n> \n> These should be moved to the Stalled category; nobody seems to be\n> discussing improvements and sending updates to the series as far as I\n> recall.\n\nI think it is just the author being slow moving; there was quite\na bit of time between subsequent versions of this patch series.\nBut if Sebastien would not resend this series in about a week,\nI'll try to clean it up, add fourth patch, and resend it.\n\nAs to lack of discussion: I think it is cause bu two issues. First,\nthere is support for tags already implemented which somewhat reduces\nneed for categories support. Second, hosting sites which have large\nnumber of projects for which categories support might be a nice thing,\nuse I guess modified gitweb with caching, don't they?\n\n>>> * gb/gitweb-patch (Thu Dec 18 08:13:19 2008 +0100) 4 commits\n>>>  - gitweb: link to patch(es) view in commit(diff) and (short)log view\n>>>  - gitweb: add patches view\n>>>  - gitweb: change call pattern for git_commitdiff\n>>>  - gitweb: add patch view\n>>\n>> If I remember correctly the only point of discussion is calling\n>> convention for git_commitdiff, and whether 'patches' view should\n>> (re)use git_commitdiff or use its own subroutine.\n> \n> Thanks; I take it that it is basically usable, useful and can be\n> incrementally improved in 'next'?\n\nYes, I think so. The changes are cosmetic in nature, and I think\nthe feature this patch adds is quite useful: you can now get patches\nand [short] patch series from gitweb which you can apply using git-am.\nNice, isn't it?\n\n-- \nJakub Narebski\nPoland\n"},{"id":"100051","messageId":"7vab9xmhc0.fsf@gitster.siamese.dyndns.org","threadId":"17084","inReplyTo":"200901120225.30175.jnareb@gmail.com","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-12T01:41:19Z","receivedAt":"2009-01-12T01:41:19Z","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> On Sun, 11 Jan 2009, Junio C Hamano wrote:\n>> Jakub Narebski <jnareb@gmail.com> writes:\n>> These should be moved to the Stalled category; nobody seems to be\n>> discussing improvements and sending updates to the series as far as I\n>> recall.\n>\n> I think it is just the author being slow moving; there was quite\n> a bit of time between subsequent versions of this patch series.\n\nOh, being slow is fine and \"Stalled\" is exactly that.\n\n> But if Sebastien would not resend this series in about a week,\n> I'll try to clean it up, add fourth patch, and resend it.\n\nThanks.\n"},{"id":"100055","messageId":"A40F4753-A81B-43FD-B1A5-B28B627F8BBD@marzelpan.de","threadId":"17084","inReplyTo":"7v63kmtbk6.fsf@gitster.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Marcel Koeppen","fromEmail":"lists@marzelpan.de","sentAt":"2009-01-12T01:58:11Z","receivedAt":"2009-01-12T01:58:11Z","isPatch":false,"sender":{"key":"lists@marzelpan.de","avatar":null},"body":"Hi,\n\nAm 11.01.2009 um 10:51 schrieb Junio C Hamano:\n\n> ----------------------------------------------------------------\n> [Will merge to \"master\" soon]\n\n> * mc/cd-p-pwd (Tue Dec 30 07:10:24 2008 -0800) 1 commit\n> + git-sh-setup: Fix scripts whose PWD is a symlink to a work-dir on\n>   OS X\n\nI think this belongs into maint - without it the testsuite fails on OSX.\n\n\n\tMarcel\n"},{"id":"100066","messageId":"7viqolkw7a.fsf@gitster.siamese.dyndns.org","threadId":"17084","inReplyTo":"A40F4753-A81B-43FD-B1A5-B28B627F8BBD@marzelpan.de","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-01-12T04:03:05Z","receivedAt":"2009-01-12T04:03:05Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marcel Koeppen <lists@marzelpan.de> writes:\n\n> Hi,\n>\n> Am 11.01.2009 um 10:51 schrieb Junio C Hamano:\n>\n>> ----------------------------------------------------------------\n>> [Will merge to \"master\" soon]\n>\n>> * mc/cd-p-pwd (Tue Dec 30 07:10:24 2008 -0800) 1 commit\n>> + git-sh-setup: Fix scripts whose PWD is a symlink to a work-dir on\n>>   OS X\n>\n> I think this belongs into maint - without it the testsuite fails on OSX.\n\nOne step at a time.\n\nI did fork the topic at v1.6.1 so that after it proves itself in next and\nthen in master it can go to maint.\n"},{"id":"101386","messageId":"1232548106.4977310a95d8f@mail.nimag.net","threadId":"17084","inReplyTo":"200901120225.30175.jnareb@gmail.com","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Sebastien Cevey","fromEmail":"seb@cine7.net","sentAt":"2009-01-21T14:28:26Z","receivedAt":"2009-01-21T14:28:26Z","isPatch":false,"sender":{"key":"seb@cine7.net","avatar":"https://avatars.githubusercontent.com/u/36964?v=4"},"body":"Selon Jakub Narebski <jnareb@gmail.com>:\n\nHello,\n\nSorry for not responding earlier, I was quite busy being ill and moving abroad\nfor a new job.\n\n> >>> ----------------------------------------------------------------\n> >>> [Actively cooking]\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> >> This I think needs some further cooking.  I guess with addition of one\n> >> more patch to series categories could be sorted together with projects\n> >> they contain, and not always have to be in fixed ordering.\n> > \n> > These should be moved to the Stalled category; nobody seems to be\n> > discussing improvements and sending updates to the series as far as I\n> > recall.\n> \n> I think it is just the author being slow moving; there was quite\n> a bit of time between subsequent versions of this patch series.\n\nI don't recall what was left to do on top of the series of patches I submitted,\ncould you refresh my mind on that if it still needs to be done? I remember the\ndiscussion trailing off as categorized ordering was being discussed..\n\n-- \nSebastien Cevey - inso.cc\n"},{"id":"101651","messageId":"200901231304.23977.jnareb@gmail.com","threadId":"17084","inReplyTo":"1232548106.4977310a95d8f@mail.nimag.net","subject":"Re: What's cooking in git.git (Jan 2009, #02; Sun, 11)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-01-23T12:04:22Z","receivedAt":"2009-01-23T12:04:22Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"On Wed, 21 Jan 09, Sebastien Cevey wrote:\n\n>>>>> ----------------------------------------------------------------\n>>>>> [Actively cooking]\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>>>> This I think needs some further cooking.  I guess with addition of one\n>>>> more patch to series categories could be sorted together with projects\n>>>> they contain, and not always have to be in fixed ordering.\n>>> \n>>> These should be moved to the Stalled category; nobody seems to be\n>>> discussing improvements and sending updates to the series as far as I\n>>> recall.\n>> \n>> I think it is just the author being slow moving; there was quite\n>> a bit of time between subsequent versions of this patch series.\n> \n> I don't recall what was left to do on top of the series of patches I submitted,\n> could you refresh my mind on that if it still needs to be done? I remember the\n> discussion trailing off as categorized ordering was being discussed..\n\nI'd have to take a fresh look at discussion but I remember two things:\nfirst, that the code dealing with filtering out projects (e.g. removing\nforks) is high incompatibile with introducing later limiting number of\nprojects per page, as it currently filters out paths _during printing_.\nSo we might want to have this cleanup before your series (which now\ninclude a bit unnecessary preparation for projects_list view pagination).\n\nSecond, there was IMHO one unnecessary sorting, as with one more commit\nwe can have quite simply categories sorted in order of sorting project\nthey contain, which means that if we sort projects by age (youngest or\nrather most recently changed first) then with one more commit we can\nhave category containing freshest project first.\n\nI'll try to review this series soon, and if you don't have time I'll\nresend them with those minor corrections.\n-- \nJakub Narebski\nPoland\n"}]}