[ruby-core:90179] [Ruby trunk Bug#15354] Remove Bundler from StdLib

[Greg.mpls@gmail.com]

Issue #15354 has been updated by MSP-Greg (Greg L).


@colby

Ok.  I decided on the longer message.

> snide comment

Snide - 'derogatory or mocking in an indirect way'.  @vo.x's comment is direct, and his point about the vendored libraries is valid.

> (Bundler) has had to deal with lots of issues from the RubyGems/Ruby project

Something just seems backward...

Just a few minutes ago, I came across a test in RubyGems that had three nested unless/end pairs, and only the innermost had any code.  So, I thought about a PR to clean it up; my first thought was who would ask 'what does this fix?'.

Also in RG, IMHO, there's code where process x and process y have become so tangled that they need to be pulled apart to allow other code to work correctly.  The tangling also mucks up testing.  I've procrastinated just because...

Conversely, here in ruby/ruby, the attitude is much more about improvement of any type.

Moving on, what are your thoughts on:

1. In what way should ruby/RubyGems/Bundler be integrated?
2. What to do about the vendored/duplicated libraries?
3. Given your involvement in both Bundler & RubyGems, and assuming infinite resources, what should change?

Thanks, Greg

----------------------------------------
Bug #15354:  Remove Bundler from StdLib
https://bugs.ruby-lang.org/issues/15354#change-75302

* Author: vo.x (Vit Ondruch)
* Status: Assigned
* Priority: Normal
* Assignee: hsbt (Hiroshi SHIBATA)
* Target version: 
* ruby -v: ruby 2.6.0dev (2018-11-26 trunk 65990) [x86_64-linux]
* Backport: 2.4: UNKNOWN, 2.5: UNKNOWN
----------------------------------------
[This is mostly a clone of #13978 which I opened a year ago. Unfortunately, the same points I mentioned there still hold true.]

I understand that given that almost every Ruby user is using Bundler, it would be convenient for quite some people to have Bundler as part of StdLib.

However, seeing two copies of Molinillo, each in a different version [1], [2], similarly to fileutils [3], [4], is a sign that things are not as they should be. Therefore, please consider removal of Bundler from StdLib unless upstream demonstrates it can maintain it properly.


[1]: https://github.com/ruby/ruby/blob/trunk/lib/bundler/vendor/molinillo/lib/molinillo/gem_metadata.rb#L5
[2]: https://github.com/ruby/ruby/blob/trunk/lib/rubygems/resolver/molinillo/lib/molinillo/gem_metadata.rb#L4
[3]: https://github.com/ruby/ruby/blob/trunk/lib/bundler/vendor/fileutils/lib/fileutils.rb
[4]: https://github.com/ruby/ruby/blob/trunk/lib/fileutils.rb



-- 
https://bugs.ruby-lang.org/