Bug 143447 - [patch] fix ports/mail/squirrelmail/bsd.squirrelmail.mk to not override USE_PHP set in plugin Makefiles
Summary: [patch] fix ports/mail/squirrelmail/bsd.squirrelmail.mk to not override USE_P...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Thomas Abthorpe
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-01 18:40 UTC by Darren Pilgrim
Modified: 2010-02-02 03:30 UTC (History)
0 users

See Also:


Attachments
file.diff (271 bytes, patch)
2010-02-01 18:40 UTC, Darren Pilgrim
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darren Pilgrim 2010-02-01 18:40:01 UTC
The bsd.squirrelmail.mk is a Makefile include used by squirrelmail-plugin ports which provides common routines for correctly installing and activating plugins.  The include file sets USE_PHP=yes unconditionally.  If a plugin requires PHP extensions, this setting overrides any USE_PHP setting (e.g., USE_PHP=xmlrpc) in the plugin Makefile.

Fix: Modify bsd.squirrelmail.mk to set USE_PHP only if unset (i.e., ?= operator).  The attached patch does exactly this.

Patch attached with submission follows:
How-To-Repeat: 1. Create a Makefile that sets USE_PHP to something other than yes;
2. Run make -V USE_PHP and observe your setting;
3. Modify the Makefile to include ${PORTSDIR}/mail/squirrelmail/bsd.squirrelmail.mk after your USE_PHP setting;
4. Run make -V USE_PHP again and observe USE_PHP is now set to "yes" instead of your setting;
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2010-02-01 18:40:10 UTC
Responsible Changed
From-To: freebsd-ports-bugs->tabthorpe

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2010-02-02 03:26:10 UTC
tabthorpe    2010-02-02 03:25:57 UTC

  FreeBSD ports repository

  Modified files:
    mail/squirrelmail    bsd.squirrelmail.mk 
  Log:
  - Set USE_PHP?= yes, so selected PHP modules can override setting
  
  PR:             ports/143447
  Submitted by:   Darren Pilgrim <darren.pilgrim gmail.com>
  
  Revision  Changes    Path
  1.7       +3 -3      ports/mail/squirrelmail/bsd.squirrelmail.mk
_______________________________________________
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 Thomas Abthorpe freebsd_committer freebsd_triage 2010-02-02 03:26:43 UTC
State Changed
From-To: open->closed

Committed. Thanks!