On 02/05/2011, at 8:19 AM, lcornelius...@apache.org wrote:
> Modified:
> incubator/npanday/trunk/components/dotnet-vendor/src/main/java/npanday/vendor/SettingsUtil.java
> URL:
> http://svn.apache.org/viewvc/incubator/npanday/trunk/components/dotnet-vendor/src/main/java/npanday/vendor/SettingsUtil.java?rev=1098424&r1=1098423&r2=1098424&view=diff
> ==============================================================================
> ---
> incubator/npanday/trunk/components/dotnet-vendor/src/main/java/npanday/vendor/SettingsUtil.java
> (original)
> +++
> incubator/npanday/trunk/components/dotnet-vendor/src/main/java/npanday/vendor/SettingsUtil.java
> Sun May 1 22:19:50 2011
> @@ -35,7 +35,7 @@ import java.util.Hashtable;
> public class SettingsUtil
> {
> /**
> - * Return the registered settings, or create from default settings file
> location (.m2/npanday-settings.xml)
> + * Return the registered settings, or create from configured
> (-Dnpanday-settings=...) or default settings file location
> (.m2/npanday-settings.xml)
> * @param repositoryRegistry The registry.
> * @return The current, or just created SettingsRepository
> * @throws SettingsException If anything goes wrong reading or
> registering the settings
> @@ -43,7 +43,13 @@ public class SettingsUtil
> public static SettingsRepository getOrPopulateSettingsRepository(
> RepositoryRegistry repositoryRegistry)
> throws SettingsException
> {
> - return getOrPopulateSettingsRepository(repositoryRegistry,
> PathUtil.getHomeM2Folder());
> + String settingsFolder = PathUtil.getHomeM2Folder();
> + String customFolder = System.getProperty( "npanday.settings" );
> + if (customFolder != null && customFolder != "")
This won't necessarily work, as string comparisons aren't native. You want
!"".equals(customFolder), or use something like !StringUtils.isEmpty for the
whole line.
> + {
> + settingsFolder = customFolder;
> + }
> + return getOrPopulateSettingsRepository(repositoryRegistry,
> settingsFolder );
> }
>
> /**
>
>
> Modified:
> incubator/npanday/trunk/plugins/netplugins/NPanday.Plugin.Settings/src/main/csharp/NPanday/Plugin/Settings/PathUtil.cs
> URL:
> http://svn.apache.org/viewvc/incubator/npanday/trunk/plugins/netplugins/NPanday.Plugin.Settings/src/main/csharp/NPanday/Plugin/Settings/PathUtil.cs?rev=1098424&r1=1098423&r2=1098424&view=diff
> ==============================================================================
> ---
> incubator/npanday/trunk/plugins/netplugins/NPanday.Plugin.Settings/src/main/csharp/NPanday/Plugin/Settings/PathUtil.cs
> (original)
> +++
> incubator/npanday/trunk/plugins/netplugins/NPanday.Plugin.Settings/src/main/csharp/NPanday/Plugin/Settings/PathUtil.cs
> Sun May 1 22:19:50 2011
> @@ -17,12 +17,29 @@
> // under the License.
> //
>
> +using System;
> using System.IO;
>
> namespace NPanday.Plugin.Settings
> {
> + // TODO: Move this to common PathUtil, when NPANDAY-422 is fixed
> public static class PathUtil
> {
> + public static string GetHomeM2Folder()
> + {
> + return Environment.GetEnvironmentVariable("USERPROFILE") +
> "/.m2";
> + }
> +
> + public static string BuildSettingsFilePath( string
> settingsPathOrFile )
> + {
> + if (settingsPathOrFile.EndsWith( "xml" ))
> + {
> + return settingsPathOrFile;
> + }
Same comment as before - this would be more robust if there was a way to check
isDirectory or isFile instead.
- Brett
--
Brett Porter
br...@apache.org
http://brettporter.wordpress.com/
http://au.linkedin.com/in/brettporter
