pcp
[Top] [All Lists]

Re: [pcp] pcp updates: some build/package re-jigging and QA

To: Nathan Scott <nathans@xxxxxxxxxx>
Subject: Re: [pcp] pcp updates: some build/package re-jigging and QA
From: Lukas Berk <lberk@xxxxxxxxxx>
Date: Wed, 11 Nov 2015 14:06:44 -0500
Cc: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>, pcp@xxxxxxxxxxx
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <2108185319.9113269.1447193300260.JavaMail.zimbra@xxxxxxxxxx> (Nathan Scott's message of "Tue, 10 Nov 2015 17:08:20 -0500 (EST)")
References: <563949B8.9020604@xxxxxxxxxxxxxxxx> <87d1vn81w8.fsf@xxxxxxxxxx> <56424CB3.5090506@xxxxxxxxxxxxxxxx> <2108185319.9113269.1447193300260.JavaMail.zimbra@xxxxxxxxxx>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
Hey,

Nathan Scott <nathans@xxxxxxxxxx> writes:
> ----- Original Message -----
>> [...]
>> elasticsearch is not the only PMDA omitted from your build, from the log
>> file ...
>> 
>> checking if the mysql PMDA should be included... no
>> checking if the postgresql PMDA should be included... no
>> checking if the elasticsearch PMDA should be included... no
>> checking if the nginx PMDA should be included... no
>> checking if the activemq PMDA should be included... no
>> ...
>> checking if the postfix PMDA should be included... no
>
> Commit 01dd8ff862 picks up all of these except for pmdapostfix.

Thanks for the follow up commit and comments.  I'm curious, why
pmdapostfix wasn't added to this commit?  I'm getting a fail for the
postfix pmda for same reason as the ealier elasticsearch commit.

With the below patch on pmdapostfix, a fresh fedora rawhide build works
(tested with a scratch build from a locally built srpm).  I'd like to
get an official build out asap, as currently pcp is broken in rawhide.

diff --git a/build/rpm/fedora.spec b/build/rpm/fedora.spec
index 7906e3f..82d9308 100644
--- a/build/rpm/fedora.spec
+++ b/build/rpm/fedora.spec
@@ -909,6 +909,7 @@ URL: http://www.pcp.io
 Requires: perl-PCP-PMDA = %{version}-%{release}
 %if 0%{?fedora} > 16 || 0%{?rhel} > 5
 Requires: postfix-perl-scripts
+BuildRequires: postfix-perl-scripts
 %endif
 %if 0%{?rhel} <= 5
 Requires: postfix

<Prev in Thread] Current Thread [Next in Thread>