Skip to content

Commit f95438f

Browse files
authored
refactor(colcon-build): nice-command to build-pre-command (#310)
Signed-off-by: M. Fatih Cırıt <[email protected]>
1 parent 0dbf0d4 commit f95438f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

colcon-build/README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
| token | false | The token for build dependencies. |
4040
| include-eol-distros | false | If true, adds `--include-eol-distros` to `rosdep update`. |
4141
| cache-key-element | false | This value is added to the github actions cache key. |
42-
| nice-command | false | This command is prepended to the `colcon build` to avoid draining resources. |
42+
| build-pre-command | false | This command is prepended to the `colcon build` to avoid draining resources. |
4343
| colcon-parallel-workers-flag | false | Will be appended to the colcon build command to limit number of packages built in parallel. e.g. "--parallel-workers 3" |
4444
| makeflags | false | Will be exported as MAKEFLAGS environment variable for colcon build step. e.g. "-j 4" |
4545

colcon-build/action.yaml

+3-3
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ inputs:
2626
description: Will be part of the cache key
2727
default: default
2828
required: false
29-
nice-command:
30-
description: See `man nice` for details
29+
build-pre-command:
30+
description: Will be prepended to the colcon build command. e.g. "nice -n 19" or "taskset --cpu-list 0-2"
3131
required: false
3232
default: nice -n 19
3333
colcon-parallel-workers-flag:
@@ -94,7 +94,7 @@ runs:
9494
cat /etc/nsswitch.conf
9595
MAKEFLAGS="${{ inputs.makeflags }}"
9696
echo "MAKEFLAGS=$MAKEFLAGS" # for debugging
97-
${{ inputs.nice-command }} colcon build ${{ inputs.colcon-parallel-workers-flag }} \
97+
${{ inputs.build-pre-command }} colcon build ${{ inputs.colcon-parallel-workers-flag }} \
9898
--event-handlers console_cohesion+ \
9999
--packages-above-and-dependencies ${{ inputs.target-packages }} \
100100
--cmake-args -DCMAKE_BUILD_TYPE=${{ inputs.cmake-build-type }} \

0 commit comments

Comments
 (0)