Skip to content

Fix missing Makefile.in in Ruby 2.7#161

Closed
qnighy wants to merge 1 commit intorake-compiler:masterfrom
qnighy:makefile-in-moved
Closed

Fix missing Makefile.in in Ruby 2.7#161
qnighy wants to merge 1 commit intorake-compiler:masterfrom
qnighy:makefile-in-moved

Conversation

@qnighy
Copy link
Copy Markdown
Contributor

@qnighy qnighy commented Nov 29, 2019

Starting from Ruby 2.7.0.preview2 (ruby/ruby@c7477c3), Makefile.in is no
longer in the root. Instead it has been moved under ./template/.

This patch fixes the problem by allowing makefiles in both places.

Note that this doesn't fix all the problems building Ruby 2.7. It fails to generate builtin_binary.inc but I'm not sure the proper way to fix it.

Starting from Ruby 2.7.0.preview2 (ruby/ruby@c7477c3), Makefile.in is no
longer in the root. Instead it has been moved under ./template/.

This patch fixes the problem by allowing makefiles in both places.
@kou kou closed this in 6822686 Dec 10, 2019
@kou
Copy link
Copy Markdown
Member

kou commented Dec 10, 2019

Thanks for your pull request.
But reusing "#{USER_HOME}/sources/#{RUBY_CC_VERSION}/Makefile.in.bak" for template/Makefile.in is ad-hok. I've fixed this by using ".../template/Makefile.in.bak for Ruby 2.7.

@qnighy
Copy link
Copy Markdown
Contributor Author

qnighy commented Dec 11, 2019

Confirmed to work. Thanks!

@larskanis
Copy link
Copy Markdown
Member

@knu Any chance to release 1.0.9? I'm just preparing rake-compiler-dock for ruby-2.7 and this patch is required.

@kou
Copy link
Copy Markdown
Member

kou commented Dec 22, 2019

Ah, sorry.
I've released it.

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.

3 participants