Re: [Boost-bugs] [Boost C++ Libraries] #5341: Patch to improve shared library behavior with serialization

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #5341: Patch to improve shared library behavior with serialization
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-04-24 20:21:40


#5341: Patch to improve shared library behavior with serialization
----------------------------------------------+-----------------------------
  Reporter: Aaron Barany <akb825@…> | Owner: ramey
      Type: Patches | Status: reopened
 Milestone: To Be Determined | Component: serialization
   Version: Boost 1.46.0 | Severity: Problem
Resolution: | Keywords: serialization shared_libraries dlls
----------------------------------------------+-----------------------------

Comment (by Aaron Barany <akb825@…>):

 That is pretty much the gist of it. When I have a chance (most likely
 tomorrow) I can set up a simple example of where the dll import/export
 fails with the current behavior.

 If you want to try it out before I can get that set up, it's fairly
 simple. First, compile unpatched boost as DLLs. Next, setup a simple
 tester and copy one of the existing archives (like text_iarchive) into
 that tester and rename it. (say, to my_iarchive) If you compile that
 against the boost DLLs, it should fail with this error.
 (http://msdn.microsoft.com/en-us/library/62688esh.aspx) If you then apply
 my patch and recompile, it should succeed. (note that your renamed
 my_archive shouldn't have any of the dll import/export macros since it's
 in an executable rather than a DLL)

 Also, just to double check, I want to make sure you're applying the first
 set of patches (shared library patch.zip) and not the second set.
 (combined shared library patch.zip) The second one includes my patches in
 issue #5340, since both patches modify iserializer.hpp and cannot be
 applied separately. As I mentioned in comment 3, the combined patch still
 has an issue, so I will re-submit it when I resolve it. The normal patch
 (shared library patch.zip) should be fine, though, and passes all the
 regression tests. I will verify that it passes all the tests with gcc when
 I have a chance.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/5341#comment:12>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.

This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:06 UTC