Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to use YARP conda-forge binary package when generating conda packages #1169

Merged
merged 1 commit into from
Jun 13, 2022

Conversation

traversaro
Copy link
Member

@traversaro traversaro commented Jun 13, 2022

As conda-forge/staged-recipes#18234 will hopefully be soon merged, we can finally fix #752 and #1094 .

In general, the changes are similar to #807 but for YARP.

@traversaro
Copy link
Member Author

@traversaro
Copy link
Member Author

First test build: https://github.com/robotology/robotology-superbuild/actions/runs/2489359263 . It should fail as conda-forge/staged-recipes#18234 is still not merged.

New build: https://github.com/robotology/robotology-superbuild/actions/runs/2489870178 . Now the yarp-cxx package should be available.

@traversaro
Copy link
Member Author

Build now fails with:

CMake Error at /usr/share/miniconda/envs/test/conda-bld/gazebo-yarp-plugins_1655143803361/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pla/lib/cmake/YARP/YARPConfig.cmake:159 (find_package):
  Could not find a package configuration file provided by "YARP_cv" with any
  of the following names:
    YARP_cvConfig.cmake
    yarp_cv-config.cmake
  Add the installation prefix of "YARP_cv" to CMAKE_PREFIX_PATH or set
  "YARP_cv_DIR" to a directory containing one of the above files.  If
  "YARP_cv" provides a separate development package or SDK, be sure it has
  been installed.
Call Stack (most recent call first):
  CMakeLists.txt:49 (find_package)

So we probably need to conda-forge/yarp-feedstock#1 .

@traversaro
Copy link
Member Author

So we probably need to conda-forge/yarp-feedstock#1 .

This was fixed. Merging so the migration is done in time for tonight generation.

@traversaro traversaro merged commit 90746d4 into master Jun 13, 2022
@traversaro traversaro deleted the switchyarptocondaforge branch June 13, 2022 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate some robotology-superbuild packages to conda-forge
1 participant