{"thread":{"id":"6844","subject":"Re: Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer","startedAt":"2007-02-16T12:45:16Z","lastAt":"2007-02-16T23:25:48Z","messageCount":4,"participants":["Mark Levedahl","Simon 'corecode' Schubert","Johannes Schindelin"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"34791","messageId":"7238711.2657861171629916957.JavaMail.root@vms064.mailsrvcs.net","threadId":"6844","inReplyTo":null,"subject":"Re: Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer","fromName":"Mark Levedahl","fromEmail":"mdl123@verizon.net","sentAt":"2007-02-16T12:45:16Z","receivedAt":"2007-02-16T12:45:16Z","isPatch":true,"sender":{"key":"mdl123@verizon.net","avatar":"https://avatars.githubusercontent.com/u/5302462?v=4"},"body":">From: Simon 'corecode' Schubert <corecode@fs.ei.tum.de>\n>Date: 2007/02/16 Fri AM 05:57:20 CST\n>To: \"Shawn O. Pearce\" <spearce@spearce.org>\n>Cc: Mark Levedahl <mdl123@verizon.net>, git@vger.kernel.org\n>Subject: Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer\n\n>Shawn O. Pearce wrote:\n>> Mark Levedahl <mdl123@verizon.net> wrote:\n>>> +# create the tar file, clean up\n>>> +tar cf \"$bfile\" --absolute-names --transform=\"s,$tmp-,,\" \\\n>>> +    --verbose --show-transformed-names \\\n>>> +    \"$references\" \"$pack\"\n>> \n>> I'm not sure this will work on FreeBSD.  Both 5.1 and 6.1 use tar\n>> that does not know about --absolute-names, --transform, --verbose,\n>> or --show-transformed-names.\n>\n>for portability, pax is the official choice :)  and it can even do path name modifications with -s.  but why again are we using tar there?  this data could easiliy be put in one mixed text/binary file, starting out with\n>\n>#!/bin/sh\n>echo \"This is a git bundle.  Use git-unbundle to process me.\" >&1\n>exit\n>### DATA ###\n>\n>or so\n>\n>cheers\n>  simon\n\n... I *tried* that, and it fails under Cygwin. Apparently cygwin's bash (or something) mangles data in the pipe (99% certain it will turn out to be a latent crlf issue)...\n\n\ncat \"$bfile\" (\n    read refs...\n    git index-pack --stdin\n)\n\nworked several times, it only failed twice out of 8 bundles I tried. That's just a trifle bit too high a failure rate for my taste. ;^)\n\nMark\n"},{"id":"34793","messageId":"45D5B0C0.3040303@fs.ei.tum.de","threadId":"6844","inReplyTo":"7238711.2657861171629916957.JavaMail.root@vms064.mailsrvcs.net","subject":"Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer","fromName":"Simon 'corecode' Schubert","fromEmail":"corecode@fs.ei.tum.de","sentAt":"2007-02-16T13:25:20Z","receivedAt":"2007-02-16T13:25:20Z","isPatch":true,"sender":{"key":"corecode@fs.ei.tum.de","avatar":"https://gravatar.com/avatar/eff9dbf0cdac0d1e6a6cd7ed0e50763edcb376b493b5253a35ff167918ad79e1?d=mp&s=160"},"body":"Mark Levedahl wrote:\n>> #!/bin/sh\n>> echo \"This is a git bundle.  Use git-unbundle to process me.\" >&1\n>> exit\n>> ### DATA ###\n>>\n>> or so\n>>\n>> cheers\n>>  simon\n> \n> ... I *tried* that, and it fails under Cygwin. Apparently cygwin's\n> bash (or something) mangles data in the pipe (99% certain it will\n> turn out to be a latent crlf issue)...\n\nyou can try something like:\n\nsed -e '1,/^##DATA##$/d' \"$bundle\" | git-index-pack --stdin\n\nand see if it works better.  no need to stream the bundle completely, it can be read several times.\n\ncheers\n  simon\n\n-- \nServe - BSD     +++  RENT this banner advert  +++    ASCII Ribbon   /\"\\\nWork - Mac      +++  space for low €€€ NOW!1  +++      Campaign     \\ /\nParty Enjoy Relax   |   http://dragonflybsd.org      Against  HTML   \\\nDude 2c 2 the max   !   http://golden-apple.biz       Mail + News   / \\\n\n"},{"id":"34794","messageId":"Pine.LNX.4.63.0702161443370.22628@wbgn013.biozentrum.uni-wuerzburg.de","threadId":"6844","inReplyTo":"7238711.2657861171629916957.JavaMail.root@vms064.mailsrvcs.net","subject":"Re: Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2007-02-16T13:44:52Z","receivedAt":"2007-02-16T13:44:52Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Fri, 16 Feb 2007, Mark Levedahl wrote:\n\n> ... I *tried* that, and it fails under Cygwin. Apparently cygwin's bash \n> (or something) mangles data in the pipe (99% certain it will turn out to \n> be a latent crlf issue)...\n\nHave you tried\n\n\texport CYGWIN=binmode\n\nbefore that? (If it works, you have to make sure that other settings as \n\"ntsec\" are retained in that environment variable, but not \"nobinmode\").\n\nHth,\nDscho\n"},{"id":"34842","messageId":"45D63D7C.40205@verizon.net","threadId":"6844","inReplyTo":"Pine.LNX.4.63.0702161443370.22628@wbgn013.biozentrum.uni-wuerzburg.de","subject":"Re: [PATCH] Add git-bundle - pack objects and references for disconnected transfer","fromName":"Mark Levedahl","fromEmail":"mdl123@verizon.net","sentAt":"2007-02-16T23:25:48Z","receivedAt":"2007-02-16T23:25:48Z","isPatch":true,"sender":{"key":"mdl123@verizon.net","avatar":"https://avatars.githubusercontent.com/u/5302462?v=4"},"body":"Johannes Schindelin wrote:\n> Hi,\n>\n> On Fri, 16 Feb 2007, Mark Levedahl wrote:\n>\n>   \n>> ... I *tried* that, and it fails under Cygwin. Apparently cygwin's bash \n>> (or something) mangles data in the pipe (99% certain it will turn out to \n>> be a latent crlf issue)...\n>>     \n>\n> Have you tried\n>\n> \texport CYGWIN=binmode\n>\n> before that? (If it works, you have to make sure that other settings as \n> \"ntsec\" are retained in that environment variable, but not \"nobinmode\").\n>\n> Hth,\n> Dscho\nIn a word, yes. I tried many things with mount and CYGWIN, none fixed \nthe problem. What I don't have is a simple test case I can push upstream \nto demonstrate the failure mode. However, if this is a crlf issue I \nsuspect this would not work in msys either, regardless of being able to \nfind the issue in Cygwin.\n\nMark\n"}]}