-
Notifications
You must be signed in to change notification settings - Fork 208
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
Remove boost::filesystem references #2073
Conversation
I still see some build errors actions logs https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556831#step:20:1483
https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556905#step:19:1251 and other places
|
Is it possible to use clang for CI builds instead of AppleClang ? |
Yeah, as long as its something we can work into the CI. The first of the build errors above was in linux tho, so need to resolve that also. |
337de85 should fix it. |
To test:
|
works fine on Linux |
Win portable/installer [Windows 11] ✅ |
@SylEze https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3623565505/jobs/6109577723#step:19:1232 We have deployment target set to 10.14 @tonymorony are we still supporting MacOS 10.14 or can we update this to 10.15 (or 11)? It appears to be EOL/no longer supported for security updates etc. ref: https://endoflife.date/macos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OSX installer/dmg ✅
boost::filesystem is not needed anymore since filesystem features are now implemented in every common c++ compiler std library
This PR gives some changes to the CI to use 10.15 macOS SDK