enapps-enorman commented on PR #10:
URL: 
https://github.com/apache/sling-org-apache-sling-discovery-impl/pull/10#issuecomment-2427841703

   > @enapps-enorman, thanks for the hint. It works locally after adding 
`${jacoco.ut.command}` to `argLine`. The report is generated under 
`target/site`, but it doesn't seem to work on Jenkins. The build fails with 
this change...
   
   @mreutegg Sorry about that.  It looks like the jenkins build is now failing 
on the attempts that do not use the jacoco-report profile because the 
jacoco.ut.command system property was never defined in that scenario.  That 
jenkins build uses the jacoco-report profile only for the java11 variant.  For 
the other java17 and java21 variants it does not do the code coverage analysis 
again so it does not try to use the jacoco-report profile.
   
   So, I guess it will need a more elaborate solution.  
   
   The build seems to work for me in both scenarios if I just remove the 
`<argLine>-Xmx2048m</argLine>` configuration.  But I don't know enough about 
this project to know if changing the max memory is important for some reason.
   
   
   
   


-- 
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]

Reply via email to