pkumar-singh commented on pull request #2841: URL: https://github.com/apache/bookkeeper/pull/2841#issuecomment-948955504
> worked for me. it'd be nice if we could keep the 'bookkeeper' script name and factor common.sh into three parts: common, mvn specific, gradle specific. maybe for a future iteration. > > a most accurate split would be: installed jars, jars fetched from mvn repo, locally built jars. for the last, that's currently maven or gradle, but will eventually be just gradle until the next build migration. Two things. - No mixing with maven that's the first principal for gradle upgrade. Let maven be as it is till we pull the plug on maven. - Once we pull the plug on maven which should be atomic operation as much as possible. Rename this script and all other scripts to default, that is remove 'gradle' from their names. For example rename bookkeeper-gradle to bookkeeper -- 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]
