Add support for openbabel 3
Needs ReviewPublic

Authored by arojas on Wed, May 6, 1:51 PM.

Details

Reviewers
yurchor
Group Reviewers
Kalzium
Summary

We require a min version 3.1 since the cmake config files in 3.0 are totally broken. In 3.1 they are still not completely right, they miss a dependency on boost in order to define the Boost::system target so we add a find_package call here.

Test Plan

Builds with openbabel 2.4 and 3.1

Diff Detail

Repository
R326 Kalzium
Lint
Lint Skipped
Unit
Unit Tests Skipped
arojas created this revision.Wed, May 6, 1:51 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptWed, May 6, 1:51 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
arojas requested review of this revision.Wed, May 6, 1:51 PM
yurchor accepted this revision.Wed, May 6, 2:08 PM
yurchor added a subscriber: yurchor.

Tested to work. Thanks.

This revision is now accepted and ready to land.Wed, May 6, 2:08 PM
arojas planned changes to this revision.Wed, May 6, 5:02 PM

Upstream is working on fixing the remaining cmake issues and they will release a 3.1.1 that will allow to simplify the patch

arojas updated this revision to Diff 82319.Fri, May 8, 9:04 PM

Removed boost dependency, no longer needed in 3.1.1. Unfortunately we can't bump the min version since they forgot to bump the version before release.

This revision is now accepted and ready to land.Fri, May 8, 9:04 PM
arojas requested review of this revision.Fri, May 8, 9:05 PM