{"thread":{"id":"10597","subject":"That new progress meter","startedAt":"2007-11-02T18:36:35Z","lastAt":"2007-11-03T16:09:50Z","messageCount":4,"participants":["Johannes Schindelin","Pierre Habouzit","Nicolas Pitre","Alex Riesen"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"58016","messageId":"Pine.LNX.4.64.0711021836000.4362@racer.site","threadId":"10597","inReplyTo":null,"subject":"That new progress meter","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2007-11-02T18:36:35Z","receivedAt":"2007-11-02T18:36:35Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Nico,\n\nthat new progress meter sure is amazing and useful!\n\nThanks,\nDscho\n"},{"id":"58110","messageId":"20071103120953.GC13417@artemis.corp","threadId":"10597","inReplyTo":"Pine.LNX.4.64.0711021836000.4362@racer.site","subject":"Re: That new progress meter","fromName":"Pierre Habouzit","fromEmail":"madcoder@debian.org","sentAt":"2007-11-03T12:09:53Z","receivedAt":"2007-11-03T12:09:53Z","isPatch":false,"sender":{"key":"madcoder@debian.org","avatar":"https://avatars.githubusercontent.com/u/44708?v=4"},"body":"On Fri, Nov 02, 2007 at 06:36:35PM +0000, Johannes Schindelin wrote:\n> Hi Nico,\n> \n> that new progress meter sure is amazing and useful!\n\n I do agree. There seems to be some glitches though, here is how my\noutput looks after a git fetch I just did on git.git:\n\n remote: Generating pack...\n remote: Done counting 310 objects.\n remote: Deltifying 310 objects... | Here we have a glitch |\n remote: 100% (310/310) done `-------vvvv----------'\n remote: Total 310 (delta 160), reused 178 (delta 112)iB/s\n Receiving objects: 100% (310/310), 379.98 KiB | 136 KiB/s, done.\n Resolving deltas: 100% (160/160), done.\n * refs/remotes/origin/html: fast forward to branch 'html' of git://git.kernel.org/pub/scm/git/git\n old..new: 1c70883..7ae0ab2\n * refs/remotes/origin/maint: fast forward to branch 'maint' of git://git.kernel.org/pub/scm/git/git\n old..new: 136e631..f45e867\n * refs/remotes/origin/man: fast forward to branch 'man' of git://git.kernel.org/pub/scm/git/git\n old..new: 9850e2e..44dd7e0\n * refs/remotes/origin/master: fast forward to branch 'master' of git://git.kernel.org/pub/scm/git/git\n old..new: 3e4bb08..e3d6d56\n * refs/remotes/origin/next: fast forward to branch 'next' of git://git.kernel.org/pub/scm/git/git\n old..new: a93d0b0..536f64a\n * refs/remotes/origin/pu: forcing update to non-fast forward branch 'pu' of git://git.kernel.org/pub/scm/git/git\n old...new: eb57be8...bf1284a\n\nFWIW, maybe instead using spaces to erase lines we could use minimal\nvt100 codes[0] like:\n\nErase End of Line [K\n Erases from the current cursor position to the end of the current line. \nErase Start of Line [1K\n Erases from the current cursor position to the start of the current line. \nErase Line [2K\n Erases the entire current line.\n\n [0] http://www.termsys.demon.co.uk/vtansi.htm\n\n-- \n·O· Pierre Habouzit\n··O madcoder@debian.org\nOOO http://www.madism.org\n"},{"id":"58141","messageId":"alpine.LFD.0.9999.0711031042390.21255@xanadu.home","threadId":"10597","inReplyTo":"20071103120953.GC13417@artemis.corp","subject":"Re: That new progress meter","fromName":"Nicolas Pitre","fromEmail":"nico@cam.org","sentAt":"2007-11-03T14:53:25Z","receivedAt":"2007-11-03T14:53:25Z","isPatch":false,"sender":{"key":"nico@fluxnic.net","avatar":"https://avatars.githubusercontent.com/u/702790?v=4"},"body":"On Sat, 3 Nov 2007, Pierre Habouzit wrote:\n\n> On Fri, Nov 02, 2007 at 06:36:35PM +0000, Johannes Schindelin wrote:\n> > Hi Nico,\n> > \n> > that new progress meter sure is amazing and useful!\n> \n> I do agree. There seems to be some glitches though, here is how my\n> output looks after a git fetch I just did on git.git:\n> \n> remote: Generating pack...\n> remote: Done counting 310 objects.\n> remote: Deltifying 310 objects... | Here we have a glitch |\n> remote: 100% (310/310) done `-------vvvv----------'\n> remote: Total 310 (delta 160), reused 178 (delta 112)iB/s\n> Receiving objects: 100% (310/310), 379.98 KiB | 136 KiB/s, done.\n> Resolving deltas: 100% (160/160), done.\n\nI know. This is why i TRIED TO KEEP THE \"Receiving objects\" line as \nshort as possible.\n\n> FWIW, maybe instead using spaces to erase lines we could use minimal\n> vt100 codes[0] like:\n> \n> Erase End of Line [K\n> Erases from the current cursor position to the end of the current line. \n\nI thought about that too, but this is not perfectly portable to all \nterminals according to a quick glance at /etc/termcap. Does the Windows \nconsole support it? Also we might prefer not to rely on termcap/terminfo \nlibrary calls.\n\nThe other solution is to make the remote object summary line a bit \nlonger, but this will be effective only when remote servers are \nupgraded. Might that be good enough?\n\n\nNicolas\n"},{"id":"58152","messageId":"20071103160950.GB3175@steel.home","threadId":"10597","inReplyTo":"alpine.LFD.0.9999.0711031042390.21255@xanadu.home","subject":"Re: That new progress meter","fromName":"Alex Riesen","fromEmail":"raa.lkml@gmail.com","sentAt":"2007-11-03T16:09:50Z","receivedAt":"2007-11-03T16:09:50Z","isPatch":false,"sender":{"key":"raa.lkml@gmail.com","avatar":"https://avatars.githubusercontent.com/u/324101?v=4"},"body":"Nicolas Pitre, Sat, Nov 03, 2007 15:53:25 +0100:\n> The other solution is to make the remote object summary line a bit \n> longer, but this will be effective only when remote servers are \n> upgraded. Might that be good enough?\n\nHow about keeping track of the length of the last lines the remote end\nsent (recv_sideband in sideband.c)? So that we always know how much\nspaces to add to clear up to the end of line.\n"}]}