Bug 164662 - databases/postgresql91-contrib: Configuration introduced overwrites postgresql91-server|client
databases/postgresql91-contrib: Configuration introduced overwrites postgresq...
Status: Closed FIXED
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s)
Latest
Any Any
: Normal Affects Only Me
Assigned To: Chris Rees
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-01-31 11:00 UTC by neil
Modified: 2012-02-01 11:00 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description neil 2012-01-31 11:00:32 UTC
The ossp-uuid option introduced into postgresql-contrib-9.1.2_3 overwrites options set through postgresql-server-9.1.2_3.

Fix: 

Can't the postgresql-contrib option be moved into postgresql91-server/Makefile?

Wouldn't the use of ossp-uuid need to be global throughout postgresql91-* builds anyway?
How-To-Repeat: 1) make -C /usr/ports/databases/postgresql91-server config
2) Close options dialog with Save
3) Observe options correctly set in /var/db/ports/postgresql91/options
4) make -C /usr/ports/databases/postgresql91-contrib config
5) Close options dialog with Save
6) Observe server options overwritten in /var/db/ports/postgresql91/options
Comment 1 jgh@FreeBSD.org freebsd_committer 2012-01-31 14:42:15 UTC
Responsible Changed
From-To: freebsd-ports-bugs->crees

update synopsis and assign
Comment 2 dfilter freebsd_committer 2012-01-31 18:21:46 UTC
crees       2012-01-31 18:21:33 UTC

  FreeBSD ports repository

  Modified files:
    databases/postgresql91-contrib Makefile 
    databases/postgresql91-server Makefile 
  Log:
  - Stop clobbering postgresql-server/-client OPTIONS with -contrib OPTIONS
  - Minor optimisation (stop testing for !SLAVE_ONLY twice)
  
  PR:             ports/164662
  
  Revision  Changes    Path
  1.5       +0 -2      ports/databases/postgresql91-contrib/Makefile
  1.242     +5 -4      ports/databases/postgresql91-server/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Chris Rees freebsd_committer 2012-01-31 18:22:39 UTC
State Changed
From-To: open->closed

Thanks for reporting.  I've fixed this by giving -contrib a different 
UNIQUENAME from -server and -client; the other two share options where 
-contrib does not.
Comment 4 neil 2012-02-01 10:39:50 UTC
The UNIQUENAME construction isn't quite correct.

PKGNAMESUFFIX is -contrib and UNIQUENAME is created with 
${...}-${PKGNAMESUFFIX}.

This leads to the creation of /var/db/ports/postgresql91--contrib instead of 
/var/db/ports/postgresql91-contrib.

That is, two hyphens instead of one.