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

Unbreak build on FreeBSD #1939

Merged
merged 2 commits into from
Oct 7, 2023
Merged

Unbreak build on FreeBSD #1939

merged 2 commits into from
Oct 7, 2023

Conversation

jbeich
Copy link
Contributor

@jbeich jbeich commented Oct 7, 2023

Regressed by e2bf7b2. Found on FreeBSD 13.2 (inside poudriere jail).

jbeich added 2 commits October 7, 2023 11:46
e2bf7b278d17In file included from plugins/single_plugins/place.cpp:1:
In file included from src/api/wayfire/signal-provider.hpp:7:
src/api/wayfire/nonstd/safe-list.hpp:144:35: error: implicit instantiation of undefined template 'std::vector<std::optional<wf::signal::connection_base_t *>>'
    std::vector<std::optional<T>> list;
                                  ^
src/api/wayfire/signal-provider.hpp:148:36: note: in instantiation of template class 'wf::safe_list_t<wf::signal::connection_base_t *>' requested here
        for (auto& [id, connected] : typed_connections)
                                   ^
/usr/include/c++/v1/iosfwd:260:28: note: template is declared here
class _LIBCPP_TEMPLATE_VIS vector;
                           ^
test/misc/safe-list-test.cpp:2:10: fatal error: 'doctest/doctest.h' file not found
 #include <doctest/doctest.h>
          ^~~~~~~~~~~~~~~~~~~
@ammen99
Copy link
Member

ammen99 commented Oct 7, 2023

Thanks, I'll update the 0.8.0 release to include these commits.

@ammen99 ammen99 merged commit 2059459 into WayfireWM:master Oct 7, 2023
4 checks passed
@ammen99
Copy link
Member

ammen99 commented Oct 7, 2023

Updated the release as well - I hope it doesn't break things for anybody since the release was very soon :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants