Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TOMEE-2274 (JAXB bump); exclusions for CXF 3.3.0; fix those existing build WARNINGs #419

Merged
merged 17 commits into from Mar 1, 2019
Merged

TOMEE-2274 (JAXB bump); exclusions for CXF 3.3.0; fix those existing build WARNINGs #419

merged 17 commits into from Mar 1, 2019

Conversation

j4fm
Copy link
Contributor

@j4fm j4fm commented Feb 26, 2019

All local build tests succeeded

Copy link
Contributor

@cesarhernandezgt cesarhernandezgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added observations and issues on the files/modules I tested locally.

I notice that the execution of mvn clean install on the polling-parent project fails with the web module test: https://github.com/apache/tomee/tree/master/examples/polling-parent.
The pooling-parent project works without issues on current master.
I'm attaching the log output with the errors:
log.log

examples/bval-evaluation-redeployment/pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
Reverted test comment
Fix exclusions miss-placement
Revert CXF/MP-restclient bump as that is in PR 409
Copy link
Contributor

@cesarhernandezgt cesarhernandezgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @j4fm ,
Thanks for the update. I just found one issue related with examples/bval-evaluation-redeployment/runner/pom.xml. I left a comment and a log file I got on my local env.

Copy link
Contributor

@cesarhernandezgt cesarhernandezgt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 great contribution!

@jgallimore jgallimore merged commit f57e6ae into apache:master Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants