Skip to content
This repository has been archived by the owner on Dec 14, 2022. It is now read-only.

Package selection & build customization via colcon defaults file #166

Merged
merged 8 commits into from
Mar 10, 2020

Conversation

emersonknapp
Copy link
Contributor

@emersonknapp emersonknapp commented Mar 5, 2020

Closes #164

Allow users to pass arbitrary arguments to the colcon list and colcon build phases. This allows for arbitrary build customization, so I have removed the hardcoding of merge-install

@codecov
Copy link

codecov bot commented Mar 5, 2020

Codecov Report

Merging #166 into master will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
+ Coverage   93.85%   93.91%   +0.05%     
==========================================
  Files           7        7              
  Lines         228      230       +2     
==========================================
+ Hits          214      216       +2     
  Misses         14       14              
Flag Coverage Δ
#unittests 93.91% <100.00%> (+0.05%) ⬆️
Impacted Files Coverage Δ
ros_cross_compile/docker_client.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 426d1c3...35b6f9b. Read the comment docs.

Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
@emersonknapp emersonknapp changed the title [WIP] colcon defaults file Package selection & build customization via colcon defaults file Mar 6, 2020
@emersonknapp emersonknapp marked this pull request as ready for review March 6, 2020 06:00
ros_cross_compile/docker/gather_rosdeps.sh Outdated Show resolved Hide resolved
test/test_dependencies.py Outdated Show resolved Hide resolved
test/test_dependencies.py Outdated Show resolved Hide resolved
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
ros_cross_compile/docker/rosdep.Dockerfile Show resolved Hide resolved
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
@emersonknapp emersonknapp merged commit 3d6ef0e into master Mar 10, 2020
@emersonknapp emersonknapp deleted the emersonknapp/colcon-defaults branch March 10, 2020 00:21
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.

Enable colcon defaults file to customize list/build
4 participants