please find attached ebuild that upgrades genlop to version 0.30.4 bye !
Created attachment 64827 [details] ebuild
epia ~ # genlop -c readline() on closed filehandle $cmdline at /usr/bin/genlop line 526. Use of uninitialized value in substitution (s///) at /usr/bin/genlop line 540. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 545, <_GEN_1> line 29609. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 550, <_GEN_1> line 29609. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 545, <_GEN_1> line 29609. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 550, <_GEN_1> line 29609. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 545, <_GEN_1> line 29609. Use of uninitialized value in concatenation (.) or string at /usr/bin/genlop line 550, <_GEN_1> line 29609. [ and so on ]
Created attachment 64950 [details] Obligatory emerge info
It does work sometimes: ------------------------------------- epia ~ # genlop -c * kde-base/kdelibs-3.4.1-r1 current merge time: 3 hours, 27 minutes and 5 seconds. ETA: any time now. epia ~ # ------------------------------------- ... as expected. Maybe the script doesn't handle certain stages of the merge process properly.
Created attachment 65005 [details, diff] Patch: genlop should remove the whitespace from the output of ps I did that with sed this time. I am *certain* Perl must have some way to handle this internally... The problem is, when ps outputs a pid of just four digits, it prefixes the pid with a space... An ugly but workable solution is in this patch.
Created attachment 65006 [details, diff] Better yet, use pgrep instead of ps (both are in sys-process/procps)
Antoinio - comments on last post before I post this?
Created attachment 65384 [details, diff] genlop-0.30.4-dash.patch: Adds a dash to the package search in emerge.log While merging dev-libs/glib (takes minutes to build), 'genlop --current' and 'genlop --pretend' calculate results for '$ebuild_arg.*' in emerge.log instead of relying on a $pattern based on "$categoria/$package_dir". The results for 'genlop --pretend' and 'genlop --current' therefore would include build times for sys-libs/glibc, which takes a lot longer to compile. By searching for '$ebuild_arg-.*' instead, the ETA is (more) accurate.
(In reply to comment #8) > While merging dev-libs/glib (takes minutes to build), 'genlop --current' and > 'genlop --pretend' calculate results for '$ebuild_arg.*' in emerge.log instead > of relying on a $pattern based on "$categoria/$package_dir". Illustration: henke ~ # genlop -i glib * dev-libs/glib Total builds: 9 Global build time: 2 hours, 33 minutes and 9 seconds. Average merge time: 17 minutes and 1 second. Info about currently installed ebuild: * dev-libs/glib-2.6.4 Install date: Wed Jul 20 00:57:04 2005 USE=" -doc -hardened -static" CFLAGS="-Os -march=i586 -pipe" * dev-libs/glib-1.2.10-r5 Install date: Thu Jul 7 15:38:36 2005 USE=" -hardened" CFLAGS="-Os -march=i586 -pipe -fPIC" henke ~ # genlop -i glibc * sys-libs/glibc Total builds: 11 Global build time: 1 day, 23 hours, 2 minutes and 11 seconds. Average merge time: 4 hours, 16 minutes and 33 seconds. Info about currently installed ebuild: * sys-libs/glibc-2.3.5 Install date: Mon Jul 18 11:12:20 2005 USE="nls pic nptl nptlonly userlocales -build -erandom -hardened -debug - multilib -selinux" CFLAGS="-march=i586 -pipe -O2 -finline-limit=2000 -O2" henke ~ # genlop -c * dev-libs/glib-2.6.4 current merge time: 1 minute and 34 seconds. ETA: 2 hours, 8 minutes and 20 seconds.
Resolving this bug by committing the two patches (either in Portage or upstream) may ultimately solve a couple of other bugs too: * bug 72162 (NEW) is fixed by the dash patch; * bug 79541 (NEW) is fixed by the space patch; * bug 80618 (NEW) is supposed to have been fixed in 0.30.3; * bug 93853 (NEW) is fixed in 0.30.4; ( * bug 92989 (NEW) is in limbo; could be the dash patch thing again, since there's an "openoffice-bin" package as well as plain "openoffice", with starkly differing merge times; ) * bug 93853 (NEW) is fixed in 0.30.4; * bug 97049 (NEW) is supposed to have been fixed in 0.30.4; * bug 99823 (NEW) is fixed in 0.30.4, but 0.30.4 is not patched for the ps space or the dash issues yet; There. Does this make life easier? ;-)
PS: There are a couple of other genlop related bugs: bug 90787 (NEW) notably, which requests marking 0.30.3 stable, and this one...
Created attachment 65420 [details, diff] Improved dash patch: genlop-0.30.4-dash-r1.patch Adds $ebuild_arg-[0-9] instead of just $ebuild_arg-, so that part of the package version is caught too (prevents confusing, say, openoffice and openoffice-bin or binutils and binutils-config).
Created attachment 65424 [details, diff] genlop--pretend-with-dash-r1.out: Test with dash-patch and dash-r1 patch
Created attachment 66095 [details, diff] Roundup patch against 0.30.4
Created attachment 66096 [details, diff] New ebuild as patch against genlop-0.30.4.ebuild Patch against genlop-0.30.4.build that applies the above roundup patch. New in that patch is that not only it checks for the obligatory -[0-9] in package names, but also for the obligatory slash prefixed to the package name. So now it does not only distinguish between, say, (sys-devel/)binutils and (sys-devel/)binutils-config, but also between (sys-devel/)m4 and (xfce-base/)xfwm4, or better yet, between (app-doc/)ebook-binutils and (sys-devel/)binutils-config.
Hello all, I have released a new version of genlop with all the fixes from this thread you can find it at: http://gelo.dolcetta.net/software/genlop-0.30.5.tar.gz A simple ebuild bump should take care of everything, ebuild attached Sorry for the buggy 0.30.4 release Thanks
Created attachment 66214 [details] new ebuild genlop-0.30.5
posted :)
*** Bug 99823 has been marked as a duplicate of this bug. ***
