[ruby-core:101710] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[ruby-core:101710] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[hsbt]

Issue #17473 has been reported by hsbt (Hiroshi SHIBATA).

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101711] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[eregontp]

Issue #17473 has been updated by Eregon (Benoit Daloze).


If we do so, could we actually define most of Pathname in Ruby, and not in C?

Right now, https://github.com/ruby/ruby/blob/3fc53de5c961cc8fa2b6acbd63874b89fe709520/ext/pathname/pathname.c is essentially just a bunch of rb_funcall() calls, which are no faster than Ruby code, but makes it harder to reuse on other implementations, and less readable and harder to maintain.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89534

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101738] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[mame]

Issue #17473 has been updated by mame (Yusuke Endoh).

Assignee set to akr (Akira Tanaka)

The proposal lacks one background, so I'd like to add it. Rubygems cannot allow uesrs to choose the version of a gem that rubygems itself are using. So, we want to make Rubygems independent with any gems. According to @hsbt, Rubygems is using Pathname, FileUtils, Tsort, etc. Though FileUtils and Tsort are relatively easy to be removed from the dependencies, Pathname looks difficult.

In addition, of course, Pathname is widely used, and the API of Pathname looks stable. So I also think it deserves built-in.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89560

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101748] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[eregontp]

Issue #17473 has been updated by Eregon (Benoit Daloze).


I forgot to mention, +1 from me for Pathname in core, if it's written mostly in Ruby.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89570

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101756] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[daniel]

Issue #17473 has been updated by Dan0042 (Daniel DeLorme).


mame (Yusuke Endoh) wrote in #note-2:
> Though FileUtils and Tsort are relatively easy to be removed from the dependencies, Pathname looks difficult.

Isn't Pathname dependent on FileUtils though?

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89583

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101762] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[mame]

Issue #17473 has been updated by mame (Yusuke Endoh).


Dan0042 (Daniel DeLorme) wrote in #note-4:
> Isn't Pathname dependent on FileUtils though?

Yes. It uses only `FileUtils.mkpath` and `rm_r`. Rubygems also uses them, so @hsbt is preparing to propose making a very limited set of FileUtils methods built-in, too.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89588

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101778] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[eregontp]

Issue #17473 has been updated by Eregon (Benoit Daloze).


I thought about FileUtils too, but it's required lazily for these two methods.
So I think it might be fine to move only Pathname to core, and accept that `Pathname#{mkpath,rmtree}` `require`s `fileutils` when used.

I think having part of FileUtils defined in core would be confusing.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89606

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101813] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[shyouhei]

Issue #17473 has been updated by shyouhei (Shyouhei Urabe).


JFYI pathname was born as a pure-ruby library, then was eventually translated into C (in commit:4bf3cb5ba9c0242bd5a6d0d55b7db9f837c09edf).  Don't know the reason behind that move though.  @akr do you remember?

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89640

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101814] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[eregontp]

Issue #17473 has been updated by Eregon (Benoit Daloze).


shyouhei (Shyouhei Urabe) wrote in #note-7:
> JFYI pathname was born as a pure-ruby library, then was eventually translated into C (in commit:4bf3cb5ba9c0242bd5a6d0d55b7db9f837c09edf).  Don't know the reason behind that move though.  @akr do you remember?

I know, I'd like to undo that change (I can make a PR), except for the parts which are significantly faster in C (I think only `Pathname#<=>` / `path_cmp`).
Then, we could share more of Pathname between Ruby implementations, and avoid maintaining both a Ruby and C version.
There is a copy in [TruffleRuby](https://github.com/oracle/truffleruby/blob/5ab35083f1cebd44b05ad5e8706196118184c328/lib/truffle/pathname.rb), in JRuby and in Rubinius right now due to that commit :/
@akr Would that be OK?

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89641

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101826] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[hsbt]

Issue #17473 has been updated by hsbt (Hiroshi SHIBATA).


