Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround build task issue in Ruby core CI #1326

Merged
merged 2 commits into from
Mar 24, 2025

Conversation

st0012
Copy link
Member

@st0012 st0012 commented Mar 23, 2025

The issue is described in rubygems/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.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
The issue is described in rubygems/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.
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.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Sutou Kouhei <kou@clear-code.com>
@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
Development

Successfully merging this pull request may close these issues.

None yet

2 participants