Il 20/02/2015 19:18, Mattias Gaertner ha scritto:
On Fri, 20 Feb 2015 18:29:15 +0100
Giuliano Colla <[email protected]> wrote:

Il 20/02/2015 17:30, Mattias Gaertner ha scritto:
On Fri, 20 Feb 2015 17:04:22 +0100
Giuliano Colla <[email protected]> wrote:

Il 20/02/2015 01:32, Mattias Gaertner ha scritto:
[...]
Do you mean line 3237?
That was fixes_1_4.

I rather mean line 3206.
I guess this line is in 1.4RC1. Please refer to fixes_1_4.
No it was in trunk.
trunk:

3205:  if APackage.AutoUpdate=pupManually then
3206:    exit(mrNo);

?

Sorry I was too hasty.

The piece of code was:

  // check the current output directory
  Result:=CheckIfCurPkgOutDirNeedsCompile(APackage,
             true,SkipDesignTimePackages,GroupCompile,
NeedBuildAllFlag,ConfigChanged,DependenciesChanged,Note);
  if Result=mrNo then begin
    // the current output is valid
    exit;
  end;

Which appeared to me to check only the default directory.
But I had not seen that the current output directory is properly set (either default or fallback) in GetOutputDirType. So forget about my observations.

Giuliano

--
Giuliano Colla

Project planning question: when it's 90% done, are we halfway or not yet?


--
_______________________________________________
Lazarus mailing list
[email protected]
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to