-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[hpx] Adding support for HPX release v1.9.1 #33106
Conversation
a9bfe88
to
0eaa722
Compare
@hkaiser, Thanks for your PR, this is error, please take look:
|
Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags. |
This should be fine now. Thanks! |
@hkaiser, There is one more bug that needs to be fixed:
|
@JonLiu1993 what compiler is that happening with? |
@hkaiser: VS2022 targeting x86. |
We currently do not really support 32bit builds (in particular on Windows). Is there a way to disable this on vcpkg? I have however opened a ticket (STEllAR-GROUP/hpx#6323) to have a look at this for the next release. |
add "supports": "!(windows & x86)" to the manifest |
@hkaiser, if this PR is ready for review please let me know. |
@JonLiu1993 Yes, this is ready to go (at least the tests pass now). Thanks! |
@hkaiser, When I tested the usage by
Usage:
CMakeLists.txt:
usageTest.cpp:
CMakeSettings.json:
|
@JonLiu1993 According to the file list: |
Tested usage again by
config:
build:
|
./vcpkg x-add-version --all
and committing the result.