>Then, we could share more of Pathname between Ruby implementations, and avoid maintaining both a Ruby and C version.

I'm not sure how embedded pure-Ruby implementation to core classes. The above request is the different request.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89652

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:101835] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[duerst]

Issue #17473 has been updated by duerst (Martin Dürst).


hsbt (Hiroshi SHIBATA) wrote in #note-9:

> I'm not sure how embedded pure-Ruby implementation to core classes. The above request is the different request.

I support making Pathname part of core Ruby. It's an extremely convenient library that I use very often.

Just for the record, String#unicode_normalize is part of a core class but is implemented in pure Ruby. It's a single case, and includes loading code on demand. If we do more pure-Ruby implementations for core classes, we should think again about the best way to do it. 

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89665

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

Unsubscribe: <mailto:ruby-core-request@ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>

[ruby-core:101848] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[eregontp]

Issue #17473 has been updated by Eregon (Benoit Daloze).


hsbt (Hiroshi SHIBATA) wrote in #note-9:
> I'm not sure how embedded pure-Ruby implementation to core classes. The above request is the different request.

It's also related, if the decision was to move all of Pathname to C (due to some reason of moving it to core), I would be strongly against it.
I think concretely we can just have `pathname.rb` at the root, similar to e.g. `dir.rb`.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89681

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:102012] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[akr]

Issue #17473 has been updated by akr (Akira Tanaka).


shyouhei (Shyouhei Urabe) wrote in #note-7:
> JFYI pathname was born as a pure-ruby library, then was eventually translated into C (in commit:4bf3cb5ba9c0242bd5a6d0d55b7db9f837c09edf).  Don't know the reason behind that move though.  @akr do you remember?

I had a plan that Pathname contains a FD optionally as root directory and 
use openat() and other *at system call.
No progress after translation to C, though


----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-89860

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:106989] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze)]

Issue #17473 has been updated by Eregon (Benoit Daloze).


I plan to make a PR to move Pathname back to Ruby for all methods which don't significantly gain from being written in C.
I'm coming from https://github.com/oracle/truffleruby/issues/2559, which is additional motivation for it, the current situation hurts compatibility between Ruby impls for seemingly no gain.

We can still decide to keep pathname as stdlib or move it to core either way (there is already `ext/pathname/lib/pathname.rb` anyway, it'd just become bigger and `ext/pathname/pathname.c` smaller).

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-95820

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:106990] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze)]

Issue #17473 has been updated by Eregon (Benoit Daloze).


Regarding openat(), [4bf3cb5ba9c0242bd5a6d0d55b7db9f837c09edf](https://bugs.ruby-lang.org/projects/ruby-master/repository/git/revisions/4bf3cb5ba9c0242bd5a6d0d55b7db9f837c09edf) was 11 years ago, so I assume there is little demand or need.
It's easy enough to just use absolute paths before `chdir` (or avoid using them after chdir) or keep the old CWD as a Pathname instance, and of course some platforms don't have `openat()`.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-95821

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:106999] [Ruby master Feature#17473] Make Pathname to embedded class of Ruby

[deivid]

Issue #17473 has been updated by deivid (David Rodríguez).


For what it's worth, although for unrelated reasons, I completely rewrote `Pathname` in pure Ruby here: https://github.com/rubygems/rubygems/pull/4992.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-95829

* Author: hsbt (Hiroshi SHIBATA)
* Status: Open
* Priority: Normal
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



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

[ruby-core:122585] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


I (finally) made a PR to define most of Pathname in Ruby: https://github.com/ruby/pathname/pull/53

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-113820

* Author: hsbt (Hiroshi SHIBATA)
* Status: Assigned
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122703] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[matz (Yukihiro Matsumoto) via ruby-core]

Issue #17473 has been updated by matz (Yukihiro Matsumoto).


