Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
it is ready
---
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
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
I have moved portlet-app away from the core project, please update your PR,
it should work now :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user cnenning commented on the issue:
https://github.com/apache/struts/pull/98
> I think we should move portlet-app away from Struts into Struts Examples
Oh, yes! We really should!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
It fails the same, I was trying many different things. I think we should
move `portlet-app` away from Struts into Struts Examples (as other apps as well)
---
If your project is set up for it,
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
@lukaszlenart can you check please the result in jenkins because I am
getting success build for Portlet here in my PC.
But here it gets
`unexpected status code [500] at URL:
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
Yep it was too fast; that fixed it.
---
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
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
Maybe It is not a good idea because this test is failing and I don't have
the clue about what is wrong. Do you know why?
>
> Tests run: 1659, Failures: 1, Errors: 0, Skipped: 0, Time
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
Yes, please
---
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
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
Here is the fix:
```
org.apache.maven.plugins
maven-surefire-plugin
2.19.1
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
NOp :)
---
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
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
Uf... I thought I'm betting crazy ;-)
---
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
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
Sorry I compiled the wrong branch, I was able to duplicate the issue
---
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
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
> openjdk version "1.8.0_91"
> OpenJDK Runtime Environment (build 1.8.0_91-b14)
> OpenJDK 64-Bit Server VM (build 25.91-b14, mixed mode)
---
If your project is set up for it, you can
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
and this is related to outdated version of
`org.apache.maven.plugins:maven-surefire-plugin:2.3.1`
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user victorsosa commented on the issue:
https://github.com/apache/struts/pull/98
On my end.
[INFO] Installing
/home/vns/workspace/struts/plugins/testng/target/struts2-testng-plugin-2.5.1-SNAPSHOT-sources.jar
to
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
And changing in `plugins/testng/pom.xml` is not enough, this dependency is
used in few other palces.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
I've got such an error when building locally:
`INFO] Struts 2 TestNG Plugin . FAILURE [
7.039 s]`
```
Github user lukaszlenart commented on the issue:
https://github.com/apache/struts/pull/98
I'm just wondering about those all merge commits, did you update your fork?
https://help.github.com/articles/syncing-a-fork/
---
If your project is set up for it, you can reply to this
18 matches
Mail list logo