changeset 812:6adcd8b4df43

minor
author eugene.petrenko@jetbrains.com
date Fri, 30 May 2014 10:47:13 +0200
parents 0f5aeafc3845
children 1dadfa948844
files mercurial-common/src/jetbrains/buildServer/buildTriggers/vcs/mercurial/command/ArchiveCommand.java
diffstat 1 files changed, 8 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial-common/src/jetbrains/buildServer/buildTriggers/vcs/mercurial/command/ArchiveCommand.java	Fri May 30 10:46:44 2014 +0200
+++ b/mercurial-common/src/jetbrains/buildServer/buildTriggers/vcs/mercurial/command/ArchiveCommand.java	Fri May 30 10:47:13 2014 +0200
@@ -30,6 +30,7 @@
 package jetbrains.buildServer.buildTriggers.vcs.mercurial.command;
 
 import com.intellij.execution.configurations.GeneralCommandLine;
+import jetbrains.buildServer.buildTriggers.vcs.mercurial.OS;
 import jetbrains.buildServer.util.FileUtil;
 import jetbrains.buildServer.vcs.VcsException;
 import org.jetbrains.annotations.NotNull;
@@ -39,8 +40,6 @@
 import java.util.List;
 
 public class ArchiveCommand extends AuthCommand {
-  private final static int MAX_CMD_LEN = 900;
-
   private File myDestination;
   private String myToId;
   private String myType = "files";
@@ -68,11 +67,14 @@
     return this;
   }
 
-  public boolean addIncludeRule(@NotNull String rule) {
-    MercurialCommandLine cmd = createCmd();
-    int cmdSize = cmd.getCommandLineString().length();
-    if (cmdSize + rule.length() + (myIncludeRules.isEmpty() ? 0 : "-I ".length()) > MAX_CMD_LEN)
+  public boolean addIncludeRule(@NotNull final String rule) {
+    final MercurialCommandLine cmd = createCmd();
+
+    final int cmdSize = cmd.getCommandLineString().length();
+    if (cmdSize + rule.length() + (myIncludeRules.isEmpty() ? 0 : "-I ".length()) > OS.getMaxCommandLineSize()) {
       return false;
+    }
+
     myIncludeRules.add(rule);
     return true;
   }