[GitHub] struts pull request: WW-4584: Upgrade tiles plugin

2016-01-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/73 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

Re: Struts 2.5 BETA3

2016-01-18 Thread Lukasz Lenart
If no objections I am going prepare BETA3 release 2016-01-11 14:53 GMT+01:00 Christoph Nenning : >> @Christoph I was able to resolve the issue with that warning about >> Custom EL ... also fixed the problem with loading ftl layouts - you >> must define a proper

Re: From Confluence to Markdown

2016-01-18 Thread Lukasz Lenart
2016-01-14 16:31 GMT+01:00 Rene Gielen : > I've made some progress on this as time permits, but still a lot to be > done. I'm convinced that moving to MarkDown / AsciiDoctor would be a > HUGE improvement both for our docs in general as well as our publishing > workflows.

[GitHub] struts pull request: WW-4584: Upgrade tiles plugin

2016-01-18 Thread cnenning
Github user cnenning commented on the pull request: https://github.com/apache/struts/pull/73#issuecomment-172460318 > Simplifies logging Still not final :wink: ? Let's merge it! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] struts-site pull request: Fixing broken link to Java EE tutorial.

2016-01-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/struts-site/pull/3 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: WW-4584: Upgrade tiles plugin

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/73#issuecomment-172462730 I have got some comments on my questions sent to Tiles Users mailing list and the current implementation isn't optimal ;-) But I have no better idea so let's merge

[GitHub] struts pull request: propose fix for ww-4456

2016-01-18 Thread victorsosa
GitHub user victorsosa opened a pull request: https://github.com/apache/struts/pull/74 propose fix for ww-4456 HI Lukasz, This is a proposal to fix WW-4456 - Got NPE when File Upload Limitation exceeds in portal env. We should catch in any case a

Build failed in Jenkins: Struts-JDK8-master #77

2016-01-18 Thread Apache Jenkins Server
See Changes: [Lukasz Lenart] Adds required dependencies [Lukasz Lenart] Re-implements tiles integration based on new API [Lukasz Lenart] Drops unused classes [Lukasz Lenart] Replaces call to deprecated api [Lukasz Lenart] Updates

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread aleksandr-m
GitHub user aleksandr-m opened a pull request: https://github.com/apache/struts/pull/75 WW-4590 - Allow to use multiple names in result Allow to use multiple values in `result` tag `name` attribute and in `Result` annotation `name`. So this typical configuration:

[GitHub] struts pull request: update pull

2016-01-18 Thread victorsosa
Github user victorsosa closed the pull request at: https://github.com/apache/struts/pull/78 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: update pull

2016-01-18 Thread victorsosa
GitHub user victorsosa opened a pull request: https://github.com/apache/struts/pull/78 update pull You can merge this pull request into a Git repository by running: $ git pull https://github.com/victorsosa/struts patch Alternatively you can review and apply these changes as

Jenkins build is back to normal : Struts-JDK7-pull-request #129

2016-01-18 Thread Apache Jenkins Server
See - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org

[GitHub] struts pull request: propose fix for ww-4456

2016-01-18 Thread victorsosa
Github user victorsosa commented on the pull request: https://github.com/apache/struts/pull/74#issuecomment-172694546 Closed; Just got contaminated with other changes --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] struts pull request: propose fix for ww-4456

2016-01-18 Thread victorsosa
Github user victorsosa closed the pull request at: https://github.com/apache/struts/pull/74 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: WW-4412 Context started without being closed

2016-01-18 Thread victorsosa
GitHub user victorsosa opened a pull request: https://github.com/apache/struts/pull/76 WW-4412 Context started without being closed WW-4412 Context started without being closed You can merge this pull request into a Git repository by running: $ git pull

Build failed in Jenkins: Struts-JDK7-pull-request #125

2016-01-18 Thread Apache Jenkins Server
See -- GitHub pull request #76 to apache/struts [EnvInject] - Loading node environment variables. Building remotely on ubuntu3 (Ubuntu ubuntu legacy-ubuntu) in workspace

[GitHub] struts pull request: WW-4412 Context started without being closed

2016-01-18 Thread victorsosa
Github user victorsosa closed the pull request at: https://github.com/apache/struts/pull/76 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: WW-4412 Context started without being closed

2016-01-18 Thread victorsosa
Github user victorsosa commented on the pull request: https://github.com/apache/struts/pull/76#issuecomment-172679977 Closed this pull request is right; I will send another one --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Build failed in Jenkins: Struts-JDK7-pull-request #126

2016-01-18 Thread Apache Jenkins Server
See -- GitHub pull request #77 to apache/struts [EnvInject] - Loading node environment variables. Building remotely on ubuntu3 (Ubuntu ubuntu legacy-ubuntu) in workspace

[GitHub] struts pull request: WW-4412 Context started without being closed

2016-01-18 Thread victorsosa
Github user victorsosa closed the pull request at: https://github.com/apache/struts/pull/77 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] struts pull request: WW-4412 Context started without being closed

2016-01-18 Thread victorsosa
GitHub user victorsosa opened a pull request: https://github.com/apache/struts/pull/77 WW-4412 Context started without being closed Patch WW-4412 Context started without being closed Now the context is closed after been used. You can merge this pull request

Build failed in Jenkins: Struts-JDK7-pull-request #128

2016-01-18 Thread Apache Jenkins Server
See -- GitHub pull request #78 to apache/struts [EnvInject] - Loading node environment variables. Building remotely on ubuntu3 (Ubuntu ubuntu legacy-ubuntu) in workspace

Jenkins build is back to normal : Struts-JDK7-pull-request #127

2016-01-18 Thread Apache Jenkins Server
See - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org

[GitHub] struts pull request: propose fix for ww-4456

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/74#issuecomment-172753439 The simplest approach it to not reverse merge `master` into your PRs till they are green. --- If your project is set up for it, you can reply to this email and

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on a diff in the pull request: https://github.com/apache/struts/pull/75#discussion_r50076520 --- Diff: core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java --- @@ -777,11 +777,21 @@ protected Class

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/75#issuecomment-172754377 Great idea! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread aleksandr-m
Github user aleksandr-m commented on a diff in the pull request: https://github.com/apache/struts/pull/75#discussion_r50079358 --- Diff: core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java --- @@ -777,11 +777,21 @@ protected Class

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on a diff in the pull request: https://github.com/apache/struts/pull/75#discussion_r50081302 --- Diff: core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java --- @@ -777,11 +777,21 @@ protected Class

[GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...

2016-01-18 Thread lukaszlenart
Github user lukaszlenart commented on a diff in the pull request: https://github.com/apache/struts/pull/75#discussion_r50080211 --- Diff: core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java --- @@ -777,11 +777,21 @@ protected Class