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

[incubator-kie-drools-6161] Review drools-distribution #6201

Merged

Conversation

tkobayas
Copy link
Contributor

  • Fix ReadMe and assembly

Issue:

This PR only deals with ReadMe, assembly, LICENSE and NOTICE.

We will then

  • Discuss the contents of distribution zip (or even whether it's required)
  • Review examples (remove or fix outdated ones)

Comment on lines +34 to +43
<file>
<source>license-for-distribution</source>
<outputDirectory>/</outputDirectory>
<destName>LICENSE</destName>
</file>
<file>
<source>notice-for-distribution</source>
<outputDirectory>/</outputDirectory>
<destName>NOTICE</destName>
</file>
Copy link
Contributor Author

@tkobayas tkobayas Dec 17, 2024

Choose a reason for hiding this comment

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

Added plain LICENSE and NOTICE. We don't need third-party license entries, because the distribution zip contains only example source codes.

Comment on lines -136 to -149
<dependencySet>
<includes>
<include>org.kie:*:jar:sources</include>
<include>org.drools:*:jar:sources</include>
</includes>
<excludes>
<exclude>org.drools:drools-examples:jar:sources</exclude>
</excludes>
<outputDirectory>sources</outputDirectory>
<unpack>false</unpack>
<useStrictFiltering>true</useStrictFiltering>
<useTransitiveFiltering>true</useTransitiveFiltering>
<useProjectArtifact>false</useProjectArtifact>
</dependencySet>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed source jars to avoid license review.

Users can download source distribution releases via Apache KIE website and more commonly, clone the repository via github.

Comment on lines -21 to -27
Third party jars
----------------

Following third party jars are optional and are not included in Drools distribution:

1.Quartz: Quartz is an optional jar only needed if you wish to adapt existing quartz calendars. It can be
downloaded from http://www.quartz-scheduler.org/.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Quartz is no longer used.

@tkobayas tkobayas force-pushed the incubator-kie-drools-6161-distribution branch from 5c1a060 to 40cda66 Compare December 17, 2024 06:53
@tkobayas tkobayas merged commit 4e7978e into apache:main Dec 18, 2024
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants