Fixed following issues and revised the webrev. 
* Reverted ::MACH64:: to 64
* Fixed copyright format in both depend file as suggested by link:
http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/prototypes/prototype.Makefile

Seema's concern of i.renamenew issue is pending, waiting for her reply.

Regards,
Basant.

On Fri, Mar 06, 2009 at 02:28:11PM +0530, Seema Alevoor wrote:
> No, please revert this change.
> See CRs 6782602, 6812225
>
> -- Seema.
>
> On 03/06/09 14:16, Basant Kukreja wrote:
>> Changed 64 to ::MACH64:: in sed.conf similar to jk.conf. Revised the webrev.
>>
>> Thanks for reviewing,
>> Basant.
>>
>> On Fri, Mar 06, 2009 at 02:14:40PM +0530, rahul wrote:
>>> | Thanks for reviewing :
>>> | | On Fri, Mar 06, 2009 at 12:20:12PM +0530, rahul wrote:
>>> | > - sed module.conf says
>>> | > +++ new/usr/src/cmd/apache2/modules/sed.conf    Thu Mar  5 18:31:19 2009
>>> | * security2.conf, fcgid.conf all takes the same approach :
>>> | * Since sed.conf is architechture independent, it has two choices:
>>> | 1. LoadModule sed_module libexec/64/mod_sed.so
>>> | or
>>> | 2. LoadModule sed_module libexec/$ARCH/mod_sed.so
>>> | | In both cases, user have to edit this file.
>>> | | * There was a plan to create 64 symlink in apache2/libexec 
>>> directory (similar
>>> | to webstack 1.4), but I don't know what is the resolution.
>>> | | We can file a separate Bug to correct all apache modules sample 
>>> configuration.
>>> ok with me.
>>>
>>> The original incantation was
>>> LoadModule jk_module libexec/::MACH64::/mod_jk.so
>>>
>>> and it was supposed to be processed by conf-install in the main
>>> Makefile.sfw, since conf-install is not doing that, it also need to be
>>> fixed as a separate bug.
>>>
>>>                                      rahul
>>> --
>>> 1. e4 _
>> _______________________________________________
>>
>>
>> webstack-discuss mailing list
>> webstack-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss

Reply via email to