[Merge] lp:~mandel/dbus-cpp/expose_executor into lp:dbus-cpp

Łukasz Zemczak lukasz.zemczak at canonical.com
Mon Jun 9 11:27:58 UTC 2014


Review: Needs Fixing

So, I found the reason for the symbols failure. It seems that dpkg-gensymbols (and related) is just stupid. The problem is in this line of debian/libdbus-cpp3.symbols.32bit:

+ (c++)"core::dbus::asio::make_executor(std::shared_ptr<core::dbus::Bus> const&, boost::asio::io_service &)@Base" 0replaceme

It should be:

+ (c++)"core::dbus::asio::make_executor(std::shared_ptr<core::dbus::Bus> const&, boost::asio::io_service&)@Base" 0replaceme

Please notice that there *cannot* be a whitespace between the typename and the pointer/reference sign. This actually causes a SYMBOL MISMATCH. So, dpkg-gensymbols is not whitespace-tolerant...
-- 
https://code.launchpad.net/~mandel/dbus-cpp/expose_executor/+merge/222200
Your team Ubuntu Phablet Team is subscribed to branch lp:dbus-cpp.



More information about the Ubuntu-reviews mailing list