rubygems / bundler

Manage your Ruby application's gem dependencies
https://bundler.io
MIT License
4.88k stars 1.99k forks source link

Don't ignore resolved platform information #7578

Closed kou closed 4 years ago

kou commented 4 years ago

This is a follow-up change of #7522.

What was the end-user problem that led to this PR?

The problem was installing wrong platform gem even when resolver resolves the correct platform gem.

See also:

What was your diagnosis of the problem?

My diagnosis was Bundler::LazySpecification#__materialize__ doesn't care about the resolved platform information.

What is your fix for the problem, implemented in this PR?

My fix cares about the resolved platform information.

Why did you choose this fix out of the possible options?

I chose this fix because we should respect the resolved platform information to install correct gem.

deivid-rodriguez commented 4 years ago

@bundlerbot merge

ghost commented 4 years ago

Build succeeded