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

Fix build failure on macOS with --enable-shared #4776

Merged
merged 2 commits into from Aug 25, 2021

Conversation

kateinoigakukun
Copy link
Member

@kateinoigakukun kateinoigakukun commented Aug 25, 2021

./spec/ruby/optional/capi/ext/array_spec.c can match with spec/%/ if
using GNU Make under version 3.81. make command installed on macOS is
3.81, so ruby can't be built with default make on macOS with
--enable-shared option since bda56a0

This patch repairs ruby-dev-builder's daily builds

./spec/ruby/optional/capi/ext/array_spec.c can match with spec/%/ if
using GNU Make under version 3.81. make command installed on macOS is
3.81, so ruby can't be built with default make on macOS with
  --enable-shared option since ruby@bda56a0
@mame mame requested a review from nobu August 25, 2021 08:07
Copy link
Member

@nobu nobu left a comment

Choose a reason for hiding this comment

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

Since NOOP is not defined makefiles, the environment variable will be used if set.
empty set in template/Makefile.in would be safer.

defs/gmake.mk Outdated Show resolved Hide resolved
This would avoid unintentional use of the `NOOP` environment variable

Co-authored-by: Nobuyoshi Nakada <nobu@ruby-lang.org>
@kateinoigakukun kateinoigakukun marked this pull request as ready for review August 25, 2021 08:51
@mame mame merged commit 01be881 into ruby:master Aug 25, 2021
@kateinoigakukun kateinoigakukun deleted the katei/fix-macos-shared-build branch August 25, 2021 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants