Re: [PR] GH-50300: [C++][CI] Update include path so Meson is able to find generated/Message_generated.h [arrow]

["Copilot (via GitHub)" <gi...@apache.org>]

Copilot commented on code in PR #50301:
URL: https://github.com/apache/arrow/pull/50301#discussion_r3497817065


##########
cpp/src/arrow/ipc/message_internal_test.cc:
##########
@@ -24,7 +24,7 @@
 
 #include <gtest/gtest.h>
 
-#include "Message_generated.h"
+#include "generated/Message_generated.h"

Review Comment:
   The PR title/description say Meson is fixed by adding an include path so code can keep including "Message_generated.h" without the `generated/` prefix, but this change instead updates the source include to `"generated/Message_generated.h"`. Please update the PR title/description to match the actual fix (or implement the include-dir change if that was the intent) to avoid confusion for reviewers and future archaeology.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org