Bug 209655 – net-analyzer/nagios-core-2.10-r1 handled apache eclasses incorrectly

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Daniel Meyer 2008-02-11 13:33:39 UTC]

99_nagios.conf is supposed to be installed to /etc/apache2/modules.d/, but ends up in /

Reproducible: Always

Steps to Reproduce:
1. install nagios-core-2.10-r1
2. do a ls -alph /


Actual Results:  
[...]
>>> /etc/conf.d/nagios
--- /etc/revdep-rebuild/
>>> /etc/revdep-rebuild/55-nagios-core-revdep
--- /etc/nagios/
>>> /etc/nagios/.keep_net-analyzer_nagios-core-0
>>> /99_nagios.conf


Expected Results:  
[...]
>>> /etc/conf.d/nagios
--- /etc/revdep-rebuild/
>>> /etc/revdep-rebuild/55-nagios-core-revdep
--- /etc/nagios/
>>> /etc/nagios/.keep_net-analyzer_nagios-core-0
>>> /etc/apache2/modules.d/99_nagios.conf

[Comment 1 Jakub Moc (RETIRED) 2008-02-11 14:20:17 UTC]

Created attachment 143210 [details, diff]
nagios-core-2.10-r2.ebuild.diff

The current ebuild has multiple issues with correct apache eclasses usage, see attached diff.

[Comment 2 Jakub Moc (RETIRED) 2008-02-11 14:34:01 UTC]

Created attachment 143211 [details, diff]
nagios-core-2.10-r2.ebuild.diff

Also tweaked the postinstall instructions for apache2 use flag.

[Comment 3 Jakub Moc (RETIRED) 2008-02-11 14:39:10 UTC]

Created attachment 143213 [details, diff]
nagios-core-2.10-r2.ebuild.diff

Geez, missed on noweb flag in there... :X

[Comment 4 Matthew R. Place 2008-02-27 19:27:01 UTC]

(In reply to comment #3)
> Created an attachment (id=143213) [edit]
> nagios-core-2.10-r2.ebuild.diff
> 
> Geez, missed on noweb flag in there... :X
> 

Jakub,

I pulled down the nagios-core-2.10-r2 diff and created an ebuild from it.
Upon emerge of virtual/nagios, I found that 99_nagios.conf was copied to the appropriate directory (/etc/apache2/modules.d).

The emerge was successful and I have encountered no errors so far with the proposed ebuild changes.

Platform: x86
If you want more system/environment stats, just let me know.

-Matt-

[Comment 5 Jakub Moc (RETIRED) 2008-02-27 19:32:27 UTC]

Nah, as long as it works it's fine ;) Need this commited.

[Comment 6 Benedikt Böhm (RETIRED) 2008-02-27 19:39:02 UTC]

ugh. i wonder why i missed this ebuild during `grep APACHE2_DEPEND` -r /usr/portage ...

[Comment 7 Tobias Scherbaum (RETIRED) 2008-02-27 20:32:01 UTC]

(In reply to comment #5)
> Nah, as long as it works it's fine ;) Need this commited.
> 

inCVS. I removed the "apache-module" eclass, from my understanding both the depend and $APACHE_MODULES_CONFDIR should be set by the depend.apache eclass. Correct me if i'm wrong.