Skip to content

[vcpkg baseline] Fix opencv4#44220

Closed
JavierMatosD wants to merge 2 commits intomicrosoft:masterfrom
JavierMatosD:fix_opencv4
Closed

[vcpkg baseline] Fix opencv4#44220
JavierMatosD wants to merge 2 commits intomicrosoft:masterfrom
JavierMatosD:fix_opencv4

Conversation

@JavierMatosD
Copy link
Contributor

Trying to fix ci failure https://dev.azure.com/vcpkg/public/_build/results?buildId=113099&view=results

Error from the logs:

/usr/local/cuda/targets/x86_64-linux/include/thrust/system/cuda/config.h:122:2: error: #error The version of CUB in your include path is not compatible with this release of Thrust. CUB is now included in the CUDA Toolkit, so you no longer need to use your own checkout of CUB. Define THRUST_IGNORE_CUB_VERSION_CHECK to ignore this.
  122 | #error The version of CUB in your include path is not compatible with this release of Thrust. CUB is now included in the CUDA Toolkit, so you no longer need to use your own checkout of CUB. Define THRUST_IGNORE_CUB_VERSION_CHECK to ignore this.
      |  ^~~~~

@JonLiu1993 JonLiu1993 changed the title Fix opencv4 [vcpkg baseline] Fix opencv4 Mar 7, 2025
@JonLiu1993 JonLiu1993 added the category:port-bug The issue is with a library, which is something the port should already support label Mar 7, 2025
@JonLiu1993 JonLiu1993 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Mar 7, 2025
@BillyONeal
Copy link
Member

It seems like we need to fix multiple CUBs getting dragged in rather than shutting up the check?

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Mar 7, 2025
cub has been 'in the CUDA box' since at least CUDA 11, and this copy conflicts with the copy bundled with CUDA 12.8 we use in the build lab:

```
/usr/local/cuda/targets/x86_64-linux/include/thrust/system/cuda/config.h:122:2: error: #error The version of CUB in your include path is not compatible with this release of Thrust. CUB is now included in the CUDA Toolkit, so you no longer need to use your own checkout of CUB. Define THRUST_IGNORE_CUB_VERSION_CHECK to ignore this.
  122 | #error The version of CUB in your include path is not compatible with this release of Thrust. CUB is now included in the CUDA Toolkit, so you no longer need to use your own checkout of CUB. Define THRUST_IGNORE_CUB_VERSION_CHECK to ignore this.
      |  ^~~~~
```

Alternative to microsoft#44220
@JavierMatosD
Copy link
Contributor Author

Closing in favor of #44245

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants