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

Remove boost::filesystem references #2073

Merged
7 commits merged into from
Dec 9, 2022
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 14, 2022

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

@ghost ghost added next iteration to not forget to include it backend labels Nov 14, 2022
@smk762 smk762 added 0.5.8 and removed next iteration to not forget to include it labels Nov 18, 2022
@ghost ghost marked this pull request as ready for review November 21, 2022 10:05
@ghost ghost requested review from smk762 and tonymorony November 21, 2022 10:05
@ghost ghost added this to Testing in 0.5.8 Nov 25, 2022
@SirSevenG SirSevenG self-requested a review November 25, 2022 11:10
SirSevenG
SirSevenG previously approved these changes Nov 28, 2022
@smk762
Copy link
Collaborator

smk762 commented Nov 28, 2022

I still see some build errors actions logs

https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556831#step:20:1483
usr/include/c++/10/bits/fs_fwd.:39: Parse error at "std"
&

2022-11-25T17:41:21.2682009Z AutoMoc subprocess error
2022-11-25T17:41:21.2682559Z ------------------------
2022-11-25T17:41:21.2682869Z The moc process failed to compile
2022-11-25T17:41:21.2683276Z   "SRC:/src/core/atomicdex/services/update/zcash.params.service.hpp"
2022-11-25T17:41:21.2683634Z into
2022-11-25T17:41:21.2684602Z   "SRC:/ci_tools_atomic_dex/build-Debug/src/atomicdex-desktop_autogen/NVWMQ4ZM6I/moc_zcash.params.service.cpp"

https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556905#step:19:1251 and other places

2022-11-25T18:03:02.5687170Z /Users/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/src/core/atomicdex/utilities/global.utilities.hpp:39:63: fatal error: 'path' is unavailable: introduced in macOS 10.15
2022-11-25T18:03:02.5688380Z     bool        create_if_doesnt_exist(const std::filesystem::path& path);
2022-11-25T18:03:02.5689120Z                                                               ^
2022-11-25T18:03:02.5690270Z /Applications/Xcode_13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/filesystem:738:24: note: 'path' has been explicitly marked unavailable here
2022-11-25T18:03:02.5691280Z class _LIBCPP_TYPE_VIS path {

@ghost
Copy link
Author

ghost commented Nov 28, 2022

I still see some build errors actions logs

https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556831#step:20:1483 usr/include/c++/10/bits/fs_fwd.:39: Parse error at "std" &

2022-11-25T17:41:21.2682009Z AutoMoc subprocess error
2022-11-25T17:41:21.2682559Z ------------------------
2022-11-25T17:41:21.2682869Z The moc process failed to compile
2022-11-25T17:41:21.2683276Z   "SRC:/src/core/atomicdex/services/update/zcash.params.service.hpp"
2022-11-25T17:41:21.2683634Z into
2022-11-25T17:41:21.2684602Z   "SRC:/ci_tools_atomic_dex/build-Debug/src/atomicdex-desktop_autogen/NVWMQ4ZM6I/moc_zcash.params.service.cpp"

https://github.com/KomodoPlatform/atomicDEX-Desktop/actions/runs/3549793097/jobs/5962556905#step:19:1251 and other places

2022-11-25T18:03:02.5687170Z /Users/runner/work/atomicDEX-Desktop/atomicDEX-Desktop/src/core/atomicdex/utilities/global.utilities.hpp:39:63: fatal error: 'path' is unavailable: introduced in macOS 10.15
2022-11-25T18:03:02.5688380Z     bool        create_if_doesnt_exist(const std::filesystem::path& path);
2022-11-25T18:03:02.5689120Z                                                               ^
2022-11-25T18:03:02.5690270Z /Applications/Xcode_13.2.1.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/filesystem:738:24: note: 'path' has been explicitly marked unavailable here
2022-11-25T18:03:02.5691280Z class _LIBCPP_TYPE_VIS path {

Is it possible to use clang for CI builds instead of AppleClang ?

@smk762
Copy link
Collaborator

smk762 commented Nov 28, 2022

Is it possible to use clang for CI builds instead of AppleClang ?

Nothing is is possible, not if you can imagine it. That's what being a scientist is all about.

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.

@smk762 smk762 assigned ghost Dec 2, 2022
@smk762
Copy link
Collaborator

smk762 commented Dec 5, 2022

337de85 should fix it.
ref: https://stackoverflow.com/questions/69407237/qt-moc-errorusr-include-c-10-bits-fs-fwd-39-parse-error-at-std
it is a qt bug, filesystem includes need to go after Qxxx includes

@smk762
Copy link
Collaborator

smk762 commented Dec 6, 2022

To test:

  • Confirm able to launch app on all OS (portable and installer)
  • Leave a comment to indicate which OS etc you have confirmed working.

@cipig
Copy link
Member

cipig commented Dec 6, 2022

works fine on Linux

cipig
cipig previously approved these changes Dec 6, 2022
@SirSevenG
Copy link
Contributor

SirSevenG commented Dec 6, 2022

Win portable/installer [Windows 11] ✅
Fedora 37 build&run ✅
@smk762 I don't see MacOS builds in CI and there's an error with artifact not being found in expected path in workflow run logs, please recheck that.

@smk762
Copy link
Collaborator

smk762 commented Dec 6, 2022

Copy link
Contributor

@SirSevenG SirSevenG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OSX installer/dmg ✅

@SirSevenG SirSevenG self-requested a review December 9, 2022 01:00
@ghost ghost requested a review from cipig December 9, 2022 13:59
@ghost ghost merged commit 5e1d217 into dev Dec 9, 2022
@ghost ghost moved this from Testing to Done in 0.5.8 Dec 9, 2022
@smk762 smk762 mentioned this pull request Mar 6, 2023
@smk762 smk762 deleted the remove_boost_filesystem_references branch August 7, 2023 07:42
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
0.5.8
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants