IDE: further migrated external tools

git-svn-id: trunk@42241 -

components/ideintf/ideexterntoolintf.pas

@@ -326,6 +326,7 @@ type
     property ExitStatus: integer read FExitStatus write FExitStatus;
     property MinUrgency: TMessageLineUrgency read FMinUrgency write FMinUrgency default DefaultETViewMinUrgency; // hide messages below this
     property MessageLineClass: TMessageLineClass read FMessageLineClass;
+    function HasFinished: boolean; virtual; // not running, no pending messages
   public
     // needs critical section
     property PendingLines: TMessageLines read FPendingLines write FPendingLines;
@@ -353,8 +354,9 @@ type
                                           FirstNewMsgLine: integer) of object;
 
   TExternalToolHandler = (
+    ethNewOutput,
     ethStopped,
-    ethNewOutput
+    ethAllViewsUpdated
     );
 
   TIDEExternalTools = class;
@@ -367,6 +369,7 @@ type
   TAbstractExternalTool = class(TComponent)
   private
     FData: TObject;
+    FEnvironmentOverrides: TStrings;
     FEstimatedLoad: int64;
     FExitStatus: integer;
     FFreeData: boolean;
@@ -377,9 +380,12 @@ type
     FTitle: string;
     FTools: TIDEExternalTools;
     FViews: TFPList; // list of TExtToolView
+    function GetCmdLineParams: string;
     function GetParserCount: integer;
     function GetParsers(Index: integer): TExtToolParser;
     function GetViews(Index: integer): TExtToolView;
