Skip to content

Ignore base64 gem during update_gems workflow#213

Merged
Sharpie merged 1 commit into
OpenVoxProject:mainfrom
Sharpie:update-gems-ignore-base64
Jun 24, 2026
Merged

Ignore base64 gem during update_gems workflow#213
Sharpie merged 1 commit into
OpenVoxProject:mainfrom
Sharpie:update-gems-ignore-base64

Conversation

@Sharpie

@Sharpie Sharpie commented Jun 23, 2026

Copy link
Copy Markdown
Member

Short description

Prior to this commit, the update_gems workflow would add a pkg.build_requires 'base64' statement when updating gem components. This would cause the update to fail tests as apt install rubygems-base64 or dnf install rubygems-base64 doesn't work on many platforms.

This commit restores base64 to an "ignored" status as it is, depending on Ruby version, a default or bundled gem that is always present in the installation in some form. There may be issues surrounding base64, but it seems better for a human to sort those out manually than for automation to repeatedly make the wrong decision every day at midnight.

This partially reverts commit 2dcd0db.

Checklist

I have:

Prior to this commit, the `update_gems` workflow would add a
`pkg.build_requires 'base64'` statement when updating gem components.
This would cause the update to fail tests as `apt install rubygems-base64`
or `dnf install rubygems-base64` doesn't work on many platforms.

This commit restores `base64` to an "ignored" status as it is, depending
on Ruby version, a default or bundled gem that is always present in the
installation in some form. There may be issues surrounding `base64`, but
it seems better for a human to sort those out manually than for
automation to repeatedly make the wrong decision every day at midnight.

This partially reverts commit 2dcd0db.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
@Sharpie

Sharpie commented Jun 23, 2026

Copy link
Copy Markdown
Member Author

Also, given pkg.build_requires is leading to apt or dnf being run instead of something happening with components/rubygem-base64.rb, I suspect these pkg.build_requires statements may not be doing what we expect them to be doing.

But, that is a dragon to slay another day.

@Sharpie Sharpie enabled auto-merge June 23, 2026 23:14
@Sharpie Sharpie merged commit 0bcd89e into OpenVoxProject:main Jun 24, 2026
39 checks passed
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