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

using rosdep db --install-method apt #77

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

k-okada
Copy link

@k-okada k-okada commented Feb 13, 2015

use rosdep db --install-method apt to build dictionaries only for apt installable packages, (to ignore pip install rosdep keys), need to use this PR ros-infrastructure/rosdep#374

use `rosdep db --install-method apt` to build dictionaries only for apt installable packages, (to ignore pip install rosdep keys), need to use this PR https://github.com/ros-infrastructure/rosdep/pull/374/files
@dirk-thomas
Copy link
Member

Can you please describe why this change is necessary?

Since this repository is used on the current (but soon to be replaced) build farm it might not be worth to change it anymore.

@k-okada
Copy link
Author

k-okada commented Feb 13, 2015

we had trouble on generating doc for our repo witch depend on pip package, one solution is to remove pip based package entry from build/run_depend, but that as drawback that we can not make use of rosdep install features, so our solution is to enable doc generator to run even if the package has dependency to pip package.
http://jenkins.ros.org/view/Hdoc/job/doc-hydro-jsk_common/107/console

cf: jsk-ros-pkg/jsk_common#727, ros/rosdistro#7074 (comment)

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.

None yet

2 participants