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

SLING-11408 m-assembly-p must use dependencies generated from #42

Merged

Conversation

kwin
Copy link
Member

@kwin kwin commented Aug 4, 2022

appassembler

@cziegeler
Copy link
Contributor

Thanks, this makes the launcher now start, but all logs from the launcher are now gone, so I guess there is something wrong with slf4j dependencies

@cziegeler
Copy link
Contributor

ok, so additional problems:

  1. slf4j-impl is missing
  2. the launch script now lists all jar files from libs as the classpath, whereas previously a wildcard was used, which allowed to just add new jars in the libs directory

@kwin
Copy link
Member Author

kwin commented Aug 4, 2022

Probably

<scope>provided</scope>
needs to have compile or runtime scope. Appassembler by default only considers Maven runtime scope.

@cziegeler
Copy link
Contributor

I fixed the two issues

@sonarcloud
Copy link

sonarcloud bot commented Aug 4, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@cziegeler cziegeler merged commit 9ee7aea into master Aug 4, 2022
@cziegeler cziegeler deleted the bugfix/m-assembly-plugin-use-libs-from-appassembler branch August 4, 2022 13:19
@@ -256,7 +257,7 @@
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<scope>provided</scope>
<scope>compile</scope>
Copy link
Member Author

Choose a reason for hiding this comment

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

I would rather have this scope runtime, as definitely not needed in the compile class path.

Copy link
Contributor

Choose a reason for hiding this comment

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

Feel free to adjust if it is still working then

Copy link
Member Author

Choose a reason for hiding this comment

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

Done in b55ca9c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants