-
Notifications
You must be signed in to change notification settings - Fork 277
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
[fix]: Iroha 1: Mac: Repair benchmark #3345
Conversation
5b34f44
to
a7a554d
Compare
18e3cdf
to
2733254
Compare
I noticed that after upgrading of VCPKG version (fdc2c1e) some of patches are not applicable anymore (new version probably fixed its). That is why I've removed them:
|
I noticed that upgrading vcpkg updated libraries, and Iroha is using some old dependencies:
|
@safinsaf Could You please install: |
I've fixed building (locally) of |
Just FYI, I recommend squashing these commits, rather than rebasing. This doesn't exactly follow our commit conventions. |
2. Removed incorrect patches to vcpkg libraries after VCPKG version bump 3. Fixed building for irohad and iroha-cli after after VCPKG version bump (and 3rd part libraries version bump) Signed-off-by: Grzegorz Bazior <[email protected]>
b79b184
to
ee3cc7d
Compare
I've fixed building (locally) of
I've created one commit with :
is the way which You wanted? |
Signed-off-by: safinsaf <[email protected]>
Description
Currently
develop
is failling to build on MacOS, after enabling build on MacOS (829357f), that is why I'm trying to fix its by doing something similar to:#2906
Hopefully it will fix the compilation error:
#3081 (comment)
Fixed build (hopefully, because I don't have MacOS), according to the instruction:
https://stackoverflow.com/questions/72588408/vcpkg-how-to-edit-package-file-when-compilation-fails-when-installing-package
Linked issue
#3081
Closes #{issue_number}
Benefits
Successful build on MacOS
Checklist
CONTRIBUTING.md