Skip to content

Add GitHub workflow for Windows CI #109

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: dashing
Choose a base branch
from
Open

Add GitHub workflow for Windows CI #109

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7701ed5
Add GitHub workflow for Windows CI
jacobperron May 1, 2020
b012698
Try bumping version of setup-ros action
jacobperron Jun 8, 2020
fa60dba
Try bumping version of action-ros-ci
jacobperron Jun 8, 2020
4f842ed
Remove quotation marks from repo file
jacobperron Jun 8, 2020
56640df
Enable linters explicitly instead of using ament_lint_common
jacobperron Jun 8, 2020
e6495c7
Fix file path separator for Windows
jacobperron Jun 8, 2020
257c376
Remove quotation marks
jacobperron Jun 9, 2020
b4fd7e5
Fix Windows problem not finding ASIO. (#116)
clalancette Jun 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 25 additions & 5 deletions .github/workflows/build_and_test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,20 +1,40 @@
name: CI

on: [push]
on: [push, pull_request]

jobs:
build_and_test:
linux_build_and_test:
runs-on: ubuntu-18.04
steps:
- uses: actions/checkout@v2
- name: Install Java
run: |
sudo apt update -qq
sudo apt install -y default-jdk
- uses: ros-tooling/[email protected].14
- uses: ros-tooling/[email protected].24
with:
required-ros-distributions: dashing
- uses: ros-tooling/action-ros-ci@8d58122
- uses: ros-tooling/action-ros-ci@658ebdf
with:
package-name: rosidl_generator_java rcljava_common rcljava
source-ros-binary-installation: dashing
vcs-repo-file-url: https://raw.githubusercontent.com/ros2-java/ros2_java/dashing/ros2_java_desktop.repos
vcs-repo-file-url: "${{ github.workspace }}/ros2_java_desktop.repos"
windows_build_and_test:
runs-on: windows-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-java@v1
with:
java-version: '11.0.6' # The JDK version to make available on the path.
java-package: jdk
architecture: x64
- uses: ros-tooling/[email protected]
with:
required-ros-distributions: dashing
- uses: ros-tooling/action-ros-ci@658ebdf
with:
colcon-extra-args: --merge-install
package-name: rosidl_generator_java rcljava_common rcljava
vcs-repo-file-url: |
https://raw.githubusercontent.com/ros2/ros2/dashing/ros2.repos
${{ github.workspace }}\ros2_java_desktop.repos
4 changes: 3 additions & 1 deletion ament_cmake_export_jars/package.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,10 @@
<build_depend>ament_java_resources</build_depend>
<exec_depend>ament_java_resources</exec_depend>

<test_depend>ament_cmake_copyright</test_depend>
<test_depend>ament_cmake_lint_cmake</test_depend>
<test_depend>ament_cmake_xmllint</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>

<export>
<build_type>ament_cmake</build_type>
Expand Down
4 changes: 3 additions & 1 deletion ament_cmake_export_jni_libraries/package.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,10 @@
<build_depend>ament_java_resources</build_depend>
<exec_depend>ament_java_resources</exec_depend>

<test_depend>ament_cmake_copyright</test_depend>
<test_depend>ament_cmake_lint_cmake</test_depend>
<test_depend>ament_cmake_xmllint</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>

<export>
<build_type>ament_cmake</build_type>
Expand Down
6 changes: 5 additions & 1 deletion rcljava/package.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,12 @@
<exec_depend>rosidl_generator_c</exec_depend>
<exec_depend>rosidl_parser</exec_depend>

<test_depend>ament_cmake_copyright</test_depend>
<test_depend>ament_cmake_cppcheck</test_depend>
<test_depend>ament_cmake_lint_cmake</test_depend>
<test_depend>ament_cmake_uncrustify</test_depend>
<test_depend>ament_cmake_xmllint</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>
<test_depend>builtin_interfaces</test_depend>
<test_depend>rcl_interfaces</test_depend>
<test_depend>rcljava_common</test_depend>
Expand Down
6 changes: 5 additions & 1 deletion rcljava_common/package.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,12 @@
<buildtool_depend>ament_cmake_export_libraries</buildtool_depend>
<buildtool_export_depend>rosidl_cmake</buildtool_export_depend>

<test_depend>ament_cmake_copyright</test_depend>
<test_depend>ament_cmake_cppcheck</test_depend>
<test_depend>ament_cmake_lint_cmake</test_depend>
<test_depend>ament_cmake_uncrustify</test_depend>
<test_depend>ament_cmake_xmllint</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>

<export>
<build_type>ament_cmake</build_type>
Expand Down
9 changes: 8 additions & 1 deletion rosidl_generator_java/package.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,16 @@
<exec_depend>rosidl_generator_c</exec_depend>
<exec_depend>rosidl_parser</exec_depend>

<test_depend>ament_cmake_copyright</test_depend>
<test_depend>ament_cmake_cppcheck</test_depend>
<!-- TODO(jacobperron): Re-enable after https://github.com/ament/ament_lint/pull/252 -->
<!--test_depend>ament_cmake_flake8</test_depend-->
<test_depend>ament_cmake_gtest</test_depend>
<test_depend>ament_cmake_lint_cmake</test_depend>
<test_depend>ament_cmake_pep257</test_depend>
<test_depend>ament_cmake_uncrustify</test_depend>
<test_depend>ament_cmake_xmllint</test_depend>
<test_depend>ament_lint_auto</test_depend>
<test_depend>ament_lint_common</test_depend>

<test_depend>rmw_implementation</test_depend>
<test_depend>rmw_implementation_cmake</test_depend>
Expand Down