This repository has been archived by the owner on Jul 4, 2023. It is now read-only.
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
shocco: move hard markdown dep to Requirement
schocco requires a `markdown` command. The previous version of the formula checked for the markdown formula. However, there's an edge case involved. The discount formula installs a command `markdown` which can also satisfy shocco's requirement. (And I would add that discount's implementation of the `markdown` command is far superior to markdown's, though I grant that's debatable.) This update checks for any `markdown`-providing install, and if one is not found tells the user what options are available. I think this is better than just installing the markdown formula. First, if discount is already installed, then when `brew` tries to install markdown, the linking stage will fail. Although the shocco install as a whole will go through, the failure message about linking markdown is likely to confuse the user. Also, if discount is already installed, that problably means the user doesn't want the markdown formula. Finally, this way of handling things is more easily extended if other formulas become available which provide a valid `markdown` command. Closes #16740. Signed-off-by: Misty De Meo <mistydemeo@gmail.com>
- Loading branch information