{"thread":{"id":"14452","subject":"[PATCH 0/6] archive: refactor and cleanup","startedAt":"2008-07-14T19:21:50Z","lastAt":"2008-07-15T15:21:54Z","messageCount":5,"participants":["René Scharfe","Lea Wiemann","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":6},"messages":[{"id":"83292","messageId":"487BA74E.5070208@lsrfire.ath.cx","threadId":"14452","inReplyTo":null,"subject":"[PATCH 0/6] archive: refactor and cleanup","fromName":"René Scharfe","fromEmail":"rene.scharfe@lsrfire.ath.cx","sentAt":"2008-07-14T19:21:50Z","receivedAt":"2008-07-14T19:21:50Z","isPatch":true,"sender":{"key":"l.s.r@web.de","avatar":"https://avatars.githubusercontent.com/u/26122331?v=4"},"body":"This series is a collection of cleanups for git archive that I finally need\nto get out, reviewed and (hopefully!) integrated.  It should make adding\nmore format backends easier.\n\nPatch 2 adds a context parameter to read_tree_recursive(), thus affecting\nall its callers.  The rest of the patches change *archive*.[ch], only.\n\nRené\n\n\n\n archive-tar.c             |  115 ++++++++++++---------------------------\n archive-zip.c             |  102 +++++------------------------------\n archive.c                 |  132 ++++++++++++++++++++++++++++++++------------\n archive.h                 |   15 ++----\n builtin-archive.c         |   75 ++++++++++++--------------\n builtin-checkout.c        |    4 +-\n builtin-log.c             |    4 +-\n builtin-ls-tree.c         |    4 +-\n builtin-merge-recursive.c |    4 +-\n builtin-upload-archive.c  |   11 ++--\n tree.c                    |   12 ++--\n tree.h                    |    4 +-\n 12 files changed, 208 insertions(+), 274 deletions(-)\n"},{"id":"83321","messageId":"487BE440.9010006@gmail.com","threadId":"14452","inReplyTo":"487BA74E.5070208@lsrfire.ath.cx","subject":"Re: [PATCH 0/6] archive: refactor and cleanup","fromName":"Lea Wiemann","fromEmail":"lewiemann@gmail.com","sentAt":"2008-07-14T23:41:52Z","receivedAt":"2008-07-14T23:41:52Z","isPatch":true,"sender":{"key":"lewiemann@gmail.com","avatar":null},"body":"René Scharfe wrote:\n> This series is a collection of cleanups for git archive\n\nA few things:\n\n* The note quoted above probably shouldn't be in the commit message but\nafter the \"---\".\n\n* You patch messages seem to be In-Reply-To a message that wasn't posted\non the list, not to PATCH 0/6.\n\n* Unless you have a specific reason, I suggest that you don't Cc Junio\non patches; he reads all messages on this list (more or less) and will\nusually simply apply your patches once they're reviewed.  (Watch for his\nperiodical \"What's cooking in git.git\" messages to see if he got them.)\n\n* And most importantly, your Thunderbird introduced line-breaks, so it's\nnot possible to apply the patches.  May I suggest you use git-send-email\ninstead?\n\nFeel free to ping me on IRC (lea_w in #git) if you need help.\n"},{"id":"83326","messageId":"487BEEED.8030104@gmail.com","threadId":"14452","inReplyTo":"487BE440.9010006@gmail.com","subject":"Re: [PATCH 0/6] archive: refactor and cleanup","fromName":"Lea Wiemann","fromEmail":"lewiemann@gmail.com","sentAt":"2008-07-15T00:27:25Z","receivedAt":"2008-07-15T00:27:25Z","isPatch":true,"sender":{"key":"lewiemann@gmail.com","avatar":null},"body":"Lea Wiemann wrote:\n> * Unless you have a specific reason, I suggest that you don't Cc Junio\n> on patches; he reads all messages on this list (more or less) and will\n> usually simply apply your patches once they're reviewed.  (Watch for his\n> periodical \"What's cooking in git.git\" messages to see if he got them.)\n\nUh, I just noticed that you have enough commits in git.git that you\nprobably didn't need this lecture. ;-)  Sorry, Rene.  (Thanks Dscho for\nthe pointer.)\n"},{"id":"83371","messageId":"487C568F.6030705@lsrfire.ath.cx","threadId":"14452","inReplyTo":"487BE440.9010006@gmail.com","subject":"Re: [PATCH 0/6] archive: refactor and cleanup","fromName":"René Scharfe","fromEmail":"rene.scharfe@lsrfire.ath.cx","sentAt":"2008-07-15T07:49:35Z","receivedAt":"2008-07-15T07:49:35Z","isPatch":true,"sender":{"key":"l.s.r@web.de","avatar":"https://avatars.githubusercontent.com/u/26122331?v=4"},"body":"Lea Wiemann schrieb:\n> René Scharfe wrote:\n>> This series is a collection of cleanups for git archive\n> \n> A few things:\n> \n> * The note quoted above probably shouldn't be in the commit message but\n> after the \"---\".\n\nSince it's in the zeroth message, it's not intended to be committed.\n\n> * You patch messages seem to be In-Reply-To a message that wasn't posted\n> on the list, not to PATCH 0/6.\n\nHmm, that's strange.  Won't happen next time, though.\n\n> * Unless you have a specific reason, I suggest that you don't Cc Junio\n> on patches; he reads all messages on this list (more or less) and will\n> usually simply apply your patches once they're reviewed.  (Watch for his\n> periodical \"What's cooking in git.git\" messages to see if he got them.)\n\nWe're both wrong, quoth Documentation/SubmittingPatches:\n\n    Note that your maintainer does not necessarily read everything\n    on the git mailing list.  If your patch is for discussion first,\n    send it \"To:\" the mailing list, and optionally \"cc:\" him.  If it\n    is trivially correct or after the list reached a consensus, send\n    it \"To:\" the maintainer and optionally \"cc:\" the list.\n\nObviously, my patches weren't trivially correct nor a consensus reached.\n\n> * And most importantly, your Thunderbird introduced line-breaks, so it's\n> not possible to apply the patches.  May I suggest you use git-send-email\n> instead?\n\nWell, it is possible to send usable patches with Thunderbird, using the\nextension Toggle Word Wrap.  Patches 1 and 3 are damaged because I\nedited them again to correct a typo -- with word wrap accidentally on.\n\nTrying git-send-email again is a good idea, though.\n\n> Feel free to ping me on IRC (lea_w in #git) if you need help.\n\nI've slacked long enough to become a complete noob again.  Oh, well.\n\nThanks,\nRené\n"},{"id":"83402","messageId":"7vtzer2mwd.fsf@gitster.siamese.dyndns.org","threadId":"14452","inReplyTo":"487C568F.6030705@lsrfire.ath.cx","subject":"Re: [PATCH 0/6] archive: refactor and cleanup","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-07-15T15:21:54Z","receivedAt":"2008-07-15T15:21:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"René Scharfe <rene.scharfe@lsrfire.ath.cx> writes:\n\n> Lea Wiemann schrieb:\n> ...\n>> Feel free to ping me on IRC (lea_w in #git) if you need help.\n>\n> I've slacked long enough to become a complete noob again.  Oh, well.\n\nLol ;-)\n\nThe series looks good.  Thanks.\n"}]}