5 ms·
I tried it a few months ago, back in November. I looked through what I did and in the end, to bring in pw_result, I had to define a failure handler, pull in th
by isopede 3y ago
I tried it a few months ago, back in November.
I looked through what I did and in the end, to bring in pw_result, I had to define a failure handler, pull in three separate cmake files for various definitions, define an ASSERT action (because a result type requires an ASSERT fail, apparently?), and set a backend and a handler for pw_assert. At some point I remember it complaining left and right about not having the right I/O methods for printing, etc. I just wanted to try the result type, and for some reason I needed to define I/O mechanisms for my platform.
Is this the correct procedure? I have no idea, because the cmake documentation had (has?) _zero_ examples. If they existed, I couldn't find them. I eventually got it to compile with the following snippet, but I can hardly believe that this is the "intended" way to do it.
FetchContent_Populate(pigweed)
include(${pigweed_SOURCE_DIR}/pw_build/pigweed.cmake)
include(${pigweed_SOURCE_DIR}/pw_assert/backend.cmake)
include(${pigweed_SOURCE_DIR}/pw_assert_basic/backend.cmake)
add_compile_definitions(PW_ASSERT_BASIC_ACTION=PW_ASSERT_BASIC_ACTION_LOOP)
pw_set_backend(pw_assert.check pw_assert_basic.check_backend)
pw_set_backend(pw_assert.assert pw_assert_basic.basic_handler)
add_subdirectory(${pigweed_SOURCE_DIR} ${pigweed_SOURCE_DIR})
- kaycebasques 3y agoVery much appreciate the details I'm off for the night but will follow up here next week We can also continue discussion on the Discord https://discord.gg/M9NSeTA https://discord.gg/M9NSeTA
- deleted 3y ago[deleted]
- kaycebasques 3y agoClosing the loop: * I'm forwarding your feedback to the team. * I will make sure we follow through on the plan to include CMake examples in our WIP "examples" repo [1] and will make sure pigweed.dev links to those examples consistently. * Our new module docs guidelines require every module to have a quickstart section that includes how to set up the module in every build system we support, including CMake [2]. That should help a bit but based on your experience it sounds like there's a lot more we need to cover in these quickstart sections. (I don't want to overcommit on behalf of other teammates but as docs lead these things are safely within my realm to change.) [1] https://pigweed.dev/seed/0122-code-samples.html#examples-repository https://pigweed.dev/seed/0122-code-samples.html#examples-rep... [2] https://pigweed.dev/pw_function/#get-started https://pigweed.dev/pw_function/#get-started