From: eregontp@... Date: 2020-04-10T11:56:37+00:00 Subject: [ruby-core:97792] [Ruby master Feature#16688] Allow #to_path object as argument to system() Issue #16688 has been updated by Eregon (Benoit Daloze). mame (Yusuke Endoh) wrote in #note-9: > I'd like to confirm: `system(Pathname("cat /etc/passwd"))` should attempt to execute a file whose path is `./cat\ /etc/passwd`, not attempt to show the content of `/etc/passwd`, right? Definitely the first, we should take advantage that we know it's supposed to be a path and a not a command line with spaces. The second would be waiting to become a security vulnerability. ---------------------------------------- Feature #16688: Allow #to_path object as argument to system() https://bugs.ruby-lang.org/issues/16688#change-85009 * Author: Dan0042 (Daniel DeLorme) * Status: Open * Priority: Normal ---------------------------------------- I often work with Pathname objects, but when passing them to a system command I find it a bit tedious that they have to be explicitly converted to a String. ```ruby file = BASE + "config.json" system(@cmd, file) #=> TypeError (no implicit conversion of Pathname into String) system(@cmd, file.to_s) #=> works ``` I propose that the system/exec/spawn family of methods should try to convert their arguments using `to_path`, if `to_str` fails. I believe it makes perfect sense, since commandline arguments are so often pathnames. This includes in/out redirection. `system("ls", out: Pathname.new("file"))` should be valid. -- https://bugs.ruby-lang.org/ Unsubscribe: