zrhoffman commented on a change in pull request #6532:
URL: https://github.com/apache/trafficcontrol/pull/6532#discussion_r811363017



##########
File path: dev/traffic_router/run.sh
##########
@@ -21,7 +21,6 @@ set -o errexit
 cd "$TC/traffic_router"
 
 mvn -Dmaven.test.skip=true compile -P \!rpm-build
-mvn -Dmaven.test.skip=true package -P \!rpm-build

Review comment:
       Looks like `mvn package` is required to generate 
`traffic_router/core/target/ROOT`. But the 2 commands can be combined to save 5 
seconds like
   ```shell
   mvn compile package
   ```




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