Page MenuHomeFreeBSD

Ent-eigen editors/calligra.
ClosedPublic

Authored by tcberner on Jun 17 2017, 8:13 AM.
Tags
None
Referenced Files
F163218880: D11243.id29742.diff
Tue, Jul 21, 4:10 AM
Unknown Object (File)
Sun, Jul 19, 9:55 PM
Unknown Object (File)
Sat, Jul 18, 3:58 PM
Unknown Object (File)
Fri, Jul 10, 5:31 PM
Unknown Object (File)
Thu, Jul 9, 12:01 PM
Unknown Object (File)
Thu, Jul 9, 3:24 AM
Unknown Object (File)
Wed, Jul 8, 9:25 AM
Unknown Object (File)
Mon, Jun 22, 1:48 AM
Subscribers

Details

Reviewers
rakuco
Group Reviewers
kde
Commits
rP443736: Ent-eigen editors/calligra.
Summary

The recent update of math/eigen3 broke editors/calligra, as opencv and eigen-3.2+ contain same named parts.
Remove the using namespace Eigen from the files, and import the qualified class names instead.

This is a simplified backport of https://phabricator.kde.org/R37:2db4438671e17002c7b02e3ad3a5208930765e4f -- where they decided to qualify the classes each time (but would it would take more time to backport it -- so I prefer the sed call this time).

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rakuco added a subscriber: rakuco.

lgtm with a nitpick (inline). Did anything else break with the Eigen update? I'm almost reverting it and asking for an exp-run.

editors/calligra/Makefile
101 ↗(On Diff #29739)

Do you really need the g at the end of the expression? I'd imagine each file has a single using namespace Eigen declaration.

This revision is now accepted and ready to land.Jun 17 2017, 8:17 AM

You're right, I don't need the 'g'.

I'll build the math/eigen3 dependencies later to check for other breaks.

This revision was automatically updated to reflect the committed changes.