search

ami-iit / bipedal-locomotion-framework

Suite of libraries for achieving bipedal locomotion on humanoid robots
https://ami-iit.github.io/bipedal-locomotion-framework/
BSD 3-Clause "New" or "Revised" License
136 stars 36 forks source link

CI: Bump vcpkg-robotology to v0.11.0 #770

Closed traversaro closed 8 months ago

traversaro commented 8 months ago

Fix https://github.com/ami-iit/bipedal-locomotion-framework/issues/769 . I do not think it make sense to document this change in the changelog.

traversaro commented 8 months ago

YARP fails with error:

 Creating library D:/a/bipedal-locomotion-framework/bipedal-locomotion-framework/yarp/build/lib/Release/yarpmanager.lib and object D:/a/bipedal-locomotion-framework/bipedal-locomotion-framework/yarp/build/lib/Release/yarpmanager.exp
LINK : warning LNK4217: symbol '?getBuilder@YarpBuilderLib@@SAPEAVBuilderWindow@@PEAVApplication@manager@yarp@@PEAVManager@45@PEAVSafeManager@@_N@Z (public: static class BuilderWindow * __cdecl YarpBuilderLib::getBuilder(class yarp::manager::Application *,class yarp::manager::Manager *,class SafeManager *,bool))' defined in 'yarpbuilderlib.obj' is imported by 'applicationviewwidget.obj' in function '"public: __cdecl ApplicationViewWidget::ApplicationViewWidget(class yarp::manager::Application *,class yarp::manager::Manager *,class yarp::os::Property *,bool,class QWidget *)" (??0ApplicationViewWidget@@QEAA@PEAVApplication@manager@yarp@@PEAVManager@23@PEAVProperty@os@3@_NPEAVQWidget@@@Z)' [D:\a\bipedal-locomotion-framework\bipedal-locomotion-framework\yarp\build\src\yarpmanager\yarpmanager.vcxproj]
  yarpmanager.vcxproj -> D:\a\bipedal-locomotion-framework\bipedal-locomotion-framework\yarp\build\bin\Release\yarpmanager.exe

We can either disable YARP GUIs, or see if the problem is solved by updating YARP.

GiulioRomualdi commented 8 months ago

Unfortunately the error remained there :(

traversaro commented 8 months ago

Probably we can just disable GUI compilation then?

GiulioRomualdi commented 8 months ago

Yes

GiulioRomualdi commented 8 months ago

Let's see if https://github.com/ami-iit/bipedal-locomotion-framework/pull/770/commits/728c0c75337e72281020afec12ac57ae5785ad4f solves the problem