databases/pglessog, databases/pg_rman problems after the last bsd.database.mk update

[Chris Rees]

On 6 March 2012 20:57, Chris Rees <utisoft at gmail.com> wrote:
> On 6 March 2012 20:50, Chris Rees <utisoft at gmail.com> wrote:
>> On 6 March 2012 10:25, Matthew Seaman <matthew at freebsd.org> wrote:
>>>
>>> Dear all,
>>>
>>> I noticed this when building indexes.  Seems that databases/pglesslog
>>> and databases/pg_rman are unhappy as a consequence of the last
>>> bsd.database.mk update, but only if postgresql (other than version 8.3)
>>> is already installed:
>>>
>>> Initially -- with postgresql-client-9.0.7_1 installed:
>>>
>>> databases/pglesslog:# make -V PGSQL_VER
>>>
>>> databases/pglesslog:# make -V BUILD_DEPENDS
>>> /nonexistent:/usr/ports/databases/postgresql-server:build
>>> gmake:/usr/ports/devel/gmake
>>>
>>>
>>> PGSQL_VER ends up empty, causing the BUILD_DEPENDS to refer to a
>>> non-existent 'postgresql-server' port.
>>>
>>> Simply deleting the postgresql client ...
>>>
>>> databases/pglesslog:# pkg delete -x postgresql
>>> The following packages will be deinstalled:
>>>        postgresql-client-9.0.7_1
>>>
>>> The deinstallation will save 8 MB
>>>
>>> Proceed with deinstalling packages [y/N]: y
>>> Deinstalling postgresql-client-9.0.7_1... done
>>>
>>>
>>> ... fixes the INDEX:
>>>
>>> databases/pglesslog:# make -V PGSQL_VER
>>> 83
>>> databases/pglesslog:# make -V BUILD_DEPENDS
>>> /nonexistent:/usr/ports/databases/postgresql83-server:build
>>> gmake:/usr/ports/devel/gmake
>>>
>>> Since this is dependent on postgresql !8.3 already being installed I
>>> guess it wouldn't have shown up in testing.  It seems that these are the
>>> only two ports affected.
>>
>> Yes-- these ports will have IGNORE set for them, so the INDEX entry
>> will be bogus anyway.
>
> I meant to add-- now the preferred way to depend on the server is to use:
>
> USE_PGSQL= server
>
> (also still depends on -client, but then so does -server)
>
> I'll work on getting server:extract to work-- looks popular.

I've just committed a fix that solves your INDEX problem for now [1],
but I'll still get kuriyama@ to use the new server:extract when it
comes in.

Chris

[1] http://www.freebsd.org/cgi/cvsweb.cgi/ports/Mk/bsd.database.mk.diff?r1=1.68;r2=1.69