Skip to content
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.

Sign up for GitHub

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

Jump to bottom

md4c: fix building for Apple non-macOS #26248

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

md4c: fix building for Apple non-macOS #26248

wants to merge 2 commits into from

Conversation

kambala-decapitator
Copy link
Contributor

@kambala-decapitator kambala-decapitator commented Dec 22, 2024 •
edited
Loading

Summary

Changes to recipe: md4c/all

Motivation

Fixes building for Apple non-macOS

Details

Building the package with default options for iOS produces the following CMake configuration error:

CMake Error at md2html/CMakeLists.txt:14 (install):
  install TARGETS given no BUNDLE DESTINATION for MACOSX_BUNDLE executable
  target "md2html".

As executable for Apple non-macOS doesn't really make any sense, it's disabled and the corresponding option is deleted. Basically same fix as #25964

Also, the option BUILD_MD2HTML_EXECUTABLE appeared only in v0.5.0 (see mity/md4c@ecce171), this is now checked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kambala-decapitator