We cannot (yet) remove the EXPECTED_BOOST_INCLUDES entry as this header is still needed in fs.h (see #14763).
Partially reverts #14718.
revert removal of fstream.hpp header in fs.h #14768
pull kallewoof wants to merge 1 commits into bitcoin:master from kallewoof:restore-fs-h-include-boost-fs-fstream changing 1 files +1 −0-
kallewoof commented at 5:42 AM on November 20, 2018: member
-
e816b341ab
revert removal of fstream.hpp header in fs.h
This is required for some architectures.
-
meshcollider commented at 5:49 AM on November 20, 2018: contributor
utACK
- meshcollider cross-referenced this on Nov 20, 2018 from issue Remove filesystem/fstream from EXPECTED_BOOST_INCLUDES by Sjors
- meshcollider added the label Refactoring on Nov 20, 2018
-
ken2812221 commented at 6:01 AM on November 20, 2018: contributor
utACK e816b341ab794fb84d89efb90b8c0f369d34d6ca
-
AkioNak commented at 8:12 AM on November 20, 2018: contributor
utACK e816b34
-
laanwj commented at 8:57 AM on November 20, 2018: member
how did this get missed on review/testing?
-
Sjors commented at 9:02 AM on November 20, 2018: member
Part of the problem is that apparantly Travis was down, and that wasn't noticed: #14763 (comment)
-
- laanwj merged this on Nov 20, 2018
- laanwj closed this on Nov 20, 2018
- laanwj referenced this in commit 1b99d153d0 on Nov 20, 2018
- kallewoof deleted the branch on Nov 20, 2018
- deadalnix referenced this in commit 4d562db5ae on Mar 20, 2020
- ftrader referenced this in commit 2084301181 on May 19, 2020
- Munkybooty referenced this in commit 5ee0ec4f56 on Jul 29, 2021
- Munkybooty referenced this in commit eba469e3b0 on Aug 3, 2021
- Munkybooty referenced this in commit eb9a3965c3 on Aug 5, 2021
- Munkybooty referenced this in commit 526fb543ca on Aug 8, 2021
- Munkybooty referenced this in commit e1c365e257 on Aug 9, 2021
- Munkybooty referenced this in commit de5d8af15d on Aug 11, 2021
- 5tefan referenced this in commit 6e92792f59 on Aug 12, 2021
- bitcoin locked this on Sep 8, 2021
