Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 39926 - nagios-core ebuild references pgsql instead of postgres
Summary: nagios-core ebuild references pgsql instead of postgres
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: rob holland (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-30 17:16 UTC by Paul Kronenwetter
Modified: 2004-02-11 05:39 UTC (History)
2 users (show)

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


Attachments
Ebuild patch to allow nagios to build with postgreSQL support. (nagios.patch,529 bytes, patch)
2004-01-30 17:47 UTC, Paul Kronenwetter
Details | Diff
nagios-core-1.1-r6.ebuild (nagios-core-1.1-r6.ebuild,5.47 KB, text/plain)
2004-02-03 06:44 UTC, Michael Boman (RETIRED)
Details
ChangeLog diff (ChangeLog.diff,829 bytes, patch)
2004-02-03 06:48 UTC, Michael Boman (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Kronenwetter 2004-01-30 17:16:10 UTC
Both 1.1-r4 and 1.1-r5 ebuilds of nagios reference:

DEPEND=">=net-mail/mailx-8.1
        !noweb? (
                >=media-libs/jpeg-6b-r3
                >=media-libs/libpng-1.2.5-r4
                >=media-libs/libgd-1.8.3-r5

                apache2? ( >=net-www/apache-2.0.43-r1 )
                !apache2? ( <net-www/apache-2 )
        )

        perl? ( >=dev-lang/perl-5.6.1-r7 )
        mysql? ( >=dev-db/mysql-3.23.56 )
        pgsql? ( >=dev-db/postgresql-7.3.2 )"

Where ufed reports that postgres is the proper USE flag to integrate into ebuilds.
Comment 1 Paul Kronenwetter 2004-01-30 17:34:50 UTC
I'm beginning that it's probably this way because it doesn't compile with postgres support enabled.  Still working with it.
Comment 2 Paul Kronenwetter 2004-01-30 17:47:06 UTC
Created attachment 24668 [details, diff]
Ebuild patch to allow nagios to build with postgreSQL support.

Ebuild patch to allow nagios to build with postgreSQL support.

NOTE: This does a --with-pgsql-inc=/usr/include/postgresql hard-coded into the
ebuild.  So if the postgresql ebuild moves it for some reason this will be
broken.
Comment 3 Paul Kronenwetter 2004-01-30 17:49:33 UTC
Ok, figured it out.  Because of the way postgreSQL is installed and the way it's referenced by nagios there are two solutions.  One is to apply the nagios.patch as attached to the 1.1-r4 and 1.1-r5 ebuilds.  

The other is to find and patch all the places where #include <pgsql/XXX.h> is used.  I'm torn as to which is the "right" way to do it for gentoo.  

I've already considered pg_config --includedir, but it outputs /usr/include doesn't help.

Thoughts?
Comment 4 Michael Boman (RETIRED) gentoo-dev 2004-02-02 07:12:04 UTC
I just want to let you know that I am in the process of verifying the updated ebuild on both stable and unstable x86. I'll post a note when you can get -r6 from your local rsync mirror.

Sorry for the inconvinience.
Comment 5 Michael Boman (RETIRED) gentoo-dev 2004-02-03 06:44:47 UTC
Created attachment 24885 [details]
nagios-core-1.1-r6.ebuild

Proposed ebuild.
Comment 6 Michael Boman (RETIRED) gentoo-dev 2004-02-03 06:48:10 UTC
Created attachment 24886 [details, diff]
ChangeLog diff

Changelog diff
Comment 7 Michael Boman (RETIRED) gentoo-dev 2004-02-03 06:49:08 UTC
I am experiancing some hardware problems at the moment. tigger^ promised to take care of this bug for the time being.
Comment 8 Lance Albertson (RETIRED) gentoo-dev 2004-02-03 07:09:09 UTC
Another thing that would be nice is to possibly move the sample config files for nagios from /etc/nagios to /usr/share/doc/nagios. I've noticed in the past that when you upgrade nagios, it will put <nagios config file names>-sample files back in /etc/nagios. I really think if you want to look for sample configs, they should be in the directory posted above. 
Comment 9 rob holland (RETIRED) gentoo-dev 2004-02-11 05:39:01 UTC
sorted and in cvs as -6