Bug 116329

Summary: www/apache20 rc-script doesn't always source %%PREFIX%%/sbin/envvars when needed
Product: Ports & Packages Reporter: Ruud Althuizen <ruud>
Component: Individual Port(s)Assignee: Clement Laforet <clement>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Ruud Althuizen 2007-09-13 12:00:14 UTC 
When you add *.env files to /usr/local/etc/apache2/envvars.d/ they only
get sourced after start_cmd(). But apache2_checkconfig() also needs those
includes. So I moved the lines.

Fix: Patch attached with submission follows:
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2007-09-13 12:33:46 UTC 
Responsible Changed
From-To: freebsd-bugs->clement

Make this a ports PR and assign.
Comment 2 Clement Laforet freebsd_committer freebsd_triage 2007-09-18 20:19:35 UTC 
State Changed
From-To: open->closed

Good catch, thanks! 
I'll backport it to www/apache22 too.
Comment 3 dfilter service freebsd_committer freebsd_triage 2007-09-18 21:03:22 UTC 
clement     2007-09-18 20:03:15 UTC

  FreeBSD ports repository

  Modified files:
    .                    UPDATING 
    www/apache22         Makefile 
    www/apache22/files   apache22.sh.in 
  Added files:
    www/apache22         Makefile.options 
  Log:
  - Add experimental support of OPTIONS.
    From UPDATING:
  
    By popular request, OPTIONS support has been added. When actived
    (default), these knobs are ignored:
            * WITH_<CATEGORY>_MODULES
            * WITHOUT_<CATEGORY>_MODULES
            * WITH_CUSTOM_<CATEGORY>
            * WITH_MODULES
            * WITHOUT_MODULES
            * WITH_STATIC_MODULES
    However, you can disable OPTIONS by defining WITHOUT_APACHE_OPTIONS.
  
  - move envvars support to the beginning of apache22_checkconfig() to be
    sure we're using envvars during configtest [1]
  
  PR:             ports/116329 [1]
  Submitted by:   Ruud Althuizen <ruud@il.fontys.nl> [1]
  
  Revision  Changes    Path
  1.543     +15 -1     ports/UPDATING
  1.210     +11 -0     ports/www/apache22/Makefile
  1.1       +75 -0     ports/www/apache22/Makefile.options (new)
  1.5       +6 -6      ports/www/apache22/files/apache22.sh.in
_______________________________________________
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"