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

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

Merged
merged 5 commits into from
Nov 27, 2023

Conversation

traversaro
Copy link
Collaborator

@traversaro traversaro commented Nov 23, 2023

Fix #769 . I do not think it make sense to document this change in the changelog.

@traversaro traversaro changed the title CI: Bump vcpkg-robotology to v0.20.0 CI: Bump vcpkg-robotology to v0.11.0 Nov 23, 2023
@traversaro
Copy link
Collaborator Author

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
Copy link
Member

Unfortunately the error remained there :(

@traversaro
Copy link
Collaborator Author

Probably we can just disable GUI compilation then?

@GiulioRomualdi
Copy link
Member

Yes

@GiulioRomualdi
Copy link
Member

Let's see if 728c0c7 solves the problem

@GiulioRomualdi GiulioRomualdi enabled auto-merge (squash) November 27, 2023 09:18
@GiulioRomualdi GiulioRomualdi merged commit 136302a into ami-iit:master Nov 27, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vcpkg CI job fails at vcpkg install tomlplusplus November 2023
3 participants