Thanks Benoit! I'll send through some more fixes and improvements within the next couple of hours :-)
On Mon, 21 Mar 2022 at 15:06, Benoit TELLIER <btell...@linagora.com> wrote: > Hello, > > On 21/03/2022 10:23, Glennos wrote: > > Hi Benoit, > > > > At the moment I'm working from > > https://github.com/ff-wl/james-project/tree/netty4 - but I'd really > > appreciate a branch which combines all the netty4 work :-) > You can use https://github.com/chibenwa/james-project/tree/netty4-combo > which would be merged when green. > > > > You were right - I forgot to submit my code review! Can see comments > > here: https://github.com/apache/james-project/pull/886 > > > > I'm working on a commit at the moment to fix some/all of the review > > comments I made - you just tell me what you like and what you don't. > Please > > just let me know the git HEAD to use and where to send the pull request > to. > I did integrate your review comments to > https://github.com/chibenwa/james-project/tree/netty4-combo already. > > Please feel free to further comment or point me to some commits I can > cherry-pick if I did not apply your suggestions well... > > Regards, > > Benoit > > > > Thanks in advance, > > > > Glennos > > > > > > On Mon, 21 Mar 2022 at 13:22, Benoit TELLIER <btell...@linagora.com> > wrote: > > > >> On 21/03/2022 06:46, Glennos wrote: > >>> Follow up question: when are you planning on merging 'JAMES-3715: > Upgrade > >>> to Netty 4.1' to master and releasing? > >> Good question! > >> > >> As explained in https://github.com/apache/james-project/pull/886 We are > >> waiting to have a grren build on > >> https://github.com/apache/james-project/pull/908 too as it fixes much > of > >> the issues we spotted with the original contribution. > >> > >> To be fairly honest I likely should merge the two PRs in one to make > >> this matter of fact explicit. > >> > >> Regards > >>> > >>> On Mon, 21 Mar 2022 at 10:41, Glennos <glennos...@gmail.com> wrote: > >>> > >>>> Hi all, > >>>> > >>>> I'm new to the project - my name is Glennos, and I've been chatting > with > >>>> @chibenwa over https://gitter.im/apache/james-project - however he > >>>> suggested I join the mailing list, so that's what I'm doing! So hi > >> everyone! > >>>> I'd like to contribute to this pull request: JAMES-3715: Upgrade to > >> Netty > >>>> 4.1; > >>>> https://github.com/apache/james-project/pull/886 > >>>> https://issues.apache.org/jira/projects/JAMES/issues/JAMES-3715 > >>>> > >>>> I've reviewed the changes made so far - and I've left a whole bunch of > >>>> review comments at the end of > >>>> https://github.com/apache/james-project/pull/886 > >>>> (however you seem to need to be a project maintainer or maybe just > >>>> logged in to see them) > >>>> > >>>> Feel free to suggest if you want me to make all the suggested changes > - > >> or > >>>> even none - doesn't especially bother me. However I have noted some > >> minor > >>>> bugs/potential issues that I do think would be valuable to fix. > >>>> > >>>> I'd also like to improve the 'Encryption' class to support *both* > >>>> SSLContext and SslContext. I've already implemented this and other > >> fixes, *just > >>>> need to know where to contribute them*; > >>>> 1. *It would be handy if someone could create a 'netty4' branch or > >>>> similar on https://github.com/apache/james-project/ > >>>> <https://github.com/apache/james-project/>* > >>>> 2. Alternatively should I fork or can I directly contribute to > >>>> https://github.com/ff-wl/james-project/tree/netty4 ? (not preferred, > >> much > >>>> rather we all contribute to a branch on the main > >> 'apache/james-project/') > >>>> Thanks in advance, > >>>> > >>>> Regards, > >>>> > >>>> Glennos > >>>> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > >> For additional commands, e-mail: server-dev-h...@james.apache.org > >> > >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org > For additional commands, e-mail: server-dev-h...@james.apache.org > >