{"thread":{"id":"21056","subject":"Interim maintainer tree","startedAt":"2009-09-25T16:05:04Z","lastAt":"2009-10-05T15:05:47Z","messageCount":8,"participants":["Shawn O. Pearce","Jeff King","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"123787","messageId":"20090925160504.GW14660@spearce.org","threadId":"21056","inReplyTo":null,"subject":"Interim maintainer tree","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2009-09-25T16:05:04Z","receivedAt":"2009-09-25T16:05:04Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio is on vaction for the next week.  In his absence Peff and I\nare trying to keep up with current patches in my fork:\n\n  git://repo.or.cz/git/spearce.git\n  http://repo.or.cz/r/git/spearce.git\n\n\nRight now the tree matches Junio's last push, I'll try to pick up\nthe patches since then and push later today.\n\n-- \nShawn.\n"},{"id":"123894","messageId":"20090927085437.GA10828@coredump.intra.peff.net","threadId":"21056","inReplyTo":"20090925160504.GW14660@spearce.org","subject":"Re: Interim maintainer tree","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-09-27T08:54:37Z","receivedAt":"2009-09-27T08:54:37Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Sep 25, 2009 at 09:05:04AM -0700, Shawn O. Pearce wrote:\n\n> Junio is on vaction for the next week.  In his absence Peff and I\n> are trying to keep up with current patches in my fork:\n> \n>   git://repo.or.cz/git/spearce.git\n>   http://repo.or.cz/r/git/spearce.git\n\nI've collected a few more topics in my repo:\n\n  git://github.com/peff/git.git\n\nRather than cook my own, slightly different 'next', I'm just publishing\nthe tips of the topics themselves. Which means they are subject to\nrewinding until you merge them into 'next', at which point I will leave\nthem stable.\n\nWhat's in there now is:\n\n  as/parseopt-ambiguous\n    Fix for \"git branch --no-merge\" with my test added. Should be ready\n    for next.\n\n  jk/reflog-date\n    My reflog fix. Should be ready for next.\n\n  mr/gitweb-snapshot\n    v5, replacing what Junio had in pu. Should probably stay in pu until\n    we hear from Jakub.\n\n  mr/instaweb\n    Looked OK to me, but I have no capacity to actually test it.\n    Unlikely to break anything else, though, so probably ready to cook\n    in next.\n\n  mv/lock-error\n    My tweaked version, which got the OK from Miklos. Ready for next, I\n    think.\n\n  np/clone-smaller\n    Nico's fix to stop copying unnecessary objects during clone. Patch\n    looks sane, I confirmed that it fixes the issue, and it doesn't seem\n    to break normal clones. Ready to cook in next, I think.\n\n-Peff\n"},{"id":"124197","messageId":"7viqevu1zt.fsf@alter.siamese.dyndns.org","threadId":"21056","inReplyTo":"20090925160504.GW14660@spearce.org","subject":"Re: Interim maintainer tree","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-04T06:06:30Z","receivedAt":"2009-10-04T06:06:30Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Shawn O. Pearce\" <spearce@spearce.org> writes:\n\n> Junio is on vaction for the next week.  In his absence Peff and I\n> are trying to keep up with current patches in my fork:\n>\n>   git://repo.or.cz/git/spearce.git\n>   http://repo.or.cz/r/git/spearce.git\n>\n> Right now the tree matches Junio's last push, I'll try to pick up\n> the patches since then and push later today.\n\nThanks, both.\n\nI've fetched, but haven't fully examined \"log ..spearce/*\" nor \"log ..peff/*\"\nyet.\n\nI noticed that some topics in 'pu' have been rebased (not complaining, but\njust making sure I am not hallucinating).\n\nDo you have preferences/suggestions as to how to proceed?  Should I just\ntake the tips over at this point, or do you have some more patches you\nwere polishing but haven't pushed out that I should wait for?\n"},{"id":"124198","messageId":"20091004064129.GB7076@coredump.intra.peff.net","threadId":"21056","inReplyTo":"7viqevu1zt.fsf@alter.siamese.dyndns.org","subject":"Re: Interim maintainer tree","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-10-04T06:41:29Z","receivedAt":"2009-10-04T06:41:29Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sat, Oct 03, 2009 at 11:06:30PM -0700, Junio C Hamano wrote:\n\n> Thanks, both.\n> \n> I've fetched, but haven't fully examined \"log ..spearce/*\" nor \"log ..peff/*\"\n> yet.\n> \n> I noticed that some topics in 'pu' have been rebased (not complaining, but\n> just making sure I am not hallucinating).\n> \n> Do you have preferences/suggestions as to how to proceed?  Should I just\n> take the tips over at this point, or do you have some more patches you\n> were polishing but haven't pushed out that I should wait for?\n\nFor my stuff, I think you can just take over my tips. I was just\ncollecting and polishing topics, while Shawn was doing integration. For\ntopics I have, please use my versions rather than applying from the\nlist; many of them have extra fixes squashed in, acked-by's collected\nfrom the list, etc.\n\nHere's a brief status report on what's in my tree:\n\n  proposed-master\n    One-off patches that I think should go straight to master.\n\n  ef/mscv-noreturn\n    This is the latest round and I think should be ready for at least\n    'next' (maybe even 'master' as it is really about the build and not\n    about functionality).\n\n  ef/msys-imap\n    This is from an RFC which has generated some comments. He should be\n    posting another round soon. 'pu' at best.\n\n  fc/mutt-alias\n    Latest round that addressed comments. Ready for 'next' if not\n    'master'.\n\n  jn/gitweb-patch\n    After some comments with Jakub, I think the code is right but he\n    promised a re-roll with more in the commit message.\n\n  mr/gitweb-snapshot\n    This is probably the pu rewind you saw. He posted a v5 of his\n    series. I didn't look at it closely, but Jakub ack'd it.\n\n  tf/doc-pt-br\n    Minor translation update, ack'd by somebody who can read it. :)\n    Ready for 'master'.\n\nThere are a few patches floating on the list that I haven't picked up or\nlooked too closely at yet. Just so we don't drop anything, they are:\n\n  - curl http auth tweak:\n    http://article.gmane.org/gmane.comp.version-control.git/129455\n    The author said he didn't really test it, and I haven't set up http\n    auth to test it with, but probably somebody should do so before\n    applying. :)\n\n  - a new rev-cache from Nick\n\n  - silence gcc warning:\n    http://article.gmane.org/gmane.comp.version-control.git/129485\n    The warning is overly cautious, I think, but it is a dubious enough\n    construct that it is probably worth fixing.\n\n  - enable openssl on msvc\n    http://article.gmane.org/gmane.comp.version-control.msysgit/7238\n    This goes on top of ef/msys-imap, but I think that will be getting a\n    re-roll.\n\n-Peff\n"},{"id":"299124","messageId":"7viqevpjr3.fsf@alter.siamese.dyndns.org","threadId":"21056","inReplyTo":"20091004064129.GB7076@coredump.intra.peff.net","subject":"Re: Interim maintainer tree","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-04T09:54:08Z","receivedAt":"2009-10-04T09:54:08Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n> For my stuff, I think you can just take over my tips. I was just\n> collecting and polishing topics, while Shawn was doing integration. For\n> topics I have, please use my versions rather than applying from the\n> list; many of them have extra fixes squashed in, acked-by's collected\n> from the list, etc.\n\nMany topics Shawn merged to his 'pu' were identical to what you have,\nexcept for mr/gitweb-snapshot.  The patch-ids in the series are the same\nbut you have Jakub's Ack added, so I'll pick yours up and rebuild.\n\n> Here's a brief status report on what's in my tree:\n>\n>   proposed-master\n>     One-off patches that I think should go straight to master.\n\nI briefly looked at them after looking at what Shawn queued to his\n'master'; they all looked sane.\n\n>   ef/mscv-noreturn\n>     This is the latest round and I think should be ready for at least\n>     'next' (maybe even 'master' as it is really about the build and not\n>     about functionality).\n>\n>   ef/msys-imap\n>     This is from an RFC which has generated some comments. He should be\n>     posting another round soon. 'pu' at best.\n>\n>   fc/mutt-alias\n>     Latest round that addressed comments. Ready for 'next' if not\n>     'master'.\n>\n>   jn/gitweb-patch\n>     After some comments with Jakub, I think the code is right but he\n>     promised a re-roll with more in the commit message.\n>\n>   mr/gitweb-snapshot\n>     This is probably the pu rewind you saw. He posted a v5 of his\n>     series. I didn't look at it closely, but Jakub ack'd it.\n>\n>   tf/doc-pt-br\n>     Minor translation update, ack'd by somebody who can read it. :)\n>     Ready for 'master'.\n\nI've queued all of these to 'pu' for tonight as I haven't still fully\nrecovered yet from jetlag, but I agree with the above assessment.\n\n> There are a few patches floating on the list that I haven't picked up or\n> looked too closely at yet. Just so we don't drop anything, they are:\n>\n>   - curl http auth tweak:\n>     http://article.gmane.org/gmane.comp.version-control.git/129455\n>     The author said he didn't really test it, and I haven't set up http\n>     auth to test it with, but probably somebody should do so before\n>     applying. :)\n\nHeh ;-)\n\n>   - a new rev-cache from Nick\n\nWhat Shawn had in his 'pu' had one patch removed from what I had earlier\n(perhaps my 'tip list' was faulty).  I'll pick the new one up later.\n\n>   - silence gcc warning:\n>     http://article.gmane.org/gmane.comp.version-control.git/129485\n>     The warning is overly cautious, I think, but it is a dubious enough\n>     construct that it is probably worth fixing.\n\nI cannot reach gmane now, but if this is about -Wextra, I'd rather not\ntouch it before the release.  \"comparison between signed and unsigned\"\ntends to be excessive and IMNSHO it is crazy to use -Wextra and -Werror \ntogether.\n\n>   - enable openssl on msvc\n>     http://article.gmane.org/gmane.comp.version-control.msysgit/7238\n>     This goes on top of ef/msys-imap, but I think that will be getting a\n>     re-roll.\n\nOk.\n\nThanks.\n\n"},{"id":"299125","messageId":"20091004132936.GA13804@sigill.intra.peff.net","threadId":"21056","inReplyTo":"7viqevpjr3.fsf@alter.siamese.dyndns.org","subject":"Re: Interim maintainer tree","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-10-04T13:29:36Z","receivedAt":"2009-10-04T13:29:36Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Oct 04, 2009 at 02:54:08AM -0700, Junio C Hamano wrote:\n\n> >   - silence gcc warning:\n> >     http://article.gmane.org/gmane.comp.version-control.git/129485\n> >     The warning is overly cautious, I think, but it is a dubious enough\n> >     construct that it is probably worth fixing.\n> \n> I cannot reach gmane now, but if this is about -Wextra, I'd rather not\n> touch it before the release.  \"comparison between signed and unsigned\"\n> tends to be excessive and IMNSHO it is crazy to use -Wextra and -Werror \n> together.\n\nActually, I mislabeled this. It is about a glibc run-time complaint not\na gcc warning, which is more worrisome. It is triggered by an argv list\nwhere we pass (argv - 1) to something expecting only to index it\nstarting from '1'. I'm not sure yet if it is glibc being picky or if we\nviolate that assumption somewhere. I'll post a followup in the thread.\n\n-Peff\n"},{"id":"124253","messageId":"20091005145615.GA9261@spearce.org","threadId":"21056","inReplyTo":"7viqevu1zt.fsf@alter.siamese.dyndns.org","subject":"Re: Interim maintainer tree","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2009-10-05T14:56:15Z","receivedAt":"2009-10-05T14:56:15Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio C Hamano <gitster@pobox.com> wrote:\n> \"Shawn O. Pearce\" <spearce@spearce.org> writes:\n> > Junio is on vaction for the next week.  In his absence Peff and I\n> > are trying to keep up with current patches in my fork:\n> >\n> >   git://repo.or.cz/git/spearce.git\n> >   http://repo.or.cz/r/git/spearce.git\n> >\n> > Right now the tree matches Junio's last push, I'll try to pick up\n> > the patches since then and push later today.\n> \n> Thanks, both.\n> \n> I've fetched, but haven't fully examined \"log ..spearce/*\" nor \"log ..peff/*\"\n> yet.\n\nSorry, I proved to be a very poor interim maintainer this cycle.\nI just didn't have the time to keep up with integration each day,\nand then dropped the ball on Thursday and Friday and failed to pull\nanything from Peff or the list.  Peff, thanks for keeping up a bit\nand at least having some tips for Junio to pick up from.\n \n> I noticed that some topics in 'pu' have been rebased (not complaining, but\n> just making sure I am not hallucinating).\n\nYes, some topics in 'pu' got rebased.  Wednesday I ran RB\nafter preparing a new master, and then rebuilt pu around it.\nUnfortunately I somehow lost a patch on Nick's rev-list series,\nnot sure how I managed to do that, but I guess I did, sorry.\n\nWere you no longer in the habit of running RB when you rebuilt pu?\n \n-- \nShawn.\n"},{"id":"124254","messageId":"20091005150546.GA18460@coredump.intra.peff.net","threadId":"21056","inReplyTo":"20091005145615.GA9261@spearce.org","subject":"Re: Interim maintainer tree","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2009-10-05T15:05:47Z","receivedAt":"2009-10-05T15:05:47Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Oct 05, 2009 at 07:56:15AM -0700, Shawn O. Pearce wrote:\n\n> Sorry, I proved to be a very poor interim maintainer this cycle.\n> I just didn't have the time to keep up with integration each day,\n> and then dropped the ball on Thursday and Friday and failed to pull\n> anything from Peff or the list.  Peff, thanks for keeping up a bit\n> and at least having some tips for Junio to pick up from.\n\nIt was an interesting experience this time, having two people doing it.\nI think the communication overhead may have overwhelmed any help we were\ngiving each other. Perhaps if there had been more, larger series, it\nmight have made sense to assign somebody to track a topic and its\niterations, and then finally sign off on it being ready. But being -rc\nperiod, things seem to have settled down, and most patches were one-off\nbugfixes all over the place.\n\n-Peff\n"}]}