+    procedure SetCmdLineParams(aParams: string);
+    procedure SetEnvironmentOverrides(AValue: TStrings);
     procedure SetGroup(AValue: TExternalToolGroup);
     procedure SetTitle(const AValue: string);
     procedure AddHandler(HandlerType: TExternalToolHandler;
@@ -417,18 +423,25 @@ type
 
     // handlers
     procedure RemoveAllHandlersOfObject(AnObject: TObject);
-    procedure AddHandlerOnStopped(const OnStopped: TNotifyEvent;
-                                  AsFirst: boolean = true);  // called in main thread
-    procedure RemoveHandlerOnStopped(const OnStopped: TNotifyEvent);
     procedure AddHandlerOnNewOutput(const OnNewOutput: TExternalToolNewOutputEvent;
                                     AsFirst: boolean = true); // called in main thread
     procedure RemoveHandlerOnNewOutput(const OnNewOutput: TExternalToolNewOutputEvent);
+    procedure AddHandlerOnStopped(const OnStopped: TNotifyEvent;
+                                  AsFirst: boolean = true);  // called in main thread
+    procedure RemoveHandlerOnStopped(const OnStopped: TNotifyEvent);
+    procedure AddHandlerOnAllViewsUpdated(const OnViewsUpdated: TNotifyEvent;
+                                  AsFirst: boolean = true);  // called in main thread
+    procedure RemoveHandlerOnAllViewsUpdated(const OnViewsUpdated: TNotifyEvent);
 
     // process
     property Process: TProcessUTF8 read FProcess;
+    property EnvironmentOverrides: TStrings read FEnvironmentOverrides
+      write SetEnvironmentOverrides; // if not empty, then this and IDE's environemnt will be merged and replace Process.Environment
+    property CmdLineParams: string read GetCmdLineParams write SetCmdLineParams;
     property Stage: TExternalToolStage read FStage;
     procedure Execute; virtual; abstract;
     procedure Terminate; virtual; abstract;
+    procedure WaitForExit; virtual; abstract;
     property Terminated: boolean read FTerminated;
     property ExitStatus: integer read FExitStatus write FExitStatus;
     property ErrorMessage: string read FErrorMessage write FErrorMessage; // error executing tool
@@ -452,11 +465,12 @@ type
     // viewers
     function ViewCount: integer;
     property Views[Index: integer]: TExtToolView read GetViews;
-    function AddView(View: TExtToolView): integer; // will *not* be freed on destroy
+    function AddView(View: TExtToolView): integer; // (main thread) will *not* be freed on destroy
-    procedure DeleteView(View: TExtToolView); // disconnect and free
+    procedure DeleteView(View: TExtToolView); // (main thread) disconnect and free
-    procedure RemoveView(View: TExtToolView); // disconnect without free
+    procedure RemoveView(View: TExtToolView); // (main thread) disconnect without free
-    function IndexOfView(View: TExtToolView): integer; // needs Enter/LeaveCriticalSection
+    function IndexOfView(View: TExtToolView): integer;
-    procedure ClearViews(Delete: boolean = false);
+    procedure ClearViews(Delete: boolean = false); // (main thread)
+    function FindUnfinishedView: TExtToolView;
 
     // dependencies
     procedure AddExecuteBefore(Tool: TAbstractExternalTool); virtual; abstract;
@@ -724,6 +738,11 @@ begin
   Result:=FParsers.Count;
 end;
 
+function TAbstractExternalTool.GetCmdLineParams: string;
+begin
+
+end;
+
 function TAbstractExternalTool.GetParsers(Index: integer): TExtToolParser;
 begin
   Result:=TExtToolParser(FParsers[Index]);
@@ -734,6 +753,26 @@ begin
   Result:=TExtToolView(FViews[Index]);
 end;
 
+procedure TAbstractExternalTool.SetCmdLineParams(aParams: string);
+var
+  sl: TStringList;
+begin
+  sl:=TStringList.Create;
+  try
+    SplitCmdLineParams(aParams,sl,PathDelim<>'\');
+    Process.Parameters:=sl;
+  finally
+    sl.Free;
+  end;
+end;
+
+procedure TAbstractExternalTool.SetEnvironmentOverrides(AValue: TStrings);
+begin
+  if (FEnvironmentOverrides=AValue)
+  or (FEnvironmentOverrides.Equals(AValue)) then Exit;
+  FEnvironmentOverrides.Assign(AValue);
+end;
+
 procedure TAbstractExternalTool.SetGroup(AValue: TExternalToolGroup);
 begin
   if FGroup=AValue then Exit;
@@ -797,6 +836,7 @@ begin
   FViews:=TFPList.Create;
   FStage:=etsInit;
   FEstimatedLoad:=1;
+  FEnvironmentOverrides:=TStringList.Create;
 end;
 
 destructor TAbstractExternalTool.Destroy;
@@ -814,6 +854,7 @@ begin
     FWorkerMessages.Clear;
     FreeAndNil(FParsers);
     FreeAndNil(FViews);
+    FreeAndNil(FEnvironmentOverrides);
     inherited Destroy;
   finally
     LeaveCriticalsection;
@@ -864,6 +905,18 @@ begin
   RemoveHandler(ethStopped,TMethod(OnStopped));
 end;
 
+procedure TAbstractExternalTool.AddHandlerOnAllViewsUpdated(
+  const OnViewsUpdated: TNotifyEvent; AsFirst: boolean);
+begin
+  AddHandler(ethAllViewsUpdated,TMethod(OnViewsUpdated),AsFirst);
+end;
+
+procedure TAbstractExternalTool.RemoveHandlerOnAllViewsUpdated(
+  const OnViewsUpdated: TNotifyEvent);
+begin
+  RemoveHandler(ethAllViewsUpdated,TMethod(OnViewsUpdated));
+end;
+
 procedure TAbstractExternalTool.AddHandlerOnNewOutput(
   const OnNewOutput: TExternalToolNewOutputEvent; AsFirst: boolean);
 begin
@@ -993,6 +1046,17 @@ begin
   end;
 end;
 
+function TAbstractExternalTool.FindUnfinishedView: TExtToolView;
+var
+  i: Integer;
+begin
+  for i:=0 to ViewCount-1 do begin
+    Result:=Views[i];
+    if not Result.HasFinished then exit;
+  end;
+  Result:=nil;
+end;
+
 { TExtToolParser }
 
 destructor TExtToolParser.Destroy;
@@ -1891,6 +1955,19 @@ begin
   end;
 end;
 
+function TExtToolView.HasFinished: boolean;
+begin
+  Result:=false;
+  EnterCriticalSection;
+  try
+    if Running then exit;
+    if PendingLines.Count>0 then exit;
+    Result:=true;
+  finally
+    LeaveCriticalSection;
+  end;
+end;
+
 end.
 {$ELSE}
 interface

components/lazutils/lazfileutils.pas

@@ -120,6 +120,7 @@ function GetDarwinSystemFilename(Filename: string): string;
 
 procedure SplitCmdLineParams(const Params: string; ParamList: TStrings;
                              ReadBackslash: boolean = false);
+function MergeCmdLineParams(ParamList: TStrings): string;
 
 type
   TInvalidateFileStateCacheEvent = procedure(const Filename: string);
@@ -1077,6 +1078,61 @@ begin
   end;
 end;
 
+function MergeCmdLineParams(ParamList: TStrings): string;
+var
+  i: Integer;
+  HasSpace: Boolean;
+  HasApos: Boolean;
+  HasQuot: Boolean;
+  Param: String;
+  j: Integer;
+begin
+  Result:='';
+  if ParamList=nil then exit;
+  for i:=0 to ParamList.Count-1 do
+  begin
+    Param:=ParamList[i];
+    if Param='' then
+      Param:='""'
+    else begin
+
+    end;
+
+    HasSpace:=false;
+    HasApos:=false;
+    HasQuot:=false;
+    for j:=1 to length(Param) do begin
+      case Param[i] of
+      ' ',#9,#10,#13: HasSpace:=true;
+      '''': HasApos:=true;
+      '"': HasQuot:=true;
+      end;
+    end;
+    if HasQuot then begin
+      if HasApos then begin
+
+      end else begin
+        // quot, no apos
+
+      end;
+    end else begin
+      // no quot
+      if HasApos then begin
+        // apos, no quot
+
+      end else begin
+        if HasSpace or (Param='') then begin
+          // no quot, no apos, has space -> quote
+          Param:=''''+Param+'''';
+        end else begin
+          // simple parameter => pass unaltered
+        end;
+      end;
+    end;
+    if i>0 then Result+=' ';
+    Result+=Param;
+  end;
+end;
 
 {
   Returns

ide/compileroptions.pp

@@ -4171,12 +4171,13 @@ function TCompilationToolOptions.Execute(const WorkingDir, ToolTitle: string
   ): TModalResult;
 var
   ProgramFilename, Params: string;
+  Filename: String;
+  CurCommand: String;
   {$IFDEF EnableNewExtTools}
+  Tool: TAbstractExternalTool;
   {$ELSE}
   ExtTool: TIDEExternalToolOptions;
   {$ENDIF}
-  Filename: String;
-  CurCommand: String;
 begin
   CurCommand:=GetParsedCommand;
   if CurCommand='' then begin
@@ -4193,6 +4194,12 @@ begin
     if Filename<>'' then ProgramFilename:=Filename;
   end;
 
+  {$IFDEF EnableNewExtTools}
+  Tool:=ExternalToolList.Add(ToolTitle);
+  Tool.Process.Executable:=ProgramFilename;
+  Tool.Process.CurrentDirectory:=WorkingDir;
+  Tool.Process.Parameters;
+  {$ELSE}
   ExtTool:=TIDEExternalToolOptions.Create;
   try
     ExtTool.Filename:=ProgramFilename;
@@ -4210,6 +4217,7 @@ begin
     // clean up
     ExtTool.Free;
   end;
+  {$ENDIF}
 end;
 
 procedure TCompilationToolOptions.IncreaseChangeStamp;