| Summary: | BPMS: Product release contains source code | ||
|---|---|---|---|
| Product: | [Retired] JBoss BPMS Platform 6 | Reporter: | Tomas Livora <tlivora> |
| Component: | Business Central | Assignee: | Marco Rietveld <mrietvel> |
| Status: | CLOSED EOL | QA Contact: | Lukáš Petrovický <lpetrovi> |
| Severity: | medium | Docs Contact: | |
| Priority: | low | ||
| Version: | 6.0.0 | CC: | gdesmet, kverlaen, lpetrovi, mbaluch, porcelli, rrajasek, rzhang |
| Target Milestone: | DR1 | ||
| Target Release: | 6.3.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2020-03-27 20:05:20 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | |||
| Bug Blocks: | 1026695 | ||
|
Description
Tomas Livora
2013-10-18 13:32:34 UTC
This is related to GWT needs - GWT compiler needs access to java source files in order to translate it to javascript. Some of the above .java files are not needed like pure backend ones like ApplicationScopedProducer.java and all jsp related as well. If you check, most of the jar's used by application includes source files as well. Of course would be possible to clean up war and all jars in the assembly phase... *** Bug 1026693 has been marked as a duplicate of this bug. *** Change component to project *** Bug 1026695 has been marked as a duplicate of this bug. *** The community build does not build business-central.war. It builds dashbuilder-demo.war, but that does not contain any java source files. I suspect this issue is purely in the product build. This is probably due to a legacy configured GWT build. A modern maven GWT plugin picks up the sources jars it needs automatically from it's normal jar dependencies, and it puts those sources jars in the GWT compilation path, but it does NOT pollute the war output with those sources jars or require them to be added as dependencies. Julian, can someone in your team verify that this issue is indeed only in the product build? The trick we use to avoid polluting source files in our wars, is the use of <compileSourcesArtifacts> in gwt-maven-plugin:
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>gwt-maven-plugin</artifactId>
<configuration>
...
<compileSourcesArtifacts>
<compileSourcesArtifact>org.guvnor:guvnor-inbox-api</compileSourcesArtifact>
<compileSourcesArtifact>org.guvnor:guvnor-inbox-client</compileSourcesArtifact>
...
</>
</>
</>
The jbpm-console-6.0.1.Final.war file from the community build includes Java files mentioned above too. However, temporary files are also listed in the first post. The actual list of all relevant Java files: ./WEB-INF/classes/org/kie/workbench/backend/UberFireIdentityProvider.java ./WEB-INF/classes/org/kie/workbench/backend/ApplicationScopedProducer.java ./WEB-INF/classes/org/kie/workbench/backend/AppSetup.java ./WEB-INF/classes/org/kie/workbench/client/navbar/ComplementNavAreaPresenter.java ./WEB-INF/classes/org/kie/workbench/client/navbar/AppNavBar.java ./WEB-INF/classes/org/kie/workbench/client/navbar/LogoWidgetView.java ./WEB-INF/classes/org/kie/workbench/client/navbar/LogoWidgetPresenter.java ./WEB-INF/classes/org/kie/workbench/client/navbar/ComplementNavAreaView.java ./WEB-INF/classes/org/kie/workbench/client/KieWorkbenchEntryPoint.java ./WEB-INF/classes/org/kie/workbench/client/security/KieWorkbenchFeatures.java ./WEB-INF/classes/org/kie/workbench/client/resources/images/AppImages.java ./WEB-INF/classes/org/kie/workbench/client/resources/i18n/AppConstants.java ./WEB-INF/classes/org/kie/workbench/client/resources/AppResource.java ./WEB-INF/classes/org/kie/workbench/client/home/HomeProducer.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/AdministrationPerspectiveActivity.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/DroolsAuthoringPerspectiveActivity.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/DroolsAuthoringPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/AdministrationPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/M2RepoPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/M2RepoPerspectiveActivity.java All platform has been upgraded today, and such upgrade enabled us to remove the source code from ALL our libraries. https://github.com/uberfire/uberfire/commit/70fffe151488dbd44cd2dec2baafdd36e99e1f84 https://github.com/uberfire/uberfire-extensions/commit/517f1b3357b5c4c21a9afd12ab2074b5853e3fa8 https://github.com/dashbuilder/dashbuilder/commit/1c8f283fafd710a2ea71b7e03c5af67f3f567435 https://github.com/droolsjbpm/guvnor/commit/6c10631b0f9994f68e578f1915a0af9467bae89c https://github.com/droolsjbpm/drools-wb/commit/446d2abe0ddc634e6bac4779075e1ecef0eb5b71 https://github.com/droolsjbpm/kie-wb-distributions/commit/ec3e0c5074c4635eafcf4e5f53f7ddfe7a278380 https://github.com/droolsjbpm/jbpm-form-modeler/commit/1325380ca4b25daead7e1f465a7bf385fc7c4e20 https://github.com/droolsjbpm/jbpm-designer/commit/3adf465fdfcf0b9e2c1fbbb64f9ae535bdccbf4f https://github.com/droolsjbpm/jbpm-console-ng/commit/47c0bdeff1f4f8bef8b20c2454f54b272a4fe123 https://github.com/droolsjbpm/kie-wb-common/commit/eaf008df78dae5b70340209bc38b677476efe846 https://github.com/droolsjbpm/kie-uberfire-extensions/commit/8bfbb41b34a74ab79b7f760d357937e543f53d63 I can still see most of the *.java files mentioned above in BPMS 6.2.0 ER1: ./WEB-INF/classes/org/kie/workbench/client/resources/i18n/AppConstants.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/DroolsAuthoringNoContextNavigationPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/DroolsAuthoringPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/M2RepoPerspective.java ./WEB-INF/classes/org/kie/workbench/client/perspectives/AdministrationPerspective.java ./WEB-INF/classes/org/kie/workbench/client/KieWorkbenchEntryPoint.java ./WEB-INF/classes/org/kie/workbench/client/RepositoryPreferencesProducer.java ./WEB-INF/classes/org/kie/workbench/client/navbar/ComplementNavAreaView.java ./WEB-INF/classes/org/kie/workbench/client/navbar/LogoWidgetView.java ./WEB-INF/classes/org/kie/workbench/client/navbar/AppNavBar.java ./WEB-INF/classes/org/kie/workbench/client/navbar/LogoWidgetPresenter.java ./WEB-INF/classes/org/kie/workbench/client/navbar/ComplementNavAreaPresenter.java Alexandre, in comment 3 you have mentioned that GWT needs some of these source code files. Is it still a valid reason to have them in the product build? Cannot they be removed? After discussion with Alexandre, I was told that it is possible to completely remove these Java files during the build process. So I am setting this BZ back to ASSIGNED. Fixed on master: https://github.com/droolsjbpm/kie-wb-distributions/commit/62e79f150c7b7bcfe7433005a5dfba0a1b9414a3 'find -name "*.java"' in kie-wb-distribution-wars/target and kie-drools-wb-distribution-wars/target return empty list after the changes. Verified on BPM Suite 6.3.0 DR2 |