Skip to content

Commit

Permalink
Refs #21585: Set requirements path as OS dependant matrix variable
Browse files Browse the repository at this point in the history
Signed-off-by: JesusPoderoso <[email protected]>
  • Loading branch information
JesusPoderoso committed Sep 9, 2024
1 parent f20c10c commit 8d0928d
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/reusable-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,16 @@ jobs:
os:
- version: ubuntu-22.04
aarch: 'gcc_64'
req_path: '/src/fastdds_monitor/.github/workflows/configurations/requirements.txt'
- version: ubuntu-24.04
aarch: 'gcc_64'
req_path: '/src/fastdds_monitor/.github/workflows/configurations/requirements.txt'
- version: windows-2019
aarch: 'win64_msvc2019_64'
req_path: '\src\fastdds_monitor\.github\workflows\configurations\requirements.txt'
- version: windows-2021
aarch: 'win64_msvc2019_64'
req_path: '\src\fastdds_monitor\.github\workflows\configurations\requirements.txt'

steps:

Expand Down Expand Up @@ -81,7 +85,7 @@ jobs:
- name: Install Python packages
uses: eProsima/eProsima-CI/multiplatform/install_python_packages@v0
with:
requirements_file_name: ${{ github.workspace }}/src/fastdds_monitor/.github/workflows/configurations/requirements.txt
requirements_file_name: '${{ github.workspace }}${{ matrix.os.req_path }}'

- name: Install Qt
uses: eProsima/eProsima-CI/external/install_qt@v0
Expand All @@ -107,7 +111,7 @@ jobs:
with:
packages_names: fastdds_monitor
cmake_args: -DCMAKE_BUILD_TYPE=${{ matrix.cmake_build_type }}
colcon_meta_file: ${{ github.workspace }}/src/fastdds_monitor/.github/workflows/configurations/${{ runner.os.version }}/colcon.meta
colcon_meta_file: ${{ github.workspace }}/src/fastdds_monitor/.github/workflows/configurations/${{ runner.os }}/colcon.meta
workspace: ${{ github.workspace }}

- name: Upload artifact
Expand Down

0 comments on commit 8d0928d

Please sign in to comment.