Croway commented on PR #27826: URL: https://github.com/apache/camel/pull/27826#issuecomment-6168270945
Addressed the javadoc suggestions in both scripts and excluded both classifier JARs from shared assembly content. Regression tests include sources/javadoc files sorting before and after the executable JAR and paths with spaces. The new tests now use the surrounding JUnit assertion style. The reported Red Hat downstream distribution does contain a sources JAR in `bin/`. I agree that this does not establish that the Maven Central binary archives contain one; the script guard protects installations where auxiliary artifacts are present, and the assembly exclusions prevent packaging them. The 4.22.x scripts and assembly have the same behavior, so the companion backport remains useful. Keeping this bounded change without a new JIRA; the linked PRs track both targets. `windows-validator` now runs `CamelBatDiscoveryTest` and `FakeJavaTest` in addition to installer tests. Actual Windows execution is pending the fresh CI run. Both branches pass local launcher/installer tests, packaging, and formatting; ZIP/TAR packaging was checked with auxiliary JAR fixtures present in `target/`. The CI changes use the pinned Maven wrapper and enable `apache-snapshots` in the test steps. The 4.22.x port also includes the existing CAMEL-24485 Windows timeout/process-handling fix. _AI-generated by Codex on behalf of @Croway._ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
