{"thread":{"id":"8643","subject":"merging git-p4 into git/fastimport.git (second try)","startedAt":"2007-06-18T22:35:55Z","lastAt":"2007-06-22T23:20:02Z","messageCount":4,"participants":["Simon Hausmann","Shawn O. Pearce"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"45296","messageId":"200706190036.01950.simon@lst.de","threadId":"8643","inReplyTo":null,"subject":"merging git-p4 into git/fastimport.git (second try)","fromName":"Simon Hausmann","fromEmail":"simon@lst.de","sentAt":"2007-06-18T22:35:55Z","receivedAt":"2007-06-18T22:35:55Z","isPatch":false,"sender":{"key":"hausmann@kde.org","avatar":"https://gravatar.com/avatar/bc9aad4fb31dce17eb66e690e7b51fe980c62da3c225c785da35dd806b8da778?d=mp&s=160"},"body":"Hi,\n\nafter reworking most of the really short commit logs and adding the missing \nSOB lines I'd like to ask for consideration of merging git-p4 as part of \ncontrib/fastimport/ from\n\n\tgit://repo.or.cz/fast-export.git git-p4\n\n\nSimon\n"},{"id":"45297","messageId":"20070618230018.GA6143@spearce.org","threadId":"8643","inReplyTo":"200706190036.01950.simon@lst.de","subject":"Re: merging git-p4 into git/fastimport.git (second try)","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2007-06-18T23:00:18Z","receivedAt":"2007-06-18T23:00:18Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Simon Hausmann wrote:\n> after reworking most of the really short commit logs and adding the missing \n> SOB lines I'd like to ask for consideration of merging git-p4 as part of \n> contrib/fastimport/ from\n> \n> \tgit://repo.or.cz/fast-export.git git-p4\n\nMost of the commits look pretty good. The effort you have put into\ncleaning them up shows.\n\nHowever, many of the oneline descriptions from Han-Wen Nienhuys\nare still really short and not very descriptive. They won't look\nvery good in Junio's \"What's in git.git\". A few of yours are\nalso pretty long winded for a typical oneline in git.git, but I'd\nrather have a descriptive-but-long oneline anytime over a short\nand undescriptive one.\n\nSo I'm seriously considering merging this at this point. I'm heading\nout to dinner and will look at it more again when I get back, but\nI think we should just bite the bullet and bring it into core Git.\n\n-- \nShawn.\n"},{"id":"45316","messageId":"200706190842.13175.simon@lst.de","threadId":"8643","inReplyTo":"20070618230018.GA6143@spearce.org","subject":"Re: merging git-p4 into git/fastimport.git (second try)","fromName":"Simon Hausmann","fromEmail":"simon@lst.de","sentAt":"2007-06-19T06:42:09Z","receivedAt":"2007-06-19T06:42:09Z","isPatch":false,"sender":{"key":"hausmann@kde.org","avatar":"https://gravatar.com/avatar/bc9aad4fb31dce17eb66e690e7b51fe980c62da3c225c785da35dd806b8da778?d=mp&s=160"},"body":"On Tuesday 19 June 2007 01:00:18 Shawn O. Pearce wrote:\n> Simon Hausmann wrote:\n> > after reworking most of the really short commit logs and adding the\n> > missing SOB lines I'd like to ask for consideration of merging git-p4 as\n> > part of contrib/fastimport/ from\n> >\n> > \tgit://repo.or.cz/fast-export.git git-p4\n>\n> Most of the commits look pretty good. The effort you have put into\n> cleaning them up shows.\n>\n> However, many of the oneline descriptions from Han-Wen Nienhuys\n> are still really short and not very descriptive. They won't look\n> very good in Junio's \"What's in git.git\". A few of yours are\n> also pretty long winded for a typical oneline in git.git, but I'd\n> rather have a descriptive-but-long oneline anytime over a short\n> and undescriptive one.\n\nWhy not just leave out the entire lot of git-p4 logs in \"What's in git.git\" \nthis one time? I'm not sure of how much value it is at this point anyway. I'm \njust interested in preserving the history to ease debugging.\n\n> So I'm seriously considering merging this at this point. I'm heading\n> out to dinner and will look at it more again when I get back, but\n> I think we should just bite the bullet and bring it into core Git.\n\nThere's also another option that might help to keep the logs clean for future \nplain \"git log\" usage and that is to prepend a \"git-p4:\" prefix to most of \nthe logs. That makes \"git-p4: Documentation cleanups\" look better I think and \nit's really easy to do.\n\n\nSimon\n"},{"id":"45587","messageId":"20070622232002.GJ17393@spearce.org","threadId":"8643","inReplyTo":"200706190842.13175.simon@lst.de","subject":"Re: merging git-p4 into git/fastimport.git (second try)","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2007-06-22T23:20:02Z","receivedAt":"2007-06-22T23:20:02Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Simon Hausmann wrote:\n> Why not just leave out the entire lot of git-p4 logs in \"What's in git.git\" \n> this one time? I'm not sure of how much value it is at this point anyway. I'm \n> just interested in preserving the history to ease debugging.\n\nYeah, maybe a good idea. That's now up to Junio. I just pulled\nyour git-p4 branch into my fastimport, and republished it.\n\n-- \nShawn.\n"}]}