[Bug 229323] Review Request: postgresql-pgpoolAdmin - web-based pgpool administration

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: postgresql-pgpoolAdmin - web-based pgpool administration


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229323





------- Additional Comments From devrim at commandprompt.com  2007-06-08 18:34 EST -------
Hi Jon,

(In reply to comment #8)
> Extra / in line 43:
> ln -s ../../../../%{_sysconfdir}/%{name}/pgmgt.conf.php
> %{buildroot}/%{_pgpoolAdmindir}/conf/pgmgt.conf.php
> 
> should be
> 
> ln -s ../../../..%{_sysconfdir}/%{name}/pgmgt.conf.php
> %{buildroot}%{_pgpoolAdmindir}/conf/pgmgt.conf.php

'k , fixed.

> Mock build fails with:
> RPM build errors:
>     File not found:
>
/var/tmp/postgresql-pgpoolAdmin-1.0.0-6.fc6-root-mockbuild/etc/httpd/conf.d/postgresql-pgpoolAdmin.conf
> 
> Looks like the conditional at line 46 is failing due to a BuildRequires issue.

Added httpd as a BuildRequires.

> Are you sure you don't want that to just install it no matter what?

It is required for security reasons + easy configuration for users.

I did not bump up spec file version; but updated SRPM:

New spec:
http://developer.postgresql.org/~devrim/rpms/other/pgpool-II/postgresql-pgpoolAdmin.spec
New SRPM:
http://developer.postgresql.org/~devrim/rpms/other/pgpool-II/postgresql-pgpoolAdmin-1.0.0-6.src.rpm

Regards, Devrim

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.