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

Adopt cleaner solution to the make/gnumake conflict. #16

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

Conversation

ke6jjj
Copy link

@ke6jjj ke6jjj commented Sep 22, 2021

This series of commits fixes builds on platforms in which "make" is not GNU-compatible. It uses the scheme that we've resolved to use for blockchain-core, which preserves the commit history for the Makefile and simultaneously reduces long-term maintenance of rebar.config.

See helium/blockchain-core#989 for the full conversation and resolution.

Part 1: Rename Makefile -> GNUmakefile

Adopt a solution to the problem where the system "make" utility is not
a GNU-compatible variant by taking advantage of the way GNU Make searches
for its Makefile differently from traditional (AT&T V7) make and redirecting
the build process to use "gmake", which is the conventional name for GNU Make
on such platforms.
Part 2: New Makefile, restore rebar.config.
@ke6jjj
Copy link
Author

ke6jjj commented Mar 3, 2022

How are we feeling about this one? Go, no go?

@madninja madninja requested a review from andymck March 4, 2022 03:40
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