Tasks #1346

Patch request: use either libboost 1.46 or 1.48 on debian systems?

Added by S. Wrede over 11 years ago. Updated over 11 years ago.

Status:RejectedStart date:01/22/2013
Priority:NormalDue date:
Assignee:J. Wienke% Done:

0%

Category:cmake
Target version:Robotics Service Bus - rsb-0.9

Description

Original request from Alexandre Tuleu:

It would be nice so we could use either libboost 1.46 or libboost 1.48 on precise system.

He proposed the patch attached to this issue. Thx for that!

fixes_rsc_for_multipleboost_version.patch Magnifier (897 Bytes) S. Wrede, 01/22/2013 11:32 AM

History

#1 Updated by S. Wrede over 11 years ago

Just a question that worried us when briefly thinking about this: Do you think this will work given the header-only implmentations contained in RSC and other dependant libs? Consider that RSC was build against libboost 1.46 and this Debian package is now used on a system with 1.48? Would that work, ABI-wise?

#2 Updated by A. Tuleu over 11 years ago

Actually I did not thought that RSC was not header only, but does have some precompiled binaries. (librsc.so.0.9 is installed)

Then ABI compatibilty is not insured between 1.46 and 1.48 (that's why package does not have same name). However the big difference between this two version is C++ support.

Anyway since nemomath is header only, it should allow this conditionnal dependency (see #1341) for precise (There is no limitation, and it leave a weight for other user). Then you should choose beween using 1.46 or 1.48 with RSC. The biggest difference with the two version is C++0x and C++11 standard support if I am right. Or maybe we could have two line of packages version, for example librsc0.9 and librsc0.9-boost1.48, but that may just be too much of an hassle for just clang support.

However I recompiled without problem RSC and nemomath with libboost1.46 and clang v3.1 and v3.2 with no problem. Only RCI show some compilation errors. Maybe this is some bug that only show up with clang in RCI.

#3 Updated by J. Wienke over 11 years ago

Alexandre Tuleu wrote:

Actually I did not thought that RSC was not header only, but does have some precompiled binaries. (librsc.so.0.9 is installed)

Then ABI compatibilty is not insured between 1.46 and 1.48 (that's why package does not have same name). However the big difference between this two version is C++ support.

RSC is mostly implemented as a shared library with the least amount of header implementations. Howerver, we are using boost, where a big fraction of code is implemented in the headers. Hence, a version compiled against 1.46 cannot correctly work with 1.48, because many parts of the boost implementation 1.46 are actually compiled into RSC due to the header implementations. We would have to provide a second package of RSC capable of working with 1.48. But this then ripples through all downstream dependencies, which results in a maintenance nightmare for us. So I see no real chance to do this.

Anyway since nemomath is header only, it should allow this conditionnal dependency (see #1341) for precise (There is no limitation, and it leave a weight for other user). Then you should choose beween using 1.46 or 1.48 with RSC. The biggest difference with the two version is C++0x and C++11 standard support if I am right. Or maybe we could have two line of packages version, for example librsc0.9 and librsc0.9-boost1.48, but that may just be too much of an hassle for just clang support.

As explained above, this ripples through all downstream dependencies like RSB etc. I don't think we can manage this additional effort. Moreover, all ubuntu packages depend on 1.46. If we add the 1.48 version,linking against ubuntu packages, which internally also use boost, is not possible anymore without conflicts. I would like to keep the common baseline for ubuntu to preserve this compatibility.

#4 Updated by J. Wienke over 11 years ago

Alexandre? Any more thoughts on this or would it be ok to close this issue?

#5 Updated by A. Tuleu over 11 years ago

No it is ok to close it

#6 Updated by J. Wienke over 11 years ago

  • Status changed from New to Rejected

Also available in: Atom PDF