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

[WFLY-12563] org.jboss.jbossfaces.WAR_BUNDLES_JSF_IMPL flag ignored w… #12714

Merged
merged 1 commit into from Oct 18, 2019

Conversation

tmiyargi
Copy link
Contributor

…hen WARs are embedded in EAR

Issue: https://issues.jboss.org/browse/WFLY-12563

@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 16, 2019
@bstansberry
Copy link
Contributor

@fjuma Please review.

@fjuma
Copy link
Contributor

fjuma commented Oct 16, 2019

@tmiyargi Thanks for working on this! The fix looks ok. Did you do a quick sanity test just to make sure this doesn't re-introduce the problem from https://issues.jboss.org/browse/WFLY-10796? (I don't think it will but it's good to make sure.)

The failure that happens in JSFDeploymentProcessorTestCase with the security manager enabled will need to be fixed.

@tmiyargi
Copy link
Contributor Author

@fjuma thanks, PR updated and included testing for that https://issues.jboss.org/browse/WFLY-10796

@bstansberry bstansberry merged commit f51b4c0 into wildfly:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
4 participants