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

Remove servlet and core FP dependencies #13924

Merged
merged 6 commits into from Feb 24, 2021

Conversation

jfdenise
Copy link
Contributor

@jfdenise jfdenise commented Jan 26, 2021

Issues:
https://issues.redhat.com/browse/WFLY-14169
https://issues.redhat.com/browse/WFLY-14170
https://issues.redhat.com/browse/WFLY-14362
https://issues.redhat.com/browse/WFLY-14363

@bstansberry , In addition to the automated tests, I have done a diff of the various distributions as well as the provisioned layers (layers tests), and all diffs are expected (related to some html and xml licenses that don't exist anymore).

Component upgrade required to properly handle core-tools:
https://issues.redhat.com/browse/WFLY-14374
diff: wildfly/galleon-plugins@5.0.0.Final...5.1.0.Final

Bootable JAR JDK 8
https://ci.wildfly.org/buildConfiguration/WF_BootableJarLinuxJdk8/239565
Bootable JAR EE9
https://ci.wildfly.org/buildConfiguration/WF_Ee9BootableJarLinuxJdk8/239566
Bootable JAR JDK 11 Windows
https://ci.wildfly.org/buildConfiguration/WF_BootableJarWindowsJdk11/239568

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Jan 26, 2021
@jfdenise jfdenise force-pushed the separated-fp-build branch 6 times, most recently from 51a54bd to 56496d7 Compare January 28, 2021 17:35
@jfdenise
Copy link
Contributor Author

@bstansberry , this PR can be reviewed.

@jfdenise jfdenise force-pushed the separated-fp-build branch 2 times, most recently from 077e1c6 to 683e4f9 Compare February 8, 2021 15:12
@bstansberry
Copy link
Contributor

/retest

1 similar comment
@bstansberry
Copy link
Contributor

/retest

This was referenced Feb 22, 2021
bstansberry and others added 6 commits February 24, 2021 09:23
…-servlet

One aspect of this is moving the input files for license.xml generation from src/license to src/main/resource/license so it can be handled consistently with other resource files. This change was implemented for all feature packs, not just servlet.

There was also some cleanup of direct declaration of the org.wildfly.core:wildfly-core-galleon-pack, which is not really on-topic for this JIRA. It was done trying to resolve problems but in the end was probably not relevant to the solution. But it's the right thing to do so I left it.
@bstansberry bstansberry added 23.x Critical Doesn't block a release but deserves higher priority than most. Use sparingly. labels Feb 24, 2021
@bstansberry bstansberry merged commit 40275ac into wildfly:master Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Critical Doesn't block a release but deserves higher priority than most. Use sparingly. deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
2 participants