FS#72971 - [usd] incompatible with tbb:2021

Attached to Project: Community Packages
Opened by bartus (bartus) - Friday, 10 December 2021, 21:05 GMT
Last edited by Sven-Hendrik Haase (Svenstaro) - Sunday, 19 December 2021, 14:20 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Sven-Hendrik Haase (Svenstaro)
Architecture All
Severity High
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: usd:21.08 using tbb::atomic which was removed in oneTBB `tbb:2021.4`
This task depends upon

Closed by  Sven-Hendrik Haase (Svenstaro)
Sunday, 19 December 2021, 14:20 GMT
Reason for closing:  Won't fix
Additional comments about closing:  I dropped usd as it doesn't build and upstream doesn't seem to care too much.
Comment by Sven-Hendrik Haase (Svenstaro) - Saturday, 11 December 2021, 17:00 GMT
The problem is basically that upstream doesn't seem to care too much: https://github.com/PixarAnimationStudios/USD/issues/1650

Not sure whether we should maybe just drop USD from the repos. Doesn't appear to get fixed any time soon and the fix doesn't appear like it could be a trivial API change.
Comment by loqs (loqs) - Saturday, 11 December 2021, 19:23 GMT
tbb::task_scheduler_init to tbb::global_control and tbb::mutex to std::mutex are trivial changes.
As you say tbb:atomic to std:atomic [1] (std:atomic does not support compare exchange with an atomic argument?) and tbb:task to ? [2] are none trivial.

[1] https://github.com/PixarAnimationStudios/USD/blob/v21.11/pxr/usdImaging/usdImaging/resolvedAttributeCache.h#L363
[2] https://github.com/PixarAnimationStudios/USD/blob/v21.11/pxr/base/work/dispatcher.h#L136
Comment by bartus (bartus) - Sunday, 12 December 2021, 11:05 GMT
""""Drop the Kinect"""" ¯\_(ツ)_/¯

Loading...