Skip to content

Commit c95a729

Browse files
author
David Roberts
authored
[7.17] [ML] Include an extra header in CDataFrameRowSlice (#2565)
Our two ARM macOS build servers have somehow ended up running different versions of Xcode - one is running version 12 and the other version 13. There have been changes to the C++ standard library between these versions that reveal a technical violation that we were previously getting away with: CDataFrameRowSlice.cc uses file streams without including the necessary header. Previously it must have been indirectly included. This PR includes it explicitly. Backport of #2563
1 parent 388d7bc commit c95a729

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

include/model/CStringStore.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
#include <atomic>
2424
#include <functional>
2525
#include <string>
26+
#include <vector>
2627

2728
namespace CResourceMonitorTest {
2829
class CTestFixture;

lib/core/CDataFrameRowSlice.cc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020

2121
#include <boost/filesystem.hpp>
2222

23+
#include <fstream>
2324
#include <memory>
2425
#include <vector>
2526

0 commit comments

Comments
 (0)