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

add missing include (for FTBFS with boost-1.83.0) #428

Merged
merged 1 commit into from
Apr 26, 2024
Merged

Conversation

ban-nobuhiro
Copy link
Contributor

Ubuntu 24.04 の Beta 版で tsubakuro のビルドが失敗します。
udf_wires.h では std::set を使用していますが #include <set> が不足しています。
同時に使用している boost ライブラリが、Ubuntu 22.04 で使用している boost のバージョン (1.74.0) では、その内部から <set> を include していたため、udf_wires.h のほうで不足していても今までビルドが通っていたものが、新しい 1.83.0 では <set> を include しなくなったため通らなくなったようです。

対応として、明示的に #include <set> を追加しました。

In file included from /home/ban/build/src/tsurugidb-1.0.0-BETA4-gcc11/tsubakuro/modules/ipc/src/main/native/src/wireJNI.cpp:20:
/home/ban/build/src/tsurugidb-1.0.0-BETA4-gcc11/tsubakuro/modules/ipc/src/main/native/include/udf_wires.h:211:10: error: 'set' in namespace 'std' does not name a template type
  211 |     std::set<resultset_wires_container*> resultset_wires_set_{};
      |          ^~~
In file included from /home/ban/build/src/tsurugidb-1.0.0-BETA4-gcc11/tsubakuro/modules/ipc/src/main/native/src/wireJNI.cpp:20:
/home/ban/build/src/tsurugidb-1.0.0-BETA4-gcc11/tsubakuro/modules/ipc/src/main/native/include/udf_wires.h:21:1: note: 'std::set' is defined in header '<set>'; did you forget to '#include <set>'?
   20 | #include "wire.h"
  +++ |+#include <set>
   21 | 

@t-horikawa
Copy link
Contributor

LGTM
マージしておきます。

@t-horikawa t-horikawa merged commit 44abe8a into master Apr 26, 2024
4 checks passed
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