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

Add irb as a runtime dependency #536

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bastelfreak
Copy link

prybug depends on irb:

require "irb"

irb isn't bundled with ruby, starting with ruby2.6. irb is a standalone
gem, so we cannot rely on it being present on the target system.

prybug depends on irb: https://github.com/deivid-rodriguez/byebug/blob/d971a2b40807faf4c8654b204ddc2b30d90351fe/lib/byebug/commands/irb.rb#L4
irb isn't bundled with ruby, starting with ruby2.6. irb is a standalone
gem, so we cannot rely on it being present on the target system.
@deivid-rodriguez
Copy link
Owner

@bastelfreak irb is a default gem in ruby 2.6. That means it can't be removed, so I think we can assume it's present, actually.

@bastelfreak
Copy link
Author

Is is actually a default gem? At least on archlinux it isn't bundled with ruby2,6, as far as I know. It's a stand alone gem.

@deivid-rodriguez
Copy link
Owner

I thought so, but maybe I'm wrong... Either that, or a packaging bug? 🤔

@MSP-Greg
Copy link
Collaborator

MSP-Greg commented Feb 4, 2019

I thought so, but maybe I'm wrong

You're correct.

See https://github.com/ruby/ruby/tree/v2_6_0/lib/irb, note the irb.gemspec file. Hence, it is a default gem, but packagers can change all sorts of things. For instance, see Travis...

All Ruby 'bundled gems' are listed in https://github.com/ruby/ruby/blob/v2_6_0/gems/bundled_gems

@deivid-rodriguez
Copy link
Owner

@deivid-rodriguez
Copy link
Owner

I opened a ticket about it: https://bugs.archlinux.org/task/62709

@deivid-rodriguez
Copy link
Owner

Hi @bastelfreak. archlinux has readded irb as a default dependency of their ruby package, so I think we can close this?

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.

None yet

3 participants