From f3771379bce9e64d985f0c1e73dbc6aae48b6d5b Mon Sep 17 00:00:00 2001 From: Michael Osipov Date: Sat, 15 Oct 2022 20:46:09 +0200 Subject: [PATCH] Don't throw IOException when none is required (#206) This fixes #206 and closes #221 --- .../java/org/codehaus/plexus/util/cli/CommandLineUtils.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/main/java/org/codehaus/plexus/util/cli/CommandLineUtils.java b/src/main/java/org/codehaus/plexus/util/cli/CommandLineUtils.java index 4a556c84..0e168c04 100644 --- a/src/main/java/org/codehaus/plexus/util/cli/CommandLineUtils.java +++ b/src/main/java/org/codehaus/plexus/util/cli/CommandLineUtils.java @@ -110,7 +110,7 @@ public static int executeCommandLine( Commandline cl, InputStream systemIn, Stre /** * Immediately forks a process, returns a callable that will block until process is complete. - * + * * @param cl The command line to execute * @param systemIn The input to read from, must be thread safe * @param systemOut A consumer that receives output, must be thread safe @@ -354,12 +354,10 @@ private static void handleException( final StreamFeeder streamFeeder, final Stri * with case-insensitive environment variables like Windows, all variable names will be normalized to upper case. * * @return The shell environment variables, can be empty but never null. - * @throws IOException If the environment variables could not be queried from the shell. * @see System#getenv() System.getenv() API, new in JDK 5.0, to get the same result since 2.0.2 System#getenv() * will be used if available in the current running jvm. */ public static Properties getSystemEnvVars() - throws IOException { return getSystemEnvVars( !Os.isFamily( Os.FAMILY_WINDOWS ) ); } @@ -370,12 +368,10 @@ public static Properties getSystemEnvVars() * * @param caseSensitive Whether environment variable keys should be treated case-sensitively. * @return Properties object of (possibly modified) envar keys mapped to their values. - * @throws IOException . * @see System#getenv() System.getenv() API, new in JDK 5.0, to get the same result since 2.0.2 System#getenv() * will be used if available in the current running jvm. */ public static Properties getSystemEnvVars( boolean caseSensitive ) - throws IOException { Properties envVars = new Properties(); Map envs = System.getenv();