Skip to content

Conversation

hsbt
Copy link
Member

@hsbt hsbt commented Jun 18, 2025

The original gemspec need to install rdoc as bundled gems at the official Ruby package.

Fix https://bugs.ruby-lang.org/issues/21312

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 18, 2025

Deploying rdoc with  Cloudflare Pages  Cloudflare Pages

Latest commit: 783adb8
Status: ✅  Deploy successful!
Preview URL: https://114ce238.rdoc-6cd.pages.dev
Branch Preview URL: https://add-gemspec-to-package.rdoc-6cd.pages.dev

View logs

@github-actions
Copy link

github-actions bot commented Jun 18, 2025

🚀 Preview deployment available at: https://114ce238.rdoc-6cd.pages.dev (commit: 783adb8)

@hsbt
Copy link
Member Author

hsbt commented Jun 18, 2025

I'm checking this change will fix to install the broken gemspec with the current Ruby HEAD.

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

rdoc.gemspec Outdated
# for ruby core repository. It was generated by
# `git ls-files -z`.split("\x0").each {|f| puts " #{f.dump}," unless f.start_with?(*%W[test/ spec/ features/ .]) }
non_lib_files = [
"rdoc.gemspec",
Copy link
Member

Choose a reason for hiding this comment

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

Could you keep this list in alphabetical order?

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure!

The original gemspec need to install rdoc as bundled gems at the
official Ruby package.

Fix https://bugs.ruby-lang.org/issues/21312
@hsbt hsbt force-pushed the add-gemspec-to-package branch from 973f2f7 to 8ba6333 Compare June 18, 2025 02:06
@hsbt
Copy link
Member Author

hsbt commented Jun 18, 2025

I'm investigating another issue for make install with rdoc installation.

The current glob pattern caused to issue with Ruby package installation.

```
/Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:1625:in 'File.stat': No such file or directory @ rb_file_s_stat - /Users/hsbt/Documents/github.com/ruby/ruby/.bundle/gems/rdoc-6.14.0/lib/English.rb (Errno::ENOENT)
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:1625:in 'block in FileUtils#install'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:2471:in 'block in FileUtils#fu_each_src_dest'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:2487:in 'FileUtils#fu_each_src_dest0'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:2469:in 'FileUtils#fu_each_src_dest'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/fileutils.rb:1624:in 'FileUtils#install'
        from ./tool/rbinstall.rb:216:in 'Object#install'
        from ./tool/rbinstall.rb:694:in 'block in RbInstall::DirPackage#extract_files'
        from ./tool/rbinstall.rb:689:in 'Array#each'
        from ./tool/rbinstall.rb:689:in 'RbInstall::DirPackage#extract_files'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/rubygems/installer.rb:853:in 'Gem::Installer#extract_files'
        from /Users/hsbt/Documents/github.com/ruby/ruby/lib/rubygems/installer.rb:312:in 'Gem::Installer#install'
        from ./tool/rbinstall.rb:746:in 'block in RbInstall::UnpackedInstaller#install'
        from ./tool/rbinstall.rb:534:in 'RbInstall.no_write'
        from ./tool/rbinstall.rb:746:in 'RbInstall::UnpackedInstaller#install'
        from ./tool/rbinstall.rb:1174:in 'block (2 levels) in <main>'
        from ./tool/rbinstall.rb:1138:in 'IO.foreach'
        from ./tool/rbinstall.rb:1138:in 'block in <main>'
        from ./tool/rbinstall.rb:1224:in 'block in <main>'
        from ./tool/rbinstall.rb:1221:in 'Array#each'
        from ./tool/rbinstall.rb:1221:in '<main>'
```
@hsbt hsbt force-pushed the add-gemspec-to-package branch from 8ba6333 to e89d2b8 Compare June 18, 2025 02:23
@hsbt
Copy link
Member Author

hsbt commented Jun 18, 2025

I confirmed 783adb8 is working fine with make install of ruby.

@hsbt hsbt merged commit 27869f5 into master Jun 18, 2025
62 checks passed
@hsbt hsbt deleted the add-gemspec-to-package branch June 18, 2025 03:20
@deivid-rodriguez
Copy link
Contributor

Thank you for fixing this @hsbt!

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.

4 participants