Conversation
|
@Flamefire any idea how to fix the failures with your new B2 + Cmake test? I tried manually adding the cmake config information in the cmake/ folder, but that doesn't seem to work out. |
There was a problem hiding this comment.
I tried manually adding the cmake config information in the cmake/ folder, but that doesn't seem to work out.
This test is for testing that the CMake config installed using the B2 workflow can actually be consumed by CMake, to mirror what a user would do.
So changing the CMake files is not required here as IIRC B2 doesn't use (or even know) about those.
However it makes sense to ensure that those are also complete.
You are missing the install part in the B2 files, for header only libs I checked Boost.System: https://github.com/boostorg/system/blob/e6595f0d6481b5759e9282d97d0205e65e358187/build.jam#L24-L26
I.e. in your build.jam below call-if : boost-library safe_numbers add install boost_safe_numbers
I added some comments while checking your changeset with things I noticed while reading the diff
I'll give that a shot. I thought |
|
This error is new as of this morning (see https://github.com/cppalliance/safe_numbers/actions/runs/25377007558/job/74414709606?pr=197): According to the GHA README Windows 2025 is supposed to have that version installed: https://github.com/actions/runner-images/blob/main/images/windows/Windows2025-Readme.md. |
|
Might be an issue with that runner. The same job in Boost.CI just passed for me: https://github.com/boostorg/boost-ci/actions/runs/25378171088/job/74418838451?pr=336 |
|
@Flamefire Is there a library that you know is using Boost.CI and passing this B2 + CMake test so I can try and copy-paste their setup? I make it to here now: |
|
The current switch from The problem is that this is a header-only library and targets for those are only created by the CMake installation process, but not (yet) by B2. See current discussion in Slack. Basically for this test using the installed library you'd need to use the I just added a commit to Boost.CI to help with that: boostorg/boost-ci@e70bf6d As for the issues with Visual Studio, I've noticed this warning in the logs:
And there VS 2022 isn't installed -.- So maybe they already doing the change randomly. Let's see what comes out of that. |
Depending on the CMake version, you either get a warning that
Thanks. I saw the conversation, but it wasn't entirely clear to me how to apply it.
Always something with GHA. |
This is only the |
Co-authored-by: Alexander Grund <Flamefire@users.noreply.github.com>
|
Looks like we're finally good here. Thanks for the help @Flamefire |
No description provided.