-
Notifications
You must be signed in to change notification settings - Fork 142
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
Simplify no wstreambuf support. #282
Conversation
@robertramey : Looks like the CI is stalled indefinitely waiting for a runner - that's because the ubuntu-18 image has been deprecated (for example) and available runners are stupidly scarce. Shall I update the images used as part of this PR? The catch is that older gcc versions will no longer be available. They are available with the drone CI runners that the cpp alliance provides for us though... |
Don’t worry about it. Boost I never really works for me anyway. I’ll except the PR, sync, my own machine, and run the tests locally. I’m sure this will be fine with a change this modest. Many thanks to you both for your help on these very Obscure issues. Robert Rameywww.rrsd.comOn Apr 30, 2023, at 1:15 AM, jzmaddock ***@***.***> wrote:
@robertramey : Looks like the CI is stalled indefinitely waiting for a runner - that's because the ubuntu-18 image has been deprecated (for example) and available runners are stupidly scarce. Shall I update the images used as part of this PR? The catch is that older gcc versions will no longer be available. They are available with the drone CI runners that the cpp alliance provides for us though...
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
looking more carefully at the PR I find a number of statements similar to the following: [ test-bsl-run_files test_helper_support : : : [ requires std_wstreambuf ] ] OK. But when I look at the source for test_helper_support , I don't see any reference to wide streams or wchar or anything else related. Why is the "requires" statement required here specifically? How does one know whether it's going to be needed or not? |
@robertramey, I added the requirement to all of the tests which had linker errors when building the tests with -DBOOST_NO_STD_LOCALE. Basically anything which used the xml_archives anywhere. |
**solution of this, ### boost serialization can not be sc stoped in windows platform #283 **
|
* develop: (108 commits) removed unordered tests tests have been taken over by developers of boost unordered uploading changes regarding unordered ... uploading tweaks to CMakeLists.txt so accommodate static linking imported/implemented boost/serialization/variant and test_variant Remove serialization support from Boost.Unordered containers Fix invalid vptr access in extended_type_info_no_rtti constructor Fix invalid downcast in ~xml_oarchive_impl altered boost optional to depend only public interface functions declared in std::optional. adjusted test_optional to test serialization of both boost::optional and std::optional Revert "Add serialization for std::optional" fix new requirement - if assigment is not defaulted, copy can't be either Fix clang-cl regression introduced by #282 correction in previous merge Remove debug output from util/test.jam Fix Sub::operator== signature to avoid ambiguity errors in C++20 Remove C++03 from GHA, because Variant and LexicalCast no longer compile under C++03 incremement archive version number fix PR 274 fix PR 283 committing most pending PRs Replace static_cast with reinterpet_cast in interface_oarchive Update xml_escape::fill() to default assign bstart, bend to an empty string Use the first instance of bos during serialization (for the case when there are multiple instances) ...
See #224.