[GitHub] [myfaces] tandraschko merged pull request #160: MYFACES-4376

2021-01-25 Thread GitBox 


tandraschko merged pull request #160:
URL: https://github.com/apache/myfaces/pull/160


   



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

RE: 3.0.0 Release Go Ahead

2021-01-25 Thread Volodymyr Siedlecki 
Hello,
 
With MYFACES-4376  (PR #159 ) now merged, I'll begin with the release within the next day or two.
 
Thank you,
 
Volodymyr
 
- Original message -From: Bernd Bohmann To: MyFaces Development Cc:Subject: [EXTERNAL] Re: 3.0.0 Release Go AheadDate: Sun, Jan 24, 2021 3:55 AM   

 

Can we apply this https://github.com/apache/myfaces/pull/152 for 3.0.x as well? 

 
Here is the pr https://github.com/apache/myfaces/pull/159
 
Regards
 
Bernd 

On Sat, Jan 23, 2021 at 5:04 PM Bill Lucy  wrote:
+1 from me, thanks! 

On Sat, Jan 23, 2021 at 9:15 AM Thomas Andraschko  wrote:
+1 from my side 

Volodymyr Siedlecki  schrieb am Sa., 23. Jan. 2021, 14:44:
Hello,
 
 
I see that there are no active PRs and all others have been merged in. 
 
Will there be more updates done to the 3.0.x branch or am I clear to redo the release?Thank you,
 
Volodymyr
 

[GitHub] [myfaces] volosied opened a new pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied opened a new pull request #161:
URL: https://github.com/apache/myfaces/pull/161


   Trying to exclude jdks in the PRs, so only openjdk8 runs



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

[GitHub] [myfaces] bohmber commented on pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


bohmber commented on pull request #161:
URL: https://github.com/apache/myfaces/pull/161#issuecomment-767064397


   Can we exclude the linux-ppc64le build as well? 
   Hmm as far I can see we should only use the github action for pull requests. 
We are still waiting for the travis build 



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

[GitHub] [myfaces] volosied commented on pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied commented on pull request #161:
URL: https://github.com/apache/myfaces/pull/161#issuecomment-767087069


   I can try to remove  linux-ppc64le, too.
   
   Queued, but not started... Let me investigate. 



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

[GitHub] [myfaces] volosied merged pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied merged pull request #161:
URL: https://github.com/apache/myfaces/pull/161


   



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

[GitHub] [myfaces] volosied commented on pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied commented on pull request #161:
URL: https://github.com/apache/myfaces/pull/161#issuecomment-767123306


   Sorry, testing in master since builds aren't running in 3.0.x. Appears to be 
a seperate issue. 



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

[GitHub] [myfaces] volosied opened a new pull request #162: [draft] exclude jdks & linux-ppc64le from ci (master branch)

2021-01-25 Thread GitBox 


volosied opened a new pull request #162:
URL: https://github.com/apache/myfaces/pull/162


   3.0.x Travis Builds are queued and not executing. I'm not sure why, but I'm 
looking into it. 
   
   For now, I'm testing the exclusions on master branch.  



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

[GitHub] [myfaces] volosied commented on pull request #162: [draft] exclude jdks & linux-ppc64le from ci (master branch)

2021-01-25 Thread GitBox 


volosied commented on pull request #162:
URL: https://github.com/apache/myfaces/pull/162#issuecomment-767127852


   Hmm... Maybe I'm going about it wrong? 
https://travis-ci.community/t/if-doesnt-appear-to-work-in-exclude/8595/7 



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

[GitHub] [myfaces] bohmber merged pull request #159: MYFACES-4376

2021-01-25 Thread GitBox 


bohmber merged pull request #159:
URL: https://github.com/apache/myfaces/pull/159


   



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

[GitHub] [myfaces] volosied commented on pull request #159: MYFACES-4376

2021-01-25 Thread GitBox 


volosied commented on pull request #159:
URL: https://github.com/apache/myfaces/pull/159#issuecomment-766374486


   Sorry I didn't see your comment from the master PR. 
   
   From what I read, we definitely should upgrade DES.  
   
   As for HmacSHA256, I don't see any harm in updating. I  found some articles 
about collisions with SHA1, and SHA256 seems to be generally more preferred.   
I'm curious about performance, but users of MyFaces always have the option to 
change these algorithms if they so choose.   
   
   +1 from me. 



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

[GitHub] [myfaces] tandraschko merged pull request #160: MYFACES-4376

2021-01-25 Thread GitBox 


tandraschko merged pull request #160:
URL: https://github.com/apache/myfaces/pull/160


   



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

[GitHub] [myfaces] bohmber commented on pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


bohmber commented on pull request #161:
URL: https://github.com/apache/myfaces/pull/161#issuecomment-767064397


   Can we exclude the linux-ppc64le build as well? 
   Hmm as far I can see we should only use the github action for pull requests. 
We are still waiting for the travis build 



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

[GitHub] [myfaces] volosied commented on pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied commented on pull request #161:
URL: https://github.com/apache/myfaces/pull/161#issuecomment-767087069







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

[GitHub] [myfaces] volosied commented on pull request #162: [draft] exclude jdks & linux-ppc64le from ci (master branch)

2021-01-25 Thread GitBox 


volosied commented on pull request #162:
URL: https://github.com/apache/myfaces/pull/162#issuecomment-767127852


   Hmm... Maybe I'm going about it wrong? 
https://travis-ci.community/t/if-doesnt-appear-to-work-in-exclude/8595/7 



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

[GitHub] [myfaces] volosied merged pull request #161: [Draft] Testing Travis Builds in 3.0.x

2021-01-25 Thread GitBox 


volosied merged pull request #161:
URL: https://github.com/apache/myfaces/pull/161


   



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