example: bash-2.05b# /usr/local/bin/emerge -puv gcc These are the packages that I would merge, in order: Calculating dependencies ...done! [ebuild UD] sys-libs/glibc-2.3.2-r1 [2.3.2-r2] -nls +pic -build +nptl 13,417 kB [ebuild U ] sys-devel/patch-2.5.9 [2.5.4-r4] -build -static 197 kB [ebuild U ] sys-apps/groff-1.18.1-r2 [1.18.1-r1] +X -cjk 2,310 kB [ebuild U ] sys-apps/sed-4.0.7 [4.0.6] -nls -static -build 677 kB [ebuild U ] sys-apps/man-1.5l-r6 [1.5l-r2] -nls 213 kB [ebuild U ] sys-libs/db-3.2.9-r5 [3.2.9-r1] 2,036 kB [ebuild U ] dev-lang/perl-5.8.0-r11 [5.8.0-r10] +berkdb +doc +gdbm -threads 10,854 kB [ebuild U ] sys-devel/binutils-2.14.90.0.2 [2.14.90.0.1-r1] -nls -bootstrap -build 10,103 kB [ebuild U ] sys-libs/ncurses-5.3-r2 [5.3-r1] 2,019 kB [ebuild UD] sys-devel/gcc-3.2.3-r1 [3.3] -static -nls -bootstrap +java -build 20,715 kB Total size of downloads: 62,544 kb I surrounded all of my code with lines of solid #'s. My changes go from line 1000 to 1104. Please consider adding this to the next version of portage. PS- I saw another topic with this, but it was abandoned, and used a differant aproch. Sorry if it was wrong to post this dup. Reproducible: Always Steps to Reproduce: 1. 2. 3.
Created attachment 13705 [details] modified emerge file
Created attachment 13747 [details] Updated file Fixes A bug where emerge -p wouldn't work without -v.
Created attachment 13748 [details] The correct file
Created attachment 13749 [details] The correct file
Comment on attachment 13747 [details] Updated file oops! wrong file
Please post 'diff -u' patches. I'm a bit busy to diff myself.
Created attachment 14034 [details, diff] diff -u -w file
Created attachment 16234 [details, diff] updated patch changed the filenames at the top of the patch. Apllies to 2.0.47 - 2.0.49-pre18 to carpaski: please include this in .49 as it is a visible improvement. btw, pre18 is worken for me so far ;)
Created attachment 16236 [details, diff] re-diffed patch to pre18
*** Bug 19596 has been marked as a duplicate of this bug. ***
diff -w diff -b diff -B are all unacceptable options for diffs. Python is very anal about kinds and counts of whitespace. Do not ignore it. diff -u only please.
*** Bug 11118 has been marked as a duplicate of this bug. ***
Created attachment 19035 [details, diff] my own version of the patch this is a mix of different patches, works fine with -r12
Can you break that out into a seperate function?
Created attachment 21441 [details, diff] new patch with functions seperated it into functions and improved a few bits
forgot to say, that patch is against cvs
Created attachment 21474 [details, diff] improved patch forgot a Integer check in format_size()
Created attachment 21894 [details, diff] new patch using digestParseFile() this patch uses digestParseFile() to get the filesizes.
I don't know if you've noticed it already, but the output is wrong when there are conditional SRC_URI. Example: % emerge -fuD world <snip, no errors> % emerge -puvD world <snip> [ebuild U ] net-www/dillo-0.7.3-r4 [0.7.3-r3] -cjk +gnome +ipv6 -kde +mozilla +nls +ssl +truetype 243 kB <snip> Total size of downloads: 243 kB This is because dillo has a "cjk? (http://...)" SRC_URI, which is 243k according to the digest. It's only a cosmetic issue, but it prevents me from using this output to decide whether yes or no I still have files to fetch before taking my laptop back home for an offline weekend :) I can't think of any easy fix so far.
Created attachment 22286 [details, diff] display-size-fix.patch Here is a patch that adds some SRC_URI parsing to "getsize", so that irrelevant files are not added to the total size. It is mainly some code stolen from doebuild, and solves the issue described in my above comment. Patch is against 2.0.50_pre1.
supposed to be fixed in 2.0.50 which is stable now. If this bug is not fixed please reopen.