Regarding the original proposal, I'd accept making Pathname built-in, considering its wide accepting among the community, especially with Rails.
Whether to keep the built-in Pathname in the C implementation or implement it in Ruby is left for future discussion.

Matz.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-113980

* Author: hsbt (Hiroshi SHIBATA)
* Status: Assigned
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122776] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[hsbt (Hiroshi SHIBATA) via ruby-core]

Issue #17473 has been updated by hsbt (Hiroshi SHIBATA).

Status changed from Assigned to Closed

I made `Pathname` to embedded core class now. 

* https://github.com/ruby/ruby/pull/13873
* https://github.com/ruby/ruby/pull/13885
* https://github.com/ruby/ruby/pull/13887

`Pathname#find`, `Pathname#mkpath`, `Pathname#rmtree` and `Pathname.mktmpdir` is remained on `lib/pathname.rb`. So, the users need to require `pathname.rb` for that methods. We should migrate them to `pathname_builtin.rb` or `pathname.c` without extra dependencies.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114054

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122777] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[deivid via ruby-core]

Issue #17473 has been updated by deivid (David Rodríguez).


Would it make sense to make `find`, `fileutils`, and `tmpdir` also builtin, given that `pathname` depends on them?

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114055

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122781] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


Discussion at https://github.com/ruby/dev-meeting-log/blob/master/2025/DevMeeting-2025-07-10.md#feature-17473-make-pathname-to-embedded-class-of-ruby-eregon

I am disappointed, https://github.com/ruby/pathname/pull/53 makes the code clearly more maintainable and a lot easier to read, there is no value to have it in C.
I don't see much value to have Pathname as a core class, `require "pathname"` is no big deal and since it depends on other gems & stdlib it's awkward.
Having a partially-defined Pathname in core (missing some methods) seems the worst outcome.

As pathname is already a gem, it also means we'll have two unsynchronized copies of the code, and with enough time maybe subtle behavior differences, this seems something nobody wants.

@matz this decision is adding a lot of work to CRuby, and other Ruby implementations.
Pathname or most of it should be defined in Ruby, having multiple copies of it in C, Java, Ruby, etc means useless duplication and things will get out of sync needlessly.

> @nobu: if Pathname is embeded to the core, I prefer the C implementation to Ruby impl.

Why? I see only disadvantages to that.

> @ioquatix for the purpose of JIT isn't it better to write more Ruby in Ruby?
> @shyouhei nobody wants to JIT a path manipulation.

https://github.com/ruby/pathname/pull/53#issuecomment-2989134661 shows the Ruby code is faster than the C extension for Pathname.
Even from a performance point of view it's bad to implement Pathname methods in C (for most methods).

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114060

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122782] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


I have merged https://github.com/ruby/pathname/pull/53, it is clearly better for the maintenance of the gem at least.
Now we need to figure out if we want to keep the copy in core in sync with the gem or not (if not it's going to be a "fun" gotcha that the pathname gem is faster than core Pathname).

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114061

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122791] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Dan0042 (Daniel DeLorme) via ruby-core]

Issue #17473 has been updated by Dan0042 (Daniel DeLorme).


> * akr: To use the following four methods, require "pathname" is needed for a while
>   *  find, mkpath, rmtree, mktmpdir
>   *  rationale: they require other libraries (find gem, fileutils, tmpdir)
>   *  in future, they (except mktmpdir) should be rewritten in pure C (or rbinc) without dependency to find and fileutils 

I don't understand this at all. What's the problem with having `require "fileutils"` inside #mkpath ? Those dependencies are part of the stdlib anyway, so it's not like they could be missing. Is it for some reason forbidden to call `require` in a method of a core class?

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114072

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122793] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[hsbt (Hiroshi SHIBATA) via ruby-core]

Issue #17473 has been updated by hsbt (Hiroshi SHIBATA).


>I have merged https://github.com/ruby/pathname/pull/53, it is clearly better for the maintenance of the gem at least.

