search

boostorg / contract

Contract programming for C++
http://www.boost.org/doc/libs/develop/libs/contract/doc/html/index.html
Boost Software License 1.0
40 stars 27 forks source link

Remove linking with Boost.System #20

Closed Lastique closed 5 years ago

Lastique commented 5 years ago

Since Boost.System is header-only now, no need to link with the library.

lcaminiti commented 5 years ago

Cool. I will make these changes in the next release of Boost.Contract.

lcaminiti commented 5 years ago

I checked and as of Boost 1.70 Boost.System DLL is still built. It doesn't look like header-only Boost.System has been released yet so I can't fully tested and for now I won't remove this linkage. This change to Boost.Contract can be made at any time by whom releases Boost.System as header-only in a future Boost release. NOTE: In the meanwhile, this explicit linkage is only there when building Boost.Contract as header-only (Boost.Contract's bjam option bc_hdr=only) and that is not the default build mode. This is probably rarely, if ever, used by users.

Lastique commented 5 years ago

The header-only Boost.System has been released in Boost 1.69. The built library is a dummy to keep projects that link against Boost.System building. It does not contain any code.

Please reopen. Linking against Boost.System introduces a useless dependency on libboost_system, in terms of libraries and packages.

lcaminiti commented 5 years ago

I am making this change in the next release. I didn't just merge this pull request because also some /boost//system needed to be removed from build/Jamfile.v2 so I making all these changes together.

Lastique commented 5 years ago

Ok, thanks.