Skip to content

Commit

Permalink
fixed performance regression that caused mutants to take over a minut…
Browse files Browse the repository at this point in the history
…e to build
  • Loading branch information
ChrisTimperley committed Jul 8, 2018
1 parent e88a80e commit e1ebbb1
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 14 deletions.
23 changes: 12 additions & 11 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,20 @@ RUN cd /tmp \
RUN . /opt/ros/${ROS_DISTRO}/setup.sh \
&& bear catkin build

RUN apt-get update \
&& apt-get install -y software-properties-common \
&& add-apt-repository ppa:deadsnakes/ppa \
&& apt-get update \
&& apt-get install -y python3.6 sudo
RUN curl https://bootstrap.pypa.io/get-pip.py -o /tmp/get-pip.py \
&& python3.6 /tmp/get-pip.py
RUN pip3.6 install pyyaml rospkg catkin_pkg netifaces gcovr

# build tests
RUN . /opt/ros/${ROS_DISTRO}/setup.sh \
&& catkin build -DENABLE_TESTS=ON --continue-on-failure --no-status --make-args tests || exit 0
#RUN catkin build -DENABLE_TESTS=ON --no-status orocos_kdl
&& catkin config --cmake-args -DENABLE_TESTS=ON \
&& (catkin build --continue-on-failure --no-status --make-args tests || exit 0) \
&& catkin build --no-status

# add entrypoint
ENV ROS_WSPACE /ros_ws
Expand All @@ -65,15 +75,6 @@ exec \"\$@\"" > "${ROS_WSPACE}/entrypoint.sh" \
ENTRYPOINT ["/ros_ws/entrypoint.sh"]
CMD ["/bin/bash"]

RUN apt-get update \
&& apt-get install -y software-properties-common \
&& add-apt-repository ppa:deadsnakes/ppa \
&& apt-get update \
&& apt-get install -y python3.6 sudo
RUN curl https://bootstrap.pypa.io/get-pip.py -o /tmp/get-pip.py \
&& python3.6 /tmp/get-pip.py
RUN pip3.6 install pyyaml rospkg catkin_pkg netifaces gcovr

ENV TURTLEBOT_STAGE_MAP_FILE /ros_ws/src/turtlebot_simulator/turtlebot_stage/maps/maze.yaml
ENV TURTLEBOT_STAGE_WORLD_FILE /ros_ws/src/turtlebot_simulator/turtlebot_stage/maps/stage/maze.world
COPY _debug.launch .
Expand Down
7 changes: 4 additions & 3 deletions Dockerfile.instrumentation
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,15 @@ RUN cd /tmp \
RUN . /opt/ros/${ROS_DISTRO}/setup.sh \
&& catkin clean -y \
&& catkin config --cmake-args \
-DENABLE_TESTS=ON \
-DCMAKE_CXX_FLAGS="--coverage" \
-DCMAKE_LD_FLAGS="--coverage" \
&& catkin build --no-status \
&& catkin build \
-DENABLE_TESTS=ON \
&& (catkin build \
--continue-on-failure \
--no-status \
--make-args tests \
|| exit 0
|| exit 0) \
&& catkin build --no-status
# clear any existing coverage information
RUN gcovr -d -r .

0 comments on commit e1ebbb1

Please sign in to comment.