Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build Execution Time in UI5 Tooling 3.0 #781

Closed
shaiSAP opened this issue Feb 15, 2023 · 4 comments
Closed

Build Execution Time in UI5 Tooling 3.0 #781

shaiSAP opened this issue Feb 15, 2023 · 4 comments
Labels
help provided Consulting given or question answered

Comments

@shaiSAP
Copy link

shaiSAP commented Feb 15, 2023

After upgrading to UI Tooling 3.0 a build which took a few seconds now takes about 100 seconds.
(It might be a result of an invalid/incomplete configuration).
I'm attaching the verbose log of the build.
build_log.txt

@RandomByte
Copy link
Member

RandomByte commented Feb 15, 2023

Thank you for providing the build log.

For reference, linking your related comment in the SAP Community.

From the build log, I can see that you are building an application project with a custom task called ui5-task-nwabap-deployer. Opposing to UI5 Tooling v2, UI5 Tooling v3 needs to build all of your projects dependencies because the custom task might require access to some of them. This new behavior is also explained here.

From what I can see, the custom task ui5-task-nwabap-deployer doesn't actually require access to dependencies. I will therefore go ahead and recommend the author of that task to implement the recommended callback to signal this to UI5 Tooling as outlined here. Alternatively, the task could also switch to specifying specVersion: 3.0.

This will probably affect other custom tasks too. However, in your case it seems that only the mentioned task is being used in your project.

@RandomByte RandomByte added the help provided Consulting given or question answered label Feb 15, 2023
@shaiSAP
Copy link
Author

shaiSAP commented Feb 15, 2023

Thanks for the quick response and the detailed info.
If this is the case with any custom task, there should be a warning regarding it.
Moreover, I wonder wouldn't it be easier to change the default behaviour: Build all the project dependencies only if a custom task explicitly requires access to them.

@RandomByte
Copy link
Member

Thanks for creating the issue!

Moreover, I wonder wouldn't it be easier to change the default behaviour: Build all the project dependencies only if a custom task explicitly requires access to them.

Yes, you are right. Sadly there is no way to tell for UI5 Tooling whether an existing custom task requires access to dependencies. This is only possible with the newly introduced determineRequiredDependencies callback.

Therefore, we have to assume that all existing custom tasks might require access to dependencies.

@RandomByte
Copy link
Member

The ui5-task-nwabap-deployer custom task has been updated as per pfefferf/ui5-nwabap-deployer#60

Updating the dependency should resolve the performance issue you are facing.

@RandomByte RandomByte changed the title Build Execution Time in UI Tooling 3.0 Build Execution Time in UI5 Tooling 3.0 Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help provided Consulting given or question answered
Projects
None yet
Development

No branches or pull requests

2 participants