feat: with_command() to obtain the command that opens a program wit…

…h a given application.

src/haiku.rs

@@ -1,16 +1,13 @@
-use std::{ffi::OsStr, io, process::Command};
-
-use crate::{CommandExt, IntoResult};
+use std::{ffi::OsStr, process::Command};
 
 pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     let mut cmd = Command::new("/bin/open");
     cmd.arg(path.as_ref());
     vec![cmd]
 }
 
-pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
+pub fn with_command<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
     let mut cmd = Command::new(app.into());
-    cmd.arg(path.as_ref())
-        .status_without_output()
-        .into_result(&cmd)
+    cmd.arg(path.as_ref());
+    cmd
 }

src/ios.rs

@@ -1,19 +1,16 @@
-use std::{ffi::OsStr, io, process::Command};
-
-use crate::{CommandExt, IntoResult};
+use std::{ffi::OsStr, process::Command};
 
 pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     let mut cmd = Command::new("uiopen");
     cmd.arg("--url").arg(path.as_ref());
     vec![cmd]
 }
 
-pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
+pub fn with_command<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
     let mut cmd = Command::new("uiopen");
     cmd.arg("--url")
         .arg(path.as_ref())
         .arg("--bundleid")
-        .arg(app.into())
-        .status_without_output()
-        .into_result(&cmd)
+        .arg(app.into());
+    cmd
 }

src/lib.rs

@@ -148,7 +148,8 @@ pub fn that<T: AsRef<OsStr>>(path: T) -> io::Result<()> {
 /// A [`std::io::Error`] is returned on failure. Because different operating systems
 /// handle errors differently it is recommend to not match on a certain error.
 pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
-    os::with(path, app)
+    let mut cmd = with_command(path, app);
+    cmd.status_without_output().into_result(&cmd)
 }
 
 /// Get multiple commands that open `path` with the default application.
@@ -168,6 +169,21 @@ pub fn commands<'a, T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     os::commands(path)
 }
 
+/// Get a command that uses `app` to open `path`.
+///
+/// # Examples
+///
+/// ```no_run
+/// # fn main() -> Result<(), Box<dyn std::error::Error>> {
+/// let path = "http://rust-lang.org";
+/// assert!(open::with_command(path, "app").status()?.success());
+/// # Ok(())
+/// # }
+/// ```
+pub fn with_command<'a, T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
+    os::with_command(path, app)
+}
+
 /// Open path with the default application in a new thread to assure it's non-blocking.
 ///
 /// See documentation of [`that()`] for more details.

src/macos.rs

@@ -1,18 +1,13 @@
-use std::{ffi::OsStr, io, process::Command};
-
-use crate::{CommandExt, IntoResult};
+use std::{ffi::OsStr, process::Command};
 
 pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     let mut cmd = Command::new("/usr/bin/open");
     cmd.arg(path.as_ref());
     vec![cmd]
 }
 
-pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
+pub fn with_command<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
     let mut cmd = Command::new("/usr/bin/open");
-    cmd.arg(path.as_ref())
-        .arg("-a")
-        .arg(app.into())
-        .status_without_output()
-        .into_result(&cmd)
+    cmd.arg(path.as_ref()).arg("-a").arg(app.into());
+    cmd
 }

src/unix.rs

@@ -1,13 +1,10 @@
 use std::{
     env,
     ffi::{OsStr, OsString},
-    io,
     path::{Path, PathBuf},
     process::Command,
 };
 
-use crate::{CommandExt, IntoResult};
-
 pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     let path = path.as_ref();
     [
@@ -26,11 +23,10 @@ pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     .collect()
 }
 
-pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
+pub fn with_command<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
     let mut cmd = Command::new(app.into());
-    cmd.arg(path.as_ref())
-        .status_without_output()
-        .into_result(&cmd)
+    cmd.arg(path.as_ref());
+    cmd
 }
 
 // Polyfill to workaround absolute path bug in wslu(wslview). In versions before

src/windows.rs

@@ -1,18 +1,13 @@
-use std::{ffi::OsStr, io, process::Command};
-
-use crate::{CommandExt, IntoResult};
+use std::{ffi::OsStr, process::Command};
 
 pub fn commands<T: AsRef<OsStr>>(path: T) -> Vec<Command> {
     let mut cmd = Command::new("cmd");
     cmd.arg("/c").arg("start").arg(path.as_ref());
     vec![cmd]
 }
 
-pub fn with<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> io::Result<()> {
+pub fn with_command<T: AsRef<OsStr>>(path: T, app: impl Into<String>) -> Command {
     let mut cmd = Command::new("cmd");
-    cmd.arg("/c")
-        .arg(app.into())
-        .arg(path.as_ref())
-        .status_without_output()
-        .into_result(&cmd)
+    cmd.arg("/c").arg(app.into()).arg(path.as_ref());
+    cmd
 }