Re: New repmgr packages

Hi,

On Thu, 2014-03-20 at 17:30 -0300, Martín Marqués wrote:

> I was working on repmgr-2.0 packages to update some systems (but with
> CFLAGS we need) and just saw that there's a new 2.0 repmgr package
> (thumbs up for that).

Yeah, I built them while working on new releases :)

> I did find a few things which I corrected differently from the way it
> was on the .src.rpm, basically I removed the makefile patch and set
> the PATHs accordingly in the spec file.

Err, IIRC, we *should not* export PATH in the spec file, per packaging
guidelines. That is why I add patches to each RPM.

> Another thing missing is the PGDG tag in Release.

We add PGDG only to PostgreSQL itself. The rest does not belong to PGDG,
so...

> I also saw some dependencies which were there before, but aren't
> anymore (rsync for example, and also openssh (which was mistakenly
> openssl before))

Ah.. Good catch. Created #164 for this:
http://wiki.pgrpms.org/ticket/164

> I'm sending a patch for repmgr.spec with what I'm pointing out here.
>
> I also changed a few things to make it look more like the
> postgresql-X.Y.spec files.

Do you mean these?

-%global pgmajorversion 91
-%global pginstdir /usr/pgsql-9.1
+%global pgmajorversion 9.1
+%define pgpackageversion 91
+%global pginstdir /usr/pgsql-%{pgmajorversion}

If so, our version is exactly the same as in the other packages. We
don't have pgpackageversion parameter anywhere. Am I missing something?

Thanks!

Regards,

--
Devrim GÜNDÜZ
Principal Systems Engineer @ EnterpriseDB: http://www.enterprisedb.com
PostgreSQL Danışmanı/Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR