need some hint about how to handle the following issue (new bsd.apache.mk)

[Olli Hauer]

On 2011-03-29 23:51, Philip M. Gollucci wrote:
> On 03/29/11 21:33, Olli Hauer wrote:
>> http://people.freebsd.org/~ohauer/diffs/Mk/PR_147009_ports__Mk__bsd.apache.mk_2011-03-29_01.diff
>> http://people.freebsd.org/~ohauer/diffs/Mk/PR_147009_bsd.apache.mk_2011-03-29_01.txt
> 
>> @Philip
>> do you know the reason for the following expression
>> at top of bsd.apache.mk
>>
>>> .if !defined(Apache_Pre_Include) || defined(PORT_IS_MODULE)
> 
> USE_APACHE= common22 is reserved specifically so that www/apache* ports
> can use Mk/bsd.apache.mk themselves.  Note how *zope* ports get this wrong.
> 
> When USE_APACHE= commmonXX AP_PORT_IS_SERVER is true meaning your in
> www/apacheX* ports.  When AP_PORT_IS_MODULE is true you're got
> USE_APACHE=XX and you're in anything not www/apacheXX.  Usually www/mod_*.
> 
> See also the layout for www/apache22 and www/apache22-$mpm-mpm which
> re-uses the commonXX.
> 

The use of commonXX is clear, maybe the following explains
better why I ask.

-.if !defined(Apache_Pre_Include) || defined(PORT_IS_MODULE)
+.if !defined(Apache_Pre_Include) || defined(AP_PORT_IS_MODULE)