Thanks Sameera, I will proceed.
Regards, Chamil On Wed, Sep 3, 2014 at 11:40 AM, Sameera Jayasoma <same...@wso2.com> wrote: > Done > > > On Tue, Sep 2, 2014 at 7:31 PM, Chamil Jeewantha <cha...@wso2.com> wrote: > >> Hi Sameera, >> >> The patch has been attached to the JIRA. Added a workaround to avoid the >> public method with proper comments. >> >> Regards, >> Chamil >> >> >> On Tue, Sep 2, 2014 at 6:13 PM, Chamil Jeewantha <cha...@wso2.com> wrote: >> >>> Hi Sameera, >>> >>> For now that method is only used by four Log Handler classes. But there >>> may be other usages in the future as a utility method. >>> >>> As we discussed I will change the code so that it avoids the public >>> method for carbon 4.2.0 patch. >>> >>> Regards, >>> Chamil >>> >>> >>> On Tue, Sep 2, 2014 at 6:00 PM, Sameera Jayasoma <same...@wso2.com> >>> wrote: >>> >>>> Hi Chamil, >>>> >>>> I noticed that you've added a public method to LoggingUtils class. >>>> Whats the purpose of that method? >>>> >>>> Thanks, >>>> Sameera. >>>> >>>> >>>> On Tue, Sep 2, 2014 at 2:07 PM, Chamil Jeewantha <cha...@wso2.com> >>>> wrote: >>>> >>>>> Hi Sameera, >>>>> >>>>> I have attached the fix-only diff to the JIRA and sent the pull >>>>> request [1] >>>>> >>>>> [1] https://github.com/wso2-dev/carbon4-kernel/pull/88 >>>>> >>>>> >>>>> On Tue, Sep 2, 2014 at 1:15 PM, Chamil Jeewantha <cha...@wso2.com> >>>>> wrote: >>>>> >>>>>> Hi Sameera, >>>>>> >>>>>> I will attach the only diff against the fresh bootstrap to the JIRA. >>>>>> >>>>>> >>>>>> On Tue, Sep 2, 2014 at 12:33 PM, Sagara Gunathunga <sag...@wso2.com> >>>>>> wrote: >>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> On Tue, Sep 2, 2014 at 12:20 PM, Sameera Jayasoma <same...@wso2.com> >>>>>>> wrote: >>>>>>> >>>>>>>> Hi Chamil, >>>>>>>> >>>>>>>> Looks like you have attached whole bootstrap component in the >>>>>>>> patch. We cannot see the patch you did in that manner. Please attach >>>>>>>> only >>>>>>>> the changes.. >>>>>>>> >>>>>>>> Send a pull request to 4.3.0 repo also >>>>>>>> >>>>>>> >>>>>>> You can find original fix from here >>>>>>> https://github.com/sagaragu/carbon4-kernel/commit/9c29e215b6c93a74c4d37c14ea3be0fb509e1100 >>>>>>> >>>>>>> >>>>>>> Thanks ! >>>>>>> >>>>>>>> >>>>>>>> Thanks, >>>>>>>> Sameera. >>>>>>>> >>>>>>>> >>>>>>>> On Mon, Sep 1, 2014 at 11:58 AM, Chamil Jeewantha <cha...@wso2.com> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Hi Sameera, >>>>>>>>> >>>>>>>>> Can you please commit the [1] >>>>>>>>> >>>>>>>>> [1] https://wso2.org/jira/browse/CARBON-14929 >>>>>>>>> >>>>>>>>> >>>>>>>>> On Mon, Sep 1, 2014 at 10:44 AM, Chamil Jeewantha <cha...@wso2.com >>>>>>>>> > wrote: >>>>>>>>> >>>>>>>>>> Hi Geeth, >>>>>>>>>> >>>>>>>>>> Supun ran the build & tests successfully in his local >>>>>>>>>> environment. Can you please commit this patch since he is not >>>>>>>>>> available. >>>>>>>>>> >>>>>>>>>> Regards, >>>>>>>>>> Chamil >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> On Mon, Sep 1, 2014 at 10:23 AM, Chamil Jeewantha < >>>>>>>>>> cha...@wso2.com> wrote: >>>>>>>>>> >>>>>>>>>>> Hi Supun, >>>>>>>>>>> >>>>>>>>>>> Any update on this? >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> On Fri, Aug 29, 2014 at 3:12 PM, Chamil Jeewantha < >>>>>>>>>>> cha...@wso2.com> wrote: >>>>>>>>>>> >>>>>>>>>>>> Hi Supun, >>>>>>>>>>>> >>>>>>>>>>>> I have re-attached the patch to the issue. Please proceed. >>>>>>>>>>>> >>>>>>>>>>>> Regards, >>>>>>>>>>>> Chamil >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> On Fri, Aug 29, 2014 at 2:57 PM, Supun Malinga <sup...@wso2.com >>>>>>>>>>>> > wrote: >>>>>>>>>>>> >>>>>>>>>>>>> Hi Chamil, >>>>>>>>>>>>> >>>>>>>>>>>>> I had a chat with Sameera as well. Let's go ahead with the >>>>>>>>>>>>> above approach. >>>>>>>>>>>>> >>>>>>>>>>>>> thanks, >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> On Fri, Aug 29, 2014 at 2:37 PM, Chamil Jeewantha < >>>>>>>>>>>>> cha...@wso2.com> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>>> Hi All, >>>>>>>>>>>>>> >>>>>>>>>>>>>> I'm adding org.wso2.carbon.bootstrap*.jar into patch0009. >>>>>>>>>>>>>> This jar should go into <CARBON_HOME>/bin >>>>>>>>>>>>>> >>>>>>>>>>>>>> So I think we can add a bin folder to the patch.zip and copy >>>>>>>>>>>>>> this jar to that folder. Theadd a step in README to copy it >>>>>>>>>>>>>> manually to >>>>>>>>>>>>>> <CARBON_HOME>/bin >>>>>>>>>>>>>> >>>>>>>>>>>>>> WDYT? Is there a better way of doing that? >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Fri, Aug 29, 2014 at 1:49 PM, Chamil Jeewantha < >>>>>>>>>>>>>> cha...@wso2.com> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi Supun, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> org.wso2.carbon.bootstrap*.jar does not go into the plugin >>>>>>>>>>>>>>> directory. it goes to the <CARBON_HOME>/bin directory. Do we >>>>>>>>>>>>>>> still need to >>>>>>>>>>>>>>> add it to the patch zip? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Regards, >>>>>>>>>>>>>>> Chamil >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Fri, Aug 29, 2014 at 12:34 PM, Supun Malinga < >>>>>>>>>>>>>>> sup...@wso2.com> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Hi Chamil, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> As discussed let's add the entries to include the built jar >>>>>>>>>>>>>>>> into patch zip as well. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> thanks, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On Fri, Aug 29, 2014 at 12:29 PM, Chamil Jeewantha < >>>>>>>>>>>>>>>> cha...@wso2.com> wrote: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> $subject. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> https://wso2.org/jira/browse/CARBON-14929 >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>>>>>>>>> Associate Technical Lead >>>>>>>>>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>>>>>>>>> Mobile: +94716813892 >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>> Supun Malinga, >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Senior Software Engineer, >>>>>>>>>>>>>>>> WSO2 Inc. >>>>>>>>>>>>>>>> http://wso2.com >>>>>>>>>>>>>>>> email: sup...@wso2.com <sup...@wso2.com> >>>>>>>>>>>>>>>> mobile: +94 (0)71 56 91 321 >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>>>>>>> Associate Technical Lead >>>>>>>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>>>>>>> Mobile: +94716813892 >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> -- >>>>>>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>>>>>> Associate Technical Lead >>>>>>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>>>>>> Mobile: +94716813892 >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> -- >>>>>>>>>>>>> Supun Malinga, >>>>>>>>>>>>> >>>>>>>>>>>>> Senior Software Engineer, >>>>>>>>>>>>> WSO2 Inc. >>>>>>>>>>>>> http://wso2.com >>>>>>>>>>>>> email: sup...@wso2.com <sup...@wso2.com> >>>>>>>>>>>>> mobile: +94 (0)71 56 91 321 >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> -- >>>>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>>>> Associate Technical Lead >>>>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>>>> Mobile: +94716813892 >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> -- >>>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>>> Associate Technical Lead >>>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>>> Mobile: +94716813892 >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> -- >>>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>>> Associate Technical Lead >>>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>>> Mobile: +94716813892 >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>> -- >>>>>>>>> K.D. Chamil Jeewantha >>>>>>>>> Associate Technical Lead >>>>>>>>> WSO2, Inc.; http://wso2.com >>>>>>>>> http://kdchamil.blogspot.com >>>>>>>>> Mobile: +94716813892 >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> Sameera Jayasoma, >>>>>>>> Software Architect, >>>>>>>> >>>>>>>> WSO2, Inc. (http://wso2.com) >>>>>>>> email: same...@wso2.com >>>>>>>> blog: http://sameera.adahas.org >>>>>>>> twitter: https://twitter.com/sameerajayasoma >>>>>>>> flickr: http://www.flickr.com/photos/sameera-jayasoma/collections >>>>>>>> Mobile: 0094776364456 >>>>>>>> >>>>>>>> Lean . Enterprise . Middleware >>>>>>>> >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> Dev mailing list >>>>>>>> Dev@wso2.org >>>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Sagara Gunathunga >>>>>>> >>>>>>> Senior Technical Lead; WSO2, Inc.; http://wso2.com >>>>>>> V.P Apache Web Services; http://ws.apache.org/ >>>>>>> Linkedin; http://www.linkedin.com/in/ssagara >>>>>>> Blog ; http://ssagara.blogspot.com >>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> K.D. Chamil Jeewantha >>>>>> Associate Technical Lead >>>>>> WSO2, Inc.; http://wso2.com >>>>>> http://kdchamil.blogspot.com >>>>>> Mobile: +94716813892 >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> K.D. Chamil Jeewantha >>>>> Associate Technical Lead >>>>> WSO2, Inc.; http://wso2.com >>>>> http://kdchamil.blogspot.com >>>>> Mobile: +94716813892 >>>>> >>>>> >>>> >>>> >>>> -- >>>> Sameera Jayasoma, >>>> Software Architect, >>>> >>>> WSO2, Inc. (http://wso2.com) >>>> email: same...@wso2.com >>>> blog: http://sameera.adahas.org >>>> twitter: https://twitter.com/sameerajayasoma >>>> flickr: http://www.flickr.com/photos/sameera-jayasoma/collections >>>> Mobile: 0094776364456 >>>> >>>> Lean . Enterprise . Middleware >>>> >>>> >>> >>> >>> -- >>> K.D. Chamil Jeewantha >>> Associate Technical Lead >>> WSO2, Inc.; http://wso2.com >>> http://kdchamil.blogspot.com >>> Mobile: +94716813892 >>> >>> >> >> >> -- >> K.D. Chamil Jeewantha >> Associate Technical Lead >> WSO2, Inc.; http://wso2.com >> http://kdchamil.blogspot.com >> Mobile: +94716813892 >> >> > > > -- > Sameera Jayasoma, > Software Architect, > > WSO2, Inc. (http://wso2.com) > email: same...@wso2.com > blog: http://sameera.adahas.org > twitter: https://twitter.com/sameerajayasoma > flickr: http://www.flickr.com/photos/sameera-jayasoma/collections > Mobile: 0094776364456 > > Lean . Enterprise . Middleware > > -- K.D. Chamil Jeewantha Associate Technical Lead WSO2, Inc.; http://wso2.com http://kdchamil.blogspot.com Mobile: +94716813892
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev