Skip to content

fix(main/mariadb): do not attempt to run a test binary during build if fmt is detected #25130

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robertkirkman
Copy link
Contributor

@robertkirkman robertkirkman commented Jun 21, 2025

-- Performing Test HAVE_SYSTEM_LIBFMT
CMake Error: try_run() invoked in cross-compiling mode, please set the following cache variables appropriately:
   HAVE_SYSTEM_LIBFMT_EXITCODE (advanced)
For details see /home/builder/.termux-build/mariadb/build/TryRunResults.cmake
-- Performing Test HAVE_SYSTEM_LIBFMT - Failed
  • After this change, if mariadb happens to build while fmt is present in $TERMUX_PREFIX,
    • no error will happen at build-time,
    • and additionally, no dependency on libfmt.so will be added to any file in mariadb,
    • verified using the command for f in $(dpkg -L mariadb); do ldd $f 2>/dev/null | grep fmt; done

…f `fmt` is detected

- Progress on termux#23492

- Fixes this error in the command `scripts/run-docker.sh ./build-package.sh -I -f fmt mariadb`:

```
-- Performing Test HAVE_SYSTEM_LIBFMT
CMake Error: try_run() invoked in cross-compiling mode, please set the following cache variables appropriately:
   HAVE_SYSTEM_LIBFMT_EXITCODE (advanced)
For details see /home/builder/.termux-build/mariadb/build/TryRunResults.cmake
-- Performing Test HAVE_SYSTEM_LIBFMT - Failed
```

- After this change, if `mariadb` happens to build while `fmt` is
  present in `$TERMUX_PREFIX`,
  - no error will happen at build-time,
  - and additionally, no dependency on `libfmt.so` will be added to any file
  in `mariadb`,
  - verified using the command `for f in $(dpkg -L mariadb); do ldd $f 2>/dev/null | grep fmt; done`

[no ci]
@robertkirkman
Copy link
Contributor Author

I will merge it 24 hours from now if no problems are found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant