Forum > General

Lazbuild concurrency issue

<< < (2/2)

Wallaby:
I have checked the lazbuild's code. Around these lines:


--- Code: Pascal  [+][-]window.onload = function(){var x1 = document.getElementById("main_content_section"); if (x1) { var x = document.getElementsByClassName("geshi");for (var i = 0; i < x.length; i++) { x[i].style.maxHeight='none'; x[i].style.height = Math.min(x[i].clientHeight+15,306)+'px'; x[i].style.resize = "vertical";}};} ---      // update all lrs files      MainBuildBoss.UpdateProjectAutomaticFiles('');       // regenerate resources      if not Project1.ProjResources.Regenerate(SrcFileName, False, True, '') then      begin        if ConsoleVerbosity>=-1 then          DebugLn('Error: (lazarus) Project1.Resources.Regenerate failed of ',SrcFilename);      end;    
This is where multiple lazbuild instances may have concurrency issues by trying to write to the same RES file in the project folder (recompiling resources) from multiple instances. I guess I will create a bug report to see what can be done here.

MarkMLl:

--- Quote from: Wallaby on June 14, 2024, 10:35:42 am ---This is where multiple lazbuild instances may have concurrency issues by trying to write to the same RES file in the project folder (recompiling resources) from multiple instances. I guess I will create a bug report to see what can be done here.

--- End quote ---

We heard you already. Raise a bug report.

MarkMLl

Wallaby:

--- Quote from: MarkMLl on June 14, 2024, 10:40:32 am ---This is where multiple lazbuild instances may have concurrency issues by trying to write to the same RES file in the project folder (recompiling resources) from multiple instances. I guess I will create a bug report to see what can be done here.
We heard you already. Raise a bug report.
MarkMLl

--- End quote ---

Done: #40986 Lazbuild concurrency issue with multiple instances

JuhaManninen:
Yes. Just remember the same code is used also by Lazarus IDE. The solution will affect both.

MarkMLl:

--- Quote from: JuhaManninen on June 14, 2024, 10:54:18 am ---Yes. Just remember the same code is used also by Lazarus IDE. The solution will affect both.

--- End quote ---

Actually, moving the .res into the lib/ tree by default would be worth doing, since "do I need to check the .res into my VCS or will it be rebuilt?" is rather an FAQ.

MarkMLl

Navigation

[0] Message Index

[*] Previous page

Go to full version