| Summary: | genlop 0.30.4 | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Antonio Dolcetta <zagarna> |
| Component: | New packages | Assignee: | Michael Cummings (RETIRED) <mcummings> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | jer, sascha-gentoo-bugzilla |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| URL: | http://gelo.dolcetta.net/software/genlop-0.30.4.tar.gz | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: |
ebuild
Obligatory emerge info Patch: genlop should remove the whitespace from the output of ps Better yet, use pgrep instead of ps (both are in sys-process/procps) genlop-0.30.4-dash.patch: Adds a dash to the package search in emerge.log Improved dash patch: genlop-0.30.4-dash-r1.patch genlop--pretend-with-dash-r1.out: Test with dash-patch and dash-r1 patch Roundup patch against 0.30.4 New ebuild as patch against genlop-0.30.4.ebuild new ebuild genlop-0.30.5 |
||
|
Description
Antonio Dolcetta
2005-07-31 15:17:29 UTC
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. *** |
