I also added an extra change to generate requirable features directly in this file list (without lib/ in front) consistently for all default gems. This way, we'll be able to eventually stop supporting the other format in RubyGems (see https://github.com/rubygems/rubygems/blob/53604cb26e23ad9b6b81b8c7eb909f5ac5d4e5fe/lib/rubygems.rb#L1182-L1215).

Sorry to late response. I'll look tomorrow.

I confirmed this change to add .so files into spec.files and remove lib/ prefix to .rb files for default gems. But I have some concerns. Sorry after my approval.

I checked all of gemspec with this PR and without one.

https://gist.github.com/hsbt/520371620754e4be7ce01d358598b6c4

I found two things:

• cgi/escape.bundle is not located on cgi.gemspec. Is it intentional?
• io/console.bundle is located with console.bundle without io/ prefix. Is it also intentional?

Thanks for checking! No, it's not expected. I will have a look and try to fix those 👍.

@hsbt I looked into both issues, I think it should be correct now!

@deivid-rodriguez Thanks. I'll look this at next week.

@deivid-rodriguez I confirmed to fix my comment.

https://gist.github.com/hsbt/91aa79a2bfdf14ba95cb83e262ab41b8

Let's try this.

This change leads to fail with cross-build. https://rubyci.s3.amazonaws.com/crossruby/crossruby-master-arm/log/20240325T034328Z.fail.html.gz

I'll look that.

Thanks! Let me know if help is needed with that failure.

This breaks Windows builds.

Below is only ucrt build, note that no extension gems are listed. First noticed in ruby-loco, as the 'CLI test' failed.

Windows mswin builds have the same problem, but the 'Windows' CI doesn't run make install. I am just now looking at the code...

Last master CI before this was committed:
https://github.com/ruby/ruby/actions/runs/8413725202/job/23036373402#step:9:293

PR CI:
https://github.com/ruby/ruby/actions/runs/8374059044/job/22928452926#step:9:294

CI in master:
https://github.com/ruby/ruby/actions/runs/8414422750/job/23038070877#step:9:290

@MSP-Greg Thanks. I fixed it with b39057f

@hsbt Thank you for the quick fix. As you've probably seen, fixed the CI here. All three ruby-loco builds also passed...

Thank you both!