Skip to content
This repository was archived by the owner on Jan 11, 2019. It is now read-only.

Conversation

nuclearsandwich
Copy link
Member

The group dependency on a logger package prevents builds from running.

@dirk-thomas
Copy link
Member

The commit contains many unrelated items.

@nuclearsandwich
Copy link
Member Author

The commit contains many unrelated items.

yes. Fixing.

The group dependency on a logger package prevents builds from running.
@@ -813,7 +813,7 @@ repositories:
url: https://github.com/ros2-gbp/rcl-release.git
version: 0.6.2-0
source:
test_pull_requests: true
test_commits: false
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http://www.ros.org/reps/rep-0143.html has both test_pull_requests and test_commits. What's the difference?

test_commits: a boolean flag used to enable CI jobs for each commit to the branch specified under versions. (default: false)
test_pull_requests: a boolean flag used to enable CI jobs for each pull request against the branch specified under version. (default: false)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One tests each commit to the source branch, the other tests each pull request targeting the source branch.

Copy link
Member Author

@nuclearsandwich nuclearsandwich Dec 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My information is based on observation rather than specification
test_commits runs a job whenever a new commit is made to a specific branch. The jobs on http://build.ros2.org/view/Cdev/ test commits to the source branch.
test_pull_requests runs a new job whenever a PR is opened against a branch. The jobs on http://build.ros2.org/view/Cpr/ test pull requests on that branch.

The defaults for crystal are here https://github.com/ros2/ros_buildfarm_config/blob/3755c1ea027e65f026b8056fc159552393bdc333/crystal/source-build.yaml#L88-L91

@nuclearsandwich nuclearsandwich merged commit a5275b3 into ros2 Dec 14, 2018
@nuclearsandwich nuclearsandwich deleted the disable-rcl-devel branch December 14, 2018 03:06
dirk-thomas pushed a commit that referenced this pull request Jan 10, 2019
The group dependency on a logger package prevents builds from running.
dirk-thomas pushed a commit to ros/rosdistro that referenced this pull request Jan 10, 2019
The group dependency on a logger package prevents builds from running.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants