Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Conversation

@tjroamer
Copy link
Contributor

@tjroamer tjroamer commented Apr 17, 2021

The long classpath is a known issue in the maven plug-in appassembler-maven-plugin. The description of the issue is here:
https://www.mojohaus.org/appassembler/appassembler-maven-plugin/faq.html#

This pull request solved this issue by implementing the "Booter" mechanism, which is described here:
https://www.mojohaus.org/appassembler/appassembler-maven-plugin/usage-booter.html

The generated any23.bat does not contain the long "set CLASSPATH=..." any more. The generated script runs on CMD even if the base path is very long.

any23 bat

@lewismc
Copy link
Member

lewismc commented Apr 18, 2021

@afs can you please check? I can't and wouldn't even trust the outcome if I did. I have not used Windows OS for a long time. Thank you both :)

@afs
Copy link
Member

afs commented Apr 18, 2021

Sorry, I don't have a Windows machine.

@lewismc
Copy link
Member

lewismc commented May 12, 2021

I've verified that this does not break compatibility on macOS. Going to merge into master branch.

@lewismc lewismc merged commit 978d2a0 into apache:master May 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants