Skip to content

depends: avoid an unset `CMAKE_OBJDUMP`

Placeholder fanquake requested to merge github/fork/fanquake/unset_objdump into master

Similar to #31840, currently our Linux toolchain file contains:

set(CMAKE_AR "aarch64-linux-gnu-ar")
set(CMAKE_RANLIB "aarch64-linux-gnu-ranlib")
set(CMAKE_STRIP "aarch64-linux-gnu-strip")
set(CMAKE_OBJCOPY "aarch64-linux-gnu-objcopy")
set(CMAKE_OBJDUMP "")

objdump is currently only used for the macOS cross build, where it's llvm-objdump, but we should be consistent in producing a toolchain file that points to actual tools, rather than leaving variables unset.

Merge request reports