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.

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

Fixing math/calc build failures. #1482

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

Conversation

fac3plant
Copy link
Contributor

This patch adds a 'DragonFly' build target which allows math/calc to build properly.

This patch adds a 'DragonFly' build target which allows math/calc to build properly.
+#
+MAKE= gmake
+
+endif # ($(target),FreeBSD)
Copy link
Member

@daftaupe daftaupe Oct 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wouldn't it make more sense to have endif # ($(target),DragonFly) here ?
even if it's just a comment :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would make more sense to do that; I just didn't see it. Thank you for catching it, and that correction has been made.

This patch corrects a missed #endif statement, changing it from `endif     # ($(target), DragonFly)`
The patch doesn't necessarily "do" anything, however, it does communicate what is going on in the Makefile better.
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.

2 participants