I reverted that. Please don't merge without consensus. At least, no one strongly against migrate C to Ruby in Dev Meetings. 

The transition from C to Ruby should be done gradually, in reviewable chunks.


----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114074

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122795] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


Dan0042 (Daniel DeLorme) wrote in #note-23:
> I don't understand this at all. What's the problem with having `require "fileutils"` inside #mkpath ? Those dependencies are part of the stdlib anyway, so it's not like they could be missing. Is it for some reason forbidden to call `require` in a method of a core class?

I had the exact same thought, this seems the obvious solution.


----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114076

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122796] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


hsbt (Hiroshi SHIBATA) wrote in #note-24:
> >I have merged https://github.com/ruby/pathname/pull/53, it is clearly better for the maintenance of the gem at least.
> 
> I reverted that. Please don't merge without consensus. At least, no one strongly against migrate C to Ruby in Dev Meetings. 
> 
> The transition from C to Ruby should be done gradually, in reviewable chunks.

That PR is actually easy to review as explained in its description, it restores pathname.rb as it used to be and includes all changes to pathname.rb since then, see https://github.com/ruby/pathname/pull/53/files/3736eab91f783cb087f873b99c25ffb7633ed041.
IOW, it only adds code in pathname.rb and removes code in pathname.c, and then has separate commits for all other changes.
Making multiple smaller PRs does not make sense, especially since the Ruby code is just the one from before the migration to C.

I remade the PR as https://github.com/ruby/pathname/pull/57, please review it and share your concerns there.

What is the plan for the code in the gem and the code in core?
Will they be sync'd, how?
>From what I have seen changes in core have been done seemingly without caring about the gem, so it seemed the intention was to not sync but effectively fork?
We also cannot make the gem empty in general as the gem needs to keep working on Ruby 3.4 and older at least.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114077

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122812] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[hsbt (Hiroshi SHIBATA) via ruby-core]

Issue #17473 has been updated by hsbt (Hiroshi SHIBATA).


Eregon (Benoit Daloze) wrote in #note-26:

> That PR is actually easy to review as explained in its description, it restores pathname.rb as it used to be and includes all changes to pathname.rb since then, see https://github.com/ruby/pathname/pull/53/files/3736eab91f783cb087f873b99c25ffb7633ed041.
> IOW, it only adds code in pathname.rb and removes code in pathname.c, and then has separate commits for all other changes.
> Making multiple smaller PRs does not make sense, especially since the Ruby code is just the one from before the migration to C.

Please separate the small PRs. I want to reduce the side effect like https://github.com/ruby/ruby/pull/13906.

And if you have another issue or proposal, please file new issue.


----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114099

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/

[ruby-core:122814] [Ruby Feature#17473] Make Pathname to embedded class of Ruby

[Eregon (Benoit Daloze) via ruby-core]

Issue #17473 has been updated by Eregon (Benoit Daloze).


hsbt (Hiroshi SHIBATA) wrote in #note-27:
> Please separate the small PRs. I want to reduce the side effect like https://github.com/ruby/ruby/pull/13906.

Let's discuss your concerns about the PR on https://github.com/ruby/pathname/pull/57 directly.

----------------------------------------
Feature #17473: Make Pathname to embedded class of Ruby
https://bugs.ruby-lang.org/issues/17473#change-114101

* Author: hsbt (Hiroshi SHIBATA)
* Status: Closed
* Assignee: akr (Akira Tanaka)
----------------------------------------
pathname is one of most useful utility class of Ruby. I'm happy to use Pathname without require it.

Any thought?



-- 
https://bugs.ruby-lang.org/
______________________________________________
 ruby-core mailing list -- ruby-core@ml.ruby-lang.org
 To unsubscribe send an email to ruby-core-leave@ml.ruby-lang.org
 ruby-core info -- https://ml.ruby-lang.org/mailman3/lists/ruby-core.ml.ruby-lang.org/