Skip to content

Conversation

@st0012
Copy link
Member

@st0012 st0012 commented Mar 23, 2025

The issue is described in ruby/rubygems#8477

The issue fails the workflow whenever RDoc's version is bumped as there will then be a mismatch between the target file name and the version that's actually built.

For example:

rdoc 6.13.0 built to pkg/rdoc-6.12.0.gem.

This commit works around the issue by manually building the gem and installing it locally so the latest installed version will match the version contained in the gemspec file.

The issue is described in ruby/rubygems#8477

The issue fails the workflow whenever RDoc's version is bumped as there
will then be a mismatch between the target file name and the version
that's actually built.

For example:

```
rdoc 6.13.0 built to pkg/rdoc-6.12.0.gem.
```

This commit works around the issue by manually building the gem and
installing it locally so the latest installed version will match the
version contained in the gemspec file.
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 23, 2025

Deploying rdoc with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5a0710a
Status: ✅  Deploy successful!
Preview URL: https://8fcdde6d.rdoc-6cd.pages.dev
Branch Preview URL: https://workaround-rubygems-issue-on.rdoc-6cd.pages.dev

View logs

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

Comment on lines 58 to 59
gem build rdoc.gemspec
gem install rdoc-*.gem
Copy link
Member

Choose a reason for hiding this comment

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

Do we need bundle exec ... here?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don’t think so…? I don’t usually run gem with bundle as it usually doesn’t need any of the project’s dependencies. Is it a recommended practice?

Copy link
Member

Choose a reason for hiding this comment

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

Ah, sorry. You're right.

Our .gemspec doesn't use external libraries. So we don't need bundle exec here.

Comment on lines 58 to 59
gem build rdoc.gemspec
gem install rdoc-*.gem
Copy link
Member

Choose a reason for hiding this comment

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

Ah, sorry. You're right.

Our .gemspec doesn't use external libraries. So we don't need bundle exec here.

@st0012 st0012 merged commit d5ec331 into master Mar 24, 2025
60 checks passed
@st0012 st0012 deleted the workaround-rubygems-issue-on-ruby-core branch March 24, 2025 20:44
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.

2 participants