iwasakims commented on PR #1212:
URL: https://github.com/apache/bigtop/pull/1212#issuecomment-1868977576

   @JiaLiangC 
   > We could save the mvn patch file in the template directory. For components 
that support concurrent compilation, we can simply copy it over.
   
   It would be nice if this can cover both rpm and deb. I prefer directly 
generating the maven.config patch (in package.gradle from string template) to 
modifiying a file with regex.
   
   
   > To determine which components support concurrent compilation, we might 
need to add an attribute in the bigtop.bom file to mark those components that 
support this feature.
   
   I think just using `project.hasProperty('compileThreads')` as done in 
current patch would be enough with comments (and commented out defaults) in 
bigtop.bom.
   


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