[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311498
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311500
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311499
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] emecas opened a new pull request #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
emecas opened a new pull request #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311505
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311504
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for mp-rest-client

2019-12-05 Thread GitBox
asf-ci commented on issue #618: [TOMEE-2748] Add Spanish translation for 
mp-rest-client
URL: https://github.com/apache/tomee/pull/618#issuecomment-562311503
 
 
   Can one of the admins verify this patch?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


Re: Releases

2019-12-05 Thread Jenkins, Rodney J (Rod)
Thank you for the response.  I am fine with this approach.  But, I am not the 
one that will be affected by it.

Rod.

 
On 12/5/19, 4:21 AM, "Jonathan Gallimore"  wrote:

  
I'll review the thread again, but the compromise I'd suggest is moving
forward with what we have now, and documenting how to use those images in
different scenarios, such as Carl's. I suspect that some small changes will
need to be added using the TomEE image as the base.

What do you think?

Jon

On Thu, Dec 5, 2019 at 2:45 AM Jenkins, Rodney J (Rod) <
jenki...@nationwide.com> wrote:

> Jon,
>
> When that is done, I’ll release new Docker images.
>
> Which leads me to the next point.  We need a decision on how we want to
> move forward with the images.
>
> Thanks,
> Rod.
>
> 
> From: Jonathan Gallimore 
> Sent: Wednesday, December 4, 2019 5:01:38 PM
> To: dev@tomee.apache.org 
> Subject: [EXTERNAL] Releases
>
> Nationwide Information Security Warning: This is an external email. Do not
> click on links or open attachments unless you trust the sender.
>
> 
--
>
> Just a heads up that I'm going to roll releases for 7.0.7, 7.1.2 and 
8.0.1.
> Shout if you need me to stop.
>
> Thanks
>
> Jon
>




Re: Releases

2019-12-05 Thread Alex The Rocker
+1 good that you unlocked the situation for this tomcat dependency
change (hoping that some time later i'll find a way to properly create
branches on 7.1.x and 7.0.x : nothing worked for me so far...)

Alexandre

Le jeu. 5 déc. 2019 à 11:20, Jonathan Gallimore
 a écrit :
>
> Done - thanks Alexandre! I merged your change to both 7.1.x and 7.0.x.
>
> Thanks
>
> Jon
>
> On Thu, Dec 5, 2019 at 7:53 AM Alex The Rocker  wrote:
>
> > Hello Jon,
> >
> > Would it be possible to include my PR to update tomcat dependency with
> > TOMEE-2746 ?
> >
> > Please note that I was only able to push this change for tomee-7.1.x -
> > see my comment in this JIRA : maybe would it be nice to have same
> > update on 7.0.x for upcoming 7.0.7 release ?
> >
> > Kind regards,
> > Alexandre
> >
> > Le jeu. 5 déc. 2019 à 00:02, Jonathan Gallimore
> >  a écrit :
> > >
> > > Just a heads up that I'm going to roll releases for 7.0.7, 7.1.2 and
> > 8.0.1.
> > > Shout if you need me to stop.
> > >
> > > Thanks
> > >
> > > Jon
> >


Re: Releases

2019-12-05 Thread Jonathan Gallimore
I'll review the thread again, but the compromise I'd suggest is moving
forward with what we have now, and documenting how to use those images in
different scenarios, such as Carl's. I suspect that some small changes will
need to be added using the TomEE image as the base.

What do you think?

Jon

On Thu, Dec 5, 2019 at 2:45 AM Jenkins, Rodney J (Rod) <
jenki...@nationwide.com> wrote:

> Jon,
>
> When that is done, I’ll release new Docker images.
>
> Which leads me to the next point.  We need a decision on how we want to
> move forward with the images.
>
> Thanks,
> Rod.
>
> 
> From: Jonathan Gallimore 
> Sent: Wednesday, December 4, 2019 5:01:38 PM
> To: dev@tomee.apache.org 
> Subject: [EXTERNAL] Releases
>
> Nationwide Information Security Warning: This is an external email. Do not
> click on links or open attachments unless you trust the sender.
>
> --
>
> Just a heads up that I'm going to roll releases for 7.0.7, 7.1.2 and 8.0.1.
> Shout if you need me to stop.
>
> Thanks
>
> Jon
>


Re: Releases

2019-12-05 Thread Jonathan Gallimore
Done - thanks Alexandre! I merged your change to both 7.1.x and 7.0.x.

Thanks

Jon

On Thu, Dec 5, 2019 at 7:53 AM Alex The Rocker  wrote:

> Hello Jon,
>
> Would it be possible to include my PR to update tomcat dependency with
> TOMEE-2746 ?
>
> Please note that I was only able to push this change for tomee-7.1.x -
> see my comment in this JIRA : maybe would it be nice to have same
> update on 7.0.x for upcoming 7.0.7 release ?
>
> Kind regards,
> Alexandre
>
> Le jeu. 5 déc. 2019 à 00:02, Jonathan Gallimore
>  a écrit :
> >
> > Just a heads up that I'm going to roll releases for 7.0.7, 7.1.2 and
> 8.0.1.
> > Shout if you need me to stop.
> >
> > Thanks
> >
> > Jon
>


[GitHub] [tomee] jgallimore commented on issue #617: Tomee 2746

2019-12-05 Thread GitBox
jgallimore commented on issue #617: Tomee 2746
URL: https://github.com/apache/tomee/pull/617#issuecomment-562061573
 
 
   Done:
   
   
https://github.com/apache/tomee/commit/64aa743bd0131d3ee1939121c349d886e83d558a
   
https://github.com/apache/tomee/commit/e0b510a07a1ef86aa54418066d083f114871ecb3
   
   Many thanks!


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] jgallimore closed pull request #617: Tomee 2746

2019-12-05 Thread GitBox
jgallimore closed pull request #617: Tomee 2746
URL: https://github.com/apache/tomee/pull/617
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] jgallimore commented on issue #617: Tomee 2746

2019-12-05 Thread GitBox
jgallimore commented on issue #617: Tomee 2746
URL: https://github.com/apache/tomee/pull/617#issuecomment-562058989
 
 
   Looks like this is pointing at the wrong branch as @rzo1 notes. I'll pull 
commit 5a8615f over to 7.1.x and backport to 7.0.x.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] rzo1 commented on issue #617: Tomee 2746

2019-12-05 Thread GitBox
rzo1 commented on issue #617: Tomee 2746
URL: https://github.com/apache/tomee/pull/617#issuecomment-562021864
 
 
   Hi,
   
   I think, that the target branch for this PR should be `7.1.x` and **not** 
the `master` as this branch uses tomcat 9.
   
   In addition, it might be possible, that your forked repo is a bit out of 
sync. I would recommend, that you merge the latest changes of the related 
branches into your fork and then reopen the PR targeting the right branch in 
this repo.
   
   Best,
   Richard


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [tomee] rzo1 edited a comment on issue #617: Tomee 2746

2019-12-05 Thread GitBox
rzo1 edited a comment on issue #617: Tomee 2746
URL: https://github.com/apache/tomee/pull/617#issuecomment-562021864
 
 
   Hi,
   
   thanks for this PR!
   
   I think, that the target branch for this PR should be `7.1.x` and **not** 
the `master` as this branch uses tomcat 9.
   
   In addition, it might be possible, that your forked repo is a bit out of 
sync. I would recommend, that you merge the latest changes of the related 
branches into your fork and then reopen the PR targeting the right branch in 
this repo.
   
   Best,
   Richard


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services