Skip to content

Commit

Permalink
Use distutils workaround for latest setuptools v50 (#182)
Browse files Browse the repository at this point in the history
* Use the workaround environment variable for the setuptools/distutils fix

Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
  • Loading branch information
emersonknapp authored Sep 1, 2020
1 parent 03a3f46 commit c86c0ae
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 16 deletions.
5 changes: 4 additions & 1 deletion Dockerfile.backwards
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,10 @@ RUN apt-get update && \
RUN rosdep update && \
rosdep install --from-paths integration/test_workspace --ignore-src -r -y

RUN pip3 install --upgrade pip setuptools!=50.0.0
# NOTE: This is a workaround for setuptools 50.* (see https://github.com/pypa/setuptools/issues/2352)
ENV SETUPTOOLS_USE_DISTUTILS=stdlib

RUN pip3 install --upgrade pip setuptools
RUN pip3 install -r requirements.txt
RUN pip3 install colcon-bundle

Expand Down
5 changes: 4 additions & 1 deletion Dockerfile.dashing
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,10 @@ RUN mkdir -p /home/builduser
RUN chown builduser /home/builduser
RUN sh -c "echo 'builduser ALL=NOPASSWD: ALL' >> /etc/sudoers"

RUN pip3 install --upgrade pip setuptools!=50.0.0
# NOTE: This is a workaround for setuptools 50.* (see https://github.com/pypa/setuptools/issues/2352)
ENV SETUPTOOLS_USE_DISTUTILS=stdlib

RUN pip3 install --upgrade pip setuptools
# RUN pip3 install -r requirements.txt

RUN pip3 install -U pytest colcon-common-extensions
Expand Down
5 changes: 4 additions & 1 deletion Dockerfile.double
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,13 @@ WORKDIR /opt/package
RUN apt-get update && \
apt-get install -y python3-pip python3-apt python-pip enchant sudo

# NOTE: This is a workaround for setuptools 50.* (see https://github.com/pypa/setuptools/issues/2352)
ENV SETUPTOOLS_USE_DISTUTILS=stdlib

RUN rosdep update && \
rosdep install --from-paths integration/test_workspace --ignore-src -r -y

RUN pip3 install --upgrade pip setuptools!=50.0.0
RUN pip3 install --upgrade pip setuptools
RUN pip3 install -r requirements.txt
RUN pip3 install -e .

Expand Down
3 changes: 2 additions & 1 deletion Dockerfile.kinetic
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ RUN mkdir -p /home/builduser
RUN chown builduser /home/builduser
RUN sh -c "echo 'builduser ALL=NOPASSWD: ALL' >> /etc/sudoers"

RUN pip3 install --upgrade pip setuptools!=50.0.0
RUN pip3 install --upgrade pip setuptools
RUN pip3 install -r requirements.txt
RUN pip3 install -e .

Expand All @@ -29,6 +29,7 @@ RUN rosdep update && \

WORKDIR /opt/package/integration/test_workspace

ENV SETUPTOOLS_USE_DISTUTILS=stdlib
RUN source /opt/ros/kinetic/setup.sh; colcon build
RUN source /opt/ros/kinetic/setup.sh; colcon bundle --bundle-version 1 --bundle-base v1 --pip-requirements py27_requirements.txt --pip3-requirements py3_requirements.txt
RUN source /opt/ros/kinetic/setup.sh; colcon bundle --bundle-version 2 --bundle-base v2 --pip-requirements py27_requirements.txt --pip3-requirements py3_requirements.txt
4 changes: 3 additions & 1 deletion Dockerfile.melodic
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ RUN mkdir -p /home/builduser
RUN chown builduser /home/builduser
RUN sh -c "echo 'builduser ALL=NOPASSWD: ALL' >> /etc/sudoers"

RUN pip3 install --upgrade pip setuptools!=50.0.0
RUN pip3 install --upgrade pip setuptools
RUN pip3 install -r requirements.txt
RUN pip3 install -e .

Expand All @@ -32,6 +32,8 @@ RUN rosdep update && \

WORKDIR /opt/package/integration/test_workspace

# NOTE: This is a workaround for setuptools 50.* (see https://github.com/pypa/setuptools/issues/2352)
ENV SETUPTOOLS_USE_DISTUTILS=stdlib
RUN source /opt/ros/melodic/setup.sh; colcon build
RUN source /opt/ros/melodic/setup.sh; colcon bundle --bundle-version 1 --bundle-base v1 --pip-requirements py27_requirements.txt --pip3-requirements py3_requirements.txt
RUN source /opt/ros/melodic/setup.sh; colcon bundle --bundle-version 2 --bundle-base v2 --pip-requirements py27_requirements.txt --pip3-requirements py3_requirements.txt
11 changes: 0 additions & 11 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -76,14 +76,3 @@ junit_suite_name = colcon-bundle

[flake8]
import-order-style = google











0 comments on commit c86c0ae

Please sign in to comment.