[pacman-dev] output issues
checking package integrity... done. (2/2) checking for file conflicts [---------------------] 100% warning: directory permissions differ on var/cache/pacman/ o o o o o ] 0% filesystem: 555 package: 755 (1/2) upgrading pacman-git [---------------------] 100% (2/2) upgrading xscreensaver [---------------------] 100% Who wants to step up to the plate and fix this stuff? I'm sure Xavier can point us to the past conversations we've had on the subject, and we really should fix it. I liked the suggestion of buffering all these messages until after the progress bar reaches 100%,. Of course, this could be a hard change to make. Output functions in general need a sprucing up and/or unification. I know there are still printf("error...") type things scattered around the front end, and most of these could be replaced with a call to one of the pm_vprintf functions. -Dan
On Nov 14, 2007 11:51 PM, Dan McGee <dpmcgee@gmail.com> wrote:
checking package integrity... done. (2/2) checking for file conflicts [---------------------] 100% warning: directory permissions differ on var/cache/pacman/ o o o o o ] 0% filesystem: 555 package: 755 (1/2) upgrading pacman-git [---------------------] 100% (2/2) upgrading xscreensaver [---------------------] 100%
Who wants to step up to the plate and fix this stuff? I'm sure Xavier can point us to the past conversations we've had on the subject, and we really should fix it.
I liked the suggestion of buffering all these messages until after the progress bar reaches 100%,. Of course, this could be a hard change to make.
Hmmm. It's a decent suggestion, but it sounds annoying.... pseudo code: def output(line): ... if progress_bar: list = alpm_list_add(list, line) ... # we'd need an event here def on_progress_complete(): for line in list: output(line) I'm not a fan of delayed execution like that, only because I slam my head against it during the day... /me smirks
On Wed, Nov 14, 2007 at 11:51:13PM -0600, Dan McGee wrote:
checking package integrity... done. (2/2) checking for file conflicts [---------------------] 100% warning: directory permissions differ on var/cache/pacman/ o o o o o ] 0% filesystem: 555 package: 755 (1/2) upgrading pacman-git [---------------------] 100% (2/2) upgrading xscreensaver [---------------------] 100%
Who wants to step up to the plate and fix this stuff? I'm sure Xavier can point us to the past conversations we've had on the subject, and we really should fix it.
That's the original thread I started, where I mentioned two kind of output issues (progress bar is the second one) : http://www.archlinux.org/pipermail/pacman-dev/2007-September/009317.html Then I found out pacman 3.0 didn't have this problem, it fixed it with padding. It only had a little issue with utf8 chars, so I proposed a patch for that : http://www.archlinux.org/pipermail/pacman-dev/2007-September/009325.html And you followed the thread there : http://www.archlinux.org/pipermail/pacman-dev/2007-October/009552.html
participants (3)
-
Aaron Griffin
-
Dan McGee
-
Xavier