[Opm] Dependencies revisited

[Bård Skaflestad]

In addition to the Boost.Spirit component that Joakim mentions there's also the practical issue that almost all of our unit tests target Boost.Test as a framework.  For instance, the header <boost/test/unit_test.hpp> is included 193 times across the OPM code base.  If we're going to replace that with another framework--Google Test might be an alternative--it is going to require some work to rewrite the tests.

That said, raising the minimum C++ language version to C++17 will allow us to replace most of our other usage of Boost; notably regular expressions and Boost.Filesystem.


Regards,

Bård Skaflestad
SINTEF Digital, Mathematics & Cybernetics
Computational Geosciences group

-----Original Message-----
From: Opm <opm-bounces at opm-project.org> On Behalf Of Joakim Hove
Sent: Tuesday, January 28, 2020 9:37 AM
To: opm at opm-project.org
Subject: Re: [Opm] Dependencies revisited

Another recurring issue is Boost. If we jump to Debian Buster and Ubuntu
> 19.04, this entails that we can bump minimum version of GCC to 8.3.0. 
> Is this sufficient to consider removing Boost as a dependency?
>

The parser uses boost spirit::qi which at some stage proved to give a performance gain for plain parsing of numerical input, i.e. "3.14" -> 3.14.
It is simple to replace with std varieties, but there might be a performance hit.

J
_______________________________________________
Opm mailing list
Opm at opm-project.org
https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fopm-project.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fopm&data=02%7C01%7CBard.Skaflestad%40sintef.no%7C46919502557d4925086f08d7a3cd5afc%7Ce1f00f39604145b0b309e0210d8b32af%7C1%7C1%7C637157974698485523&sdata=cb0Kik6qSaFcvgHXAzyqQiQPz%2BJz6TJ96agL5%2FvlUMQ%3D&reserved=0