Bug 252785 - Making graphics/opencv* depend on onetbb
Summary: Making graphics/opencv* depend on onetbb
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Tobias C. Berner
Depends on:
Blocks: 252648 252868 252870
  Show dependency treegraph
Reported: 2021-01-17 18:25 UTC by Ganael LAPLANCHE
Modified: 2021-02-19 21:15 UTC (History)
1 user (show)

See Also:
tcberner: maintainer-feedback+
martymac: maintainer-feedback? (tcberner)

Patch to disable TBB option (1.76 KB, patch)
2021-01-20 18:48 UTC, Ganael LAPLANCHE
no flags Details | Diff
Patch to disable TBB (1018 bytes, patch)
2021-02-03 11:35 UTC, Ganael LAPLANCHE
martymac: maintainer-approval? (tcberner)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ganael LAPLANCHE freebsd_committer 2021-01-17 18:25:57 UTC

As a maintainer of graphics/opencv-core, could you have a look at PR #252648:


In short, TBB version 2021 will soon be available as devel/onetbb but the port you maintain does not build correctly with it, see the following Poudriere run:


and error logs:


/wrkdirs/usr/ports/graphics/opencv-core/work/opencv-3.4.1/modules/core/src/parallel.cpp:92:14: fatal error: 'tbb/tbb_stddef.h' file not found
    #include "tbb/tbb_stddef.h"
1 warning and 1 error generated.

Could you try to provide a patch (important: *by replying to this PR*) to facilitate the migration to devel/onetbb ?

To help you modifying your port, expected patches to devel/tbb and devel/onetbb are available in PR #252648.

You will also need suitesparse patches from PR #252651 as suitesparse does not build with oneTbb.

Thanks for your contribution,
Best regards,

Comment 1 Ganael LAPLANCHE freebsd_committer 2021-01-17 18:45:40 UTC
Sorry, full run URL is the following:

Comment 2 Ganael LAPLANCHE freebsd_committer 2021-01-20 18:48:32 UTC
Created attachment 221773 [details]
Patch to disable TBB option

Hello Tobias,

I suggest temporarily removing TBB option while waiting for a proper fix. Would you agree with the attached patch ?
Comment 3 Tobias C. Berner freebsd_committer 2021-01-21 21:02:19 UTC
Moin moin

Sure, ship it :) -- thanks

mfg Tobias
Comment 4 commit-hook freebsd_committer 2021-01-22 11:28:06 UTC
A commit references this bug:

Author: martymac
Date: Fri Jan 22 11:27:44 UTC 2021
New revision: 562296
URL: https://svnweb.freebsd.org/changeset/ports/562296

  Opencv ports: remove TBB option to facilitate oneTbb transition

  PR:		252785
  Submitted by:	martymac
  Approved by:	tcberner (maintainer)

Comment 5 Ganael LAPLANCHE freebsd_committer 2021-01-22 11:31:50 UTC
Hello Tobias,

Patch committed, thanks!

Can you check if upstream can make it oneTbb-compatible ?

Best regards,

Comment 6 Tobias C. Berner freebsd_committer 2021-01-30 19:21:22 UTC
Moin moin

Needs to be looked at again, now that graphics/opencv is at 4.5.1.

mfg Tobias
Comment 7 Ganael LAPLANCHE freebsd_committer 2021-01-31 12:51:13 UTC
Thanks, I'll see if version 4.5.1 can be built against onetbb. Stay tuned!
Comment 8 Ganael LAPLANCHE freebsd_committer 2021-02-03 11:34:12 UTC
Hello Tobias,

Well, it is still not compatible with onetbb:


Would you agree with the attached patch ?

If yes, do you want me to commit it now or wait for the switch ?
Comment 9 Ganael LAPLANCHE freebsd_committer 2021-02-03 11:35:23 UTC
Created attachment 222109 [details]
Patch to disable TBB
Comment 10 Ganael LAPLANCHE freebsd_committer 2021-02-11 11:16:06 UTC
Hello Tobias,

Any news ? Would you agree with the attached patch ?

Thanks in advance,

Best regards,

Comment 11 Ganael LAPLANCHE freebsd_committer 2021-02-19 21:15:23 UTC
Patch committed (maintainer timeout)