ViSP bottle needs to be updated

[fspindle]

Output of brew config

% brew config
HOMEBREW_VERSION: 4.4.9
ORIGIN: https://github.com/Homebrew/brew
HEAD: 2e780004c92f16a96036d3d43569c2924c4e54f5
Last commit: 3 days ago
Branch: stable
Core tap JSON: 04 Dec 08:34 UTC
Core cask tap JSON: 04 Dec 08:34 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.hIAhMy71s4/org.xquartz:0
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 3.3.6 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.3.6/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 16.0.0 build 1600
Git: 2.47.1 => /opt/homebrew/bin/git
Curl: 8.7.1 => /usr/bin/curl
macOS: 14.7.1-arm64
CLT: 16.1.0.0.1.1729049160
Xcode: 16.1
Rosetta 2: false

Output of brew doctor

% brew doctor
Your system is ready to brew.

Description of issue

ViSP bottle is no more compatible with a recent update of OpenCV bottle (OpenCV is a dependency of ViSP).
See lagadic/visp#1515

$ brew install opencv

will bring OpenCV installation in /opt/homebrew/Cellar/opencv/4.10.0_14/ folder while ViSP installed with

$ brew install visp

will contain in /opt/homebrew/Cellar/visp/3.6.0_6/lib/cmake/visp/VISPConfig.cmake a reference to /opt/homebrew/Cellar/opencv/4.10.0/include/opencv4 instead of /opt/homebrew/Cellar/opencv/4.10.0_14/include/opencv4

Can some one (@cho-m, @issyl0, @chenrui333, @branchvincent, @fxcoudert ...) update visp bottle ?

[p-linnane]

Thanks for the report. I've opened Homebrew/homebrew-core#200220 to rebuild the bottle.

[sulibo]

OpenCV was updated two days ago, now it is 4.10.0_15, see update 4.10.0_15 bottle. Is it possible to update the visp bottle accordingly? Thanks!

[p-linnane]

If visp needs to rebuilt every time opencv is, then we need to update the visp test to fail so we can catch this.

[alebcay]

Can you share more about what the cmake file looks like for which you are seeing the issues? Tentatively, I have tried something like the following and it seems to be working fine (maybe I'm not testing a part that relies on OpenCV):

cmake_minimum_required(VERSION 3.10 FATAL_ERROR)
project(visp-check LANGUAGES CXX)
find_package(VISP REQUIRED visp_core)
include_directories(${VISP_INCLUDE_DIRS})
add_executable(visp-check test.cpp)
target_link_libraries(visp-check ${VISP_LIBRARIES})

[sulibo]

Sure, here is the cmake file I used:

cmake_minimum_required(VERSION 3.10)
project(tutorial-image)
find_package(VISP REQUIRED)
include_directories(${VISP_INCLUDE_DIRS})
add_executable(tutorial-viewer tutorial-viewer.cpp)
target_link_libraries(tutorial-viewer ${VISP_LIBRARIES})

More details can be found at lagadic/visp#1515.

[cho-m]

More details can be found at lagadic/visp#1515.

There aren't any details on running with the fixed bottle (Homebrew/homebrew-core#200267).

---

Can you make sure you are on the latest revision of visp (brew info visp should show /usr/local/Cellar/visp/3.6.0_8) and provide output from running CMake