Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 300171 - net-analyzer/munin-1.4.3 version bump
Summary: net-analyzer/munin-1.4.3 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Robin Johnson
URL: http://munin.projects.linpro.no/
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-08 15:38 UTC by Matthias Siedler
Modified: 2010-02-03 15:19 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
munin-1.3.4-r2 - 1.4.3 ebuild patch [needs works] (munin-1.3.4-r2-1.4.3.ebuild.patch,2.85 KB, patch)
2010-01-08 17:13 UTC, Jeroen Roovers (RETIRED)
Details | Diff
ebuild for 1.4.3 (munin-1.4.3.ebuild,4.17 KB, text/plain)
2010-01-08 23:42 UTC, Brian De Wolf
Details
revised patch from 1.3.4 (munin-1.4.3-fw_conntrack_plugins.patch,724 bytes, patch)
2010-01-08 23:43 UTC, Brian De Wolf
Details | Diff
revised Makefile.config patch (munin-1.4.3-Makefile.patch,2.25 KB, patch)
2010-01-08 23:43 UTC, Brian De Wolf
Details | Diff
Revised parallel make patch (munin-1.4.3-parallel-make.patch,336 bytes, patch)
2010-01-08 23:43 UTC, Brian De Wolf
Details | Diff
revised plugin cleanup patch (munin-1.4.3-plugin-cleanup.patch,953 bytes, patch)
2010-01-08 23:44 UTC, Brian De Wolf
Details | Diff
Part of the install hit parallel make problems as well and I missed it (munin-1.4.3-parallel-make.patch,1.67 KB, patch)
2010-01-09 00:44 UTC, Brian De Wolf
Details | Diff
Protect a new directory in /etc/munin not present in 1.3.x (munin-1.4.3.ebuild,4.21 KB, text/plain)
2010-01-11 21:43 UTC, Brian De Wolf
Details
Update patch to improve parallel make (munin-1.4.3-parallel-make.patch,1.89 KB, patch)
2010-01-12 23:12 UTC, Brian De Wolf
Details | Diff
re-revised Makefile.config patch: Also install into vendor_perl, not site_perl (munin-1.4.3-Makefile.patch,2.47 KB, patch)
2010-01-18 18:45 UTC, Jeroen Roovers (RETIRED)
Details | Diff
ebuild without parallel makes (munin-1.4.3.ebuild,4.23 KB, text/plain)
2010-01-28 18:36 UTC, Brian De Wolf
Details
munin-1.4.3.ebuild (munin-1.4.3.ebuild,4.09 KB, text/plain)
2010-02-01 04:37 UTC, Jeremy Olexa (darkside) (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Siedler 2010-01-08 15:38:02 UTC
Upstream is already at 1.43 for a while. 



Reproducible: Always

Steps to Reproduce:
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-08 16:46:52 UTC
HOMEPAGE redirects to URL now.
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-08 17:13:08 UTC
Created attachment 215692 [details, diff]
munin-1.3.4-r2 - 1.4.3 ebuild patch [needs works]

This shows some of the changes between the latest stable and what is needed for 1.4.3 to enter the tree:

(1) None of the old patches apply (and will likely need to be replaced - see
    also (3)).

(2) Two deps added (dev-perl/Net-CIDR from bug #287718) and dev-perl/Log-
    Log4perl because src_compile() failed when it couldn't find this.

(3) The build still fails in src_compile:

[...]
cd node && /usr/bin/perl Build.PL
Looks good

Creating new 'Build' script for 'Munin::Master' version '0.0.0'
cd plugins && /usr/bin/perl Build.PL
Can't locate Munin/Common/Defaults.pm in @INC (@INC contains: ../common/blib/lib /etc/perl /usr/lib/perl5/vendor_perl/5.8.8/i686-linux /usr/lib/perl5/vendor_perl/5.8.8 /usr/lib/perl5/vendor_perl /usr/lib/perl5/site_perl/5.8.8/i686-linux /usr/lib/perl5/site_perl/5.8.8 /usr/lib/perl5/site_perl /usr/lib/perl5/5.8.8/i686-linux /usr/lib/perl5/5.8.8 /usr/local/lib/site_perl .) at NodeBuilder.pm line 14.
BEGIN failed--compilation aborted at NodeBuilder.pm line 14.
Compilation failed in require at Build.PL line 1.
BEGIN failed--compilation aborted at Build.PL line 1.
make: *** [node/Build] Error 2
make: *** Waiting for unfinished jobs....
Checking prerequisites...
Looks good

Creating new 'Build' script for 'Munin::Plugins' version '0.0.0'
 * ERROR: net-analyzer/munin-1.4.3 failed:
 *   build/build-man failed
 *
 * Call stack:
 *     ebuild.sh, line  48:  Called src_compile
 *   environment, line 2353:  Called die
 * The specific snippet of code:
 *       emake build build-man || die "build/build-man failed";
Comment 3 Brian De Wolf 2010-01-08 23:42:42 UTC
Created attachment 215735 [details]
ebuild for 1.4.3

I've been looking for a 1.4.x ebuild for a bit, so I decided to give updating this one a shot.  I'm able to build/install this ebuild and everything appears correct, but I haven't tested it.
Comment 4 Brian De Wolf 2010-01-08 23:43:11 UTC
Created attachment 215737 [details, diff]
revised patch from 1.3.4
Comment 5 Brian De Wolf 2010-01-08 23:43:36 UTC
Created attachment 215739 [details, diff]
revised Makefile.config patch
Comment 6 Brian De Wolf 2010-01-08 23:43:55 UTC
Created attachment 215741 [details, diff]
Revised parallel make patch
Comment 7 Brian De Wolf 2010-01-08 23:44:28 UTC
Created attachment 215742 [details, diff]
revised plugin cleanup patch
Comment 8 Brian De Wolf 2010-01-09 00:44:47 UTC
Created attachment 215753 [details, diff]
Part of the install hit parallel make problems as well and I missed it
Comment 9 Brian De Wolf 2010-01-11 21:43:27 UTC
Created attachment 216120 [details]
Protect a new directory in /etc/munin not present in 1.3.x
Comment 10 Matthias Siedler 2010-01-12 12:59:51 UTC
Tried to use the ebuild but got an error when emerging:

From the build.log:

--------------
...
for f in build/master/doc/munin.conf node/doc/munin-node.conf; do \
           pod2man --section=5 --release=1.4.3 --center="Munin Documentation" "$f".pod > build/doc/`basename $f .pod`.5; \
        done
cd common && /usr/bin/perl Build code
cd node && /usr/bin/perl Build.PL
rm -f common/blib/lib/Munin/Common/Defaults.pm
cd plugins && /usr/bin/perl Build.PL
Can't locate Munin/Common/Defaults.pm in @INC (@INC contains: ../common/blib/lib /etc/perl /usr/lib/perl5/vendor_perl/5.8.8/i686-linux /usr/lib/perl5/vendor_perl/5.8.8 /usr/lib/perl5/vendor_perl /usr/lib/perl5/site_perl/5.8.8/i686-linux /usr/lib/perl5/site_perl/5.8.8 /usr/lib/perl5/site_perl /usr/lib/perl5/5.8.8/i686-linux /usr/lib/perl5/5.8.8 /usr/local/lib/site_perl .) at NodeBuilder.pm line 14.
BEGIN failed--compilation aborted at NodeBuilder.pm line 14.
Compilation failed in require at Build.PL line 1.
BEGIN failed--compilation aborted at Build.PL line 1.
make: *** [node/Build] Error 2
make: *** Waiting for unfinished jobs....
Checking prerequisites...
Looks good
----

Why is the common/blib/lib/Munin/Common/Defaults.pm removed after it was build?
Comment 11 Brian De Wolf 2010-01-12 23:12:27 UTC
Created attachment 216304 [details, diff]
Update patch to improve parallel make
Comment 12 Brian De Wolf 2010-01-12 23:15:55 UTC
Let me know if the updated parallel make patch helps fix it.  I'm currently poking at the developers to find someone I can speak to to hopefully work out the issues with the Makefile, as it boggles me.

Also, I'm unable to reproduce your build failure.  If the updated patch doesn't resolve the issue, can you respond with more details?  I've currently only tested the ebuild on amd64.
Comment 13 Matthias Siedler 2010-01-13 16:07:29 UTC
It is quite strange, I had to emerge twice and then it was working (emerge succeeded).

I'm facing some other issues now, mainly that the cron can't be setup and the /etc/init.d/munin-node is missing. When emerging I can see:

 * QA Notice: file does not exist:
 *
 *      newins: /usr/local/portage/net-analyzer/munin/files/munin-1.3.2-plugins.conf does not exist
 *      newins: /usr/local/portage/net-analyzer/munin/files/munin-1.3.3-crontab does not exist
 *      newinitd: /usr/local/portage/net-analyzer/munin/files/munin-node_init.d_1.3.3-r1 does not exist
 *      newconfd: /usr/local/portage/net-analyzer/munin/files/munin-node_conf.d_1.3.3-r1 does not exist
 *      newins: /usr/local/portage/net-analyzer/munin/files/logrotate.d-munin does not exist
Comment 14 Matthias Siedler 2010-01-13 17:32:15 UTC
I have copied them from the 1.3.3 ebuild for now.
Comment 15 Brian De Wolf 2010-01-13 17:51:25 UTC
Sorry, I didn't make it clear but it uses some of the older files because they didn't have to be changed for the newer version.  It would be best to copy the files/ directory from the portage tree if you are housing this ebuild in an overlay.
Comment 16 Matthias Siedler 2010-01-13 17:55:37 UTC
Thanks, that's what I have done right now....and it is running again...yipiyeah
Comment 17 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-13 19:08:28 UTC
The parallel make patch doesn't seem to work as intended.

cp plugins/javalib/org/munin/plugin/jmx/MemorySurvivorPeak.class build/plugins/javalib/
org/munin/plugin/jmx/MemorySurvivorPeak.class
mkdir -p build/`dirname plugins/javalib/org/munin/plugin/jmx/Threads.class`
cp plugins/javalib/org/munin/plugin/jmx/MemoryPermGenUsagePostGC.class build/plugins/ja
valib/org/munin/plugin/jmx/MemoryPermGenUsagePostGC.class
/usr/bin/perl -pe 's{(PREFIX     \s+=\s).*}{\1q{/usr};}x;   \
                  s{(CONFDIR    \s+=\s).*}{\1q{/etc/munin};}x;     \
                  s{(BINDIR     \s+=\s).*}{\1q{/usr/bin};}x;      \
                  s{(SBINDIR    \s+=\s).*}{\1q{/usr/sbin};}x;     \
                  s{(DOCDIR     \s+=\s).*}{\1q{/usr/share/doc/munin-1.4.3};}x;      \
                  s{(LIBDIR     \s+=\s).*}{\1q{/usr/libexec/munin};}x;      \
                  s{(MANDIR     \s+=\s).*}{\1q{/usr/share/man};}x;      \
                  s{(LOGDIR     \s+=\s).*}{\1q{/var/log/munin};}x;      \
                  s{(HTMLDIR    \s+=\s).*}{\1q{/usr/var/www/localhost/htdocs/munin};}x;
     \
                  s{(DBDIR      \s+=\s).*}{\1q{/var/lib/munin};}x;       \
                  s{(STATEDIR   \s+=\s).*}{\1q{/var/run/munin};}x;    \
                  s{(PERL       \s+=\s).*}{\1q{/usr/bin/perl};}x;        \
                  s{(PERLLIB    \s+=\s).*}{\1q{/usr/lib/perl5/site_perl/5.8.8};}x;     
\
                  s{(PYTHON     \s+=\s).*}{\1q{/usr/bin/env python};}x;      \
                  s{(RUBY       \s+=\s).*}{\1q{/usr/bin/env ruby};}x;        \
                  s{(OSTYPE     \s+=\s).*}{\1q{linux};}x;      \
                  s{(HOSTNAME   \s+=\s).*}{\1q{astrid};}x;    \
                  s{(MKTEMP     \s+=\s).*}{\1q{mktemp -p /tmp/ $1};}x;      \
                  s{(VERSION    \s+=\s).*}{\1q{1.4.3};}x;     \
                  s{(PLUGSTATE  \s+=\s).*}{\1q{/var/lib/munin/plugin-state};}x;   \
                  s{(CGIDIR     \s+=\s).*}{\1q{/usr/var/www/localhost/cgi-bin};}x;     
 \
                  s{(USER       \s+=\s).*}{\1q{munin};}x;        \
                  s{(GROUP      \s+=\s).*}{\1q{munin};}x;       \
                  s{(PLUGINUSER \s+=\s).*}{\1q{nobody};}x;  \
                  s{(GOODSH     \s+=\s).*}{\1q{/bin/sh};}x;      \
                  s{(BASH       \s+=\s).*}{\1q{/bin/bash};}x;        \
                  s{(HASSETR    \s+=\s).*}{\1q{1};}x;     \
                  s{(SSPOOLDIR  \s+=\s).*}{\1q{/var/run/munin/spool};}x;'  \
                  common/lib/Munin/Common/Defaults.pm > common/blib/lib/Munin/Common/De
faults.pm
cp plugins/javalib/org/munin/plugin/jmx/Threads.class build/plugins/javalib/org/munin/p
lugin/jmx/Threads.class
/bin/sh: common/blib/lib/Munin/Common/Defaults.pm: Permission denied
make: *** [common/blib/lib/Munin/Common/Defaults.pm] Error 1
make: *** Waiting for unfinished jobs....
Comment 18 Brian De Wolf 2010-01-13 19:56:48 UTC
(In reply to comment #17)
> The parallel make patch doesn't seem to work as intended.
...
>                   common/lib/Munin/Common/Defaults.pm >
> common/blib/lib/Munin/Common/Defaults.pm
...
> /bin/sh: common/blib/lib/Munin/Common/Defaults.pm: Permission denied
> make: *** [common/blib/lib/Munin/Common/Defaults.pm] Error 1
> make: *** Waiting for unfinished jobs....
> 

It looks like the file is created, but it doesn't look to be caused by parallel make.  Is there something about your build environment that would make a file created by a command in make to not be readable by a later command spawned by make?
Comment 19 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-13 20:14:39 UTC
We'll need this change too if we want to support FEATURES=test properly. It fails even then at a lot of trivial tests, though.

--- munin-1.4.3.ebuild  2010-01-13 21:13:25.000000000 +0100
+++ munin-1.4.3-r1.ebuild       2010-01-13 20:37:59.000000000 +0100
@@ -11,8 +11,8 @@
 SRC_URI="mirror://sourceforge/munin/${P}.tar.gz"
 LICENSE="GPL-2"
 SLOT="0"
-KEYWORDS="amd64 ~mips ppc sparc x86"
-IUSE="doc minimal irc mysql postgres ssl"
+KEYWORDS="~amd64 ~mips ~x86"
+IUSE="doc minimal irc mysql postgres ssl test"
 
 # Upstream's listing of required modules is NOT correct!
 # Some of the postgres plugins use DBD::Pg, while others call psql directly.
@@ -42,7 +42,10 @@
                        #munin-sybase? (         dev-perl/DBD-Sybase )
 
 # Keep this seperate, as previous versions have had other deps here
-DEPEND="${DEPEND_COM}"
+DEPEND="
+       ${DEPEND_COM}
+       test? ( dev-perl/Test-Perl-Critic dev-perl/Test-Pod-Coverage )
+"
 RDEPEND="${DEPEND_COM}
                !minimal? ( virtual/cron )"
 
Comment 20 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-13 20:18:44 UTC
Maybe this will help get the ebuild into the tree too:

@@ -68,7 +72,7 @@
 }
 
 src_compile() {
-       emake build build-man || die "build/build-man failed"
+       emake -j1 build build-man || die "build build-man failed"
        if use doc; then
                emake build-doc || die "build-doc failed"
        fi
Comment 21 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-13 20:22:36 UTC
(In reply to comment #18)
> > /bin/sh: common/blib/lib/Munin/Common/Defaults.pm: Permission denied
[...]

> It looks like the file is created, but it doesn't look to be caused by parallel
> make.  Is there something about your build environment that would make a file
> created by a command in make to not be readable by a later command spawned by
> make?

Er, to me that looks like /bin/sh is trying to *execute*, not just *read* common/blib/lib/Munin/Common/Defaults.pm
Comment 22 Brian De Wolf 2010-01-13 21:57:30 UTC
(In reply to comment #21)
> Er, to me that looks like /bin/sh is trying to *execute*, not just *read*
> common/blib/lib/Munin/Common/Defaults.pm

Good point.  I've had so many problems with it not existing that my mind went right down that path.  I can't find anything in the Makefile that would attempt to execute it, though.  I'm starting to give up on attempting to fix the Makefile.  Does the ebuild emerge correctly if you add "-j1" and remove the parallel make patch?

I got the impression from previous bugs that using "-j1" is unacceptable, but if it means the ebuild works, I'll be happy with that.
Comment 23 Jeroen Roovers (RETIRED) gentoo-dev 2010-01-18 18:45:13 UTC
Created attachment 216830 [details, diff]
re-revised Makefile.config patch: Also install into vendor_perl, not site_perl
Comment 24 Brian De Wolf 2010-01-28 18:36:59 UTC
Created attachment 217758 [details]
ebuild without parallel makes

Here is an ebuild without the parallel make patch and with "-j 1" added to the emake calls.  I have given up on fixing parallel make, for now.
Comment 25 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-02-01 04:37:52 UTC
Created attachment 218051 [details]
munin-1.4.3.ebuild

This ebuild is based of the one in Attachment 217758 [details] (comment #24). It has dropped EPATCH_OPTS (Makefile patch wouldn't apply unless this was dropped).

Also applied:
-style changes for the global scope variables to be more aligned with common practices.
-additional die statements

TODO: I will be testing this version as a node with a 1.3.4 master in the next few days.
Comment 26 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-02-03 15:19:46 UTC
(In reply to comment #25)
> TODO: I will be testing this version as a node with a 1.3.4 master in the next
> few days.

Tested fine. I got the ACK from Robin to commit.

+*munin-1.4.3 (03 Feb 2010)
+
+  03 Feb 2010; Jeremy Olexa <darkside@gentoo.org>
+  -files/munin-1.3.2-plugin-cleanup.patch,
+  -files/munin-1.3.2-rrdtool-comments.patch, -munin-1.3.3.ebuild,
+  -munin-1.3.3-r1.ebuild, -munin-1.3.3-r2.ebuild,
+  -files/munin-1.3.3-Makefile.patch, -munin-1.3.4.ebuild,
+  -munin-1.3.4-r1.ebuild, +munin-1.4.3.ebuild,
+  +files/munin-1.4.3-Makefile.patch,
+  +files/munin-1.4.3-fw_conntrack_plugins.patch,
+  +files/munin-1.4.3-ping6_fix.patch,
+  +files/munin-1.4.3-plugin-cleanup.patch, metadata.xml:
+  Version bump to 1.4.3, many thanks to all the contributors on bug 300171.
+  Include ping6 fix from bug 294722. Include QA fix for perl site-dir, bug
+  298528. Remove old versions and unused patches