Skip to content

Commit

Permalink
Refs #20091: Add macos actions to multiplaform
Browse files Browse the repository at this point in the history
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
  • Loading branch information
EduPonz committed Feb 17, 2024
1 parent eb18688 commit a06b03a
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 11 deletions.
9 changes: 6 additions & 3 deletions macos/install_python_packages/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,20 +28,23 @@ runs:
echo "::group::Install Python packages"
# Set upgrade flag
if [ "${{ inputs.upgrade }}" = "true" ]; then
if [[ "${{ inputs.upgrade }}" == "true" ]]
then
export UPGRADE_FLAG="--upgrade"
else
export UPGRADE_FLAG=""
fi
# Install python packages if any
if [[ ! -z "${{ inputs.packages }}" ]] ; then
if [[ ! -z "${{ inputs.packages }}" ]]
then
pip3 install ${UPGRADE_FLAG} -U \
${{ inputs.packages }}
fi
# Install requirements file if any
if [[ ! -z "${{ inputs.requirements_file_name }}" ]] ; then
if [[ ! -z "${{ inputs.requirements_file_name }}" ]]
then
pip3 install ${UPGRADE_FLAG} -U \
-r ${{ inputs.requirements_file_name }}
fi
Expand Down
4 changes: 2 additions & 2 deletions multiplatform/colcon_build/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ runs:
steps:

- name: Run in ubuntu or macOS
uses: eProsima/eProsima-CI/ubuntu/colcon_build@v0
uses: eProsima/eProsima-CI/ubuntu/colcon_build@main
if: runner.os == 'Linux' || runner.os == 'macOS'
with:
colcon_meta_file: ${{ inputs.colcon_meta_file }}
Expand All @@ -61,7 +61,7 @@ runs:
cmake_build_type: ${{ inputs.cmake_build_type }}

- name: Run in windows
uses: eProsima/eProsima-CI/windows/colcon_build@v0
uses: eProsima/eProsima-CI/windows/colcon_build@main
if: runner.os == 'Windows'
with:
colcon_meta_file: ${{ inputs.colcon_meta_file }}
Expand Down
8 changes: 6 additions & 2 deletions multiplatform/install_colcon/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,13 @@ runs:
steps:

- name: Run in ubuntu
uses: eProsima/eProsima-CI/ubuntu/install_colcon@main
uses: eProsima/eProsima-CI/ubuntu/install_colcon@feature/fastdds_mac_ci_support
if: runner.os == 'Linux'

- name: Run in macOS
uses: eProsima/eProsima-CI/macos/install_colcon@feature/fastdds_mac_ci_support
if: runner.os == 'macOS'

- name: Run in windows
uses: eProsima/eProsima-CI/windows/install_colcon@main
uses: eProsima/eProsima-CI/windows/install_colcon@feature/fastdds_mac_ci_support
if: runner.os == 'Windows'
12 changes: 10 additions & 2 deletions multiplatform/install_python_packages/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,15 +23,23 @@ runs:
steps:

- name: Run in ubuntu
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@main
uses: eProsima/eProsima-CI/ubuntu/install_python_packages@feature/fastdds_mac_ci_support
if: runner.os == 'Linux'
with:
packages: ${{ inputs.packages }}
upgrade: ${{ inputs.upgrade }}
requirements_file_name: ${{ inputs.requirements_file_name }}

- name: Run in macOS
uses: eProsima/eProsima-CI/macos/install_python_packages@feature/fastdds_mac_ci_support
if: runner.os == 'macOS'
with:
packages: ${{ inputs.packages }}
upgrade: ${{ inputs.upgrade }}
requirements_file_name: ${{ inputs.requirements_file_name }}

- name: Run in windows
uses: eProsima/eProsima-CI/windows/install_python_packages@main
uses: eProsima/eProsima-CI/windows/install_python_packages@feature/fastdds_mac_ci_support
if: runner.os == 'Windows'
with:
packages: ${{ inputs.packages }}
Expand Down
4 changes: 2 additions & 2 deletions multiplatform/junit_summary/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ runs:
using: composite
steps:

- name: Run in ubuntu
if: runner.os == 'Linux'
- name: Run in ubuntu or macOS
if: runner.os == 'Linux' || runner.os == 'macOS'
uses: eProsima/eProsima-CI/ubuntu/junit_summary@main
with:
junit_reports_dir: ${{ inputs.junit_reports_dir }}
Expand Down

0 comments on commit a06b03a

Please sign in to comment.