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

Experimental Microsoft.AspNetCore.ClientAssets and multi-target framework builds fails #48270

Open
dp-sgr opened this issue May 17, 2023 · 2 comments
Labels
feature-blazor-client-assets Priority:2 Work that is important, but not critical for the release
Milestone

Comments

@dp-sgr
Copy link

dp-sgr commented May 17, 2023

At first:

I really love the Microsoft.AspNetCore.ClientAssets package. Its wonderful to automate everything that has to do with static web assets.
As stateted in #38445 we may get a implementation direkty into the SDK. This would also be wonderful.

The current Issue:
As we started using .NET7 we needed to implement some Workarounds. So our RCL now targets net6 & net7.
Since then we get Build Errors from time to time (Sporadically).
It seems that the Mutli-Target build runs npm install two times. If this would not be parallel everything would be ok for me. But it seems that the install will be invoked in parallel (or at least slighty one after another) without waiting the first invocation to be completed. And this leads to the current error.

For now i adjusted my build process and disabled parallel building with BuildInParallel set to false:
https://learn.microsoft.com/en-us/visualstudio/msbuild/building-multiple-projects-in-parallel-with-msbuild?view=vs-2022

So i got an Workaround up-and running fine for me.
This Issue is just an Info for you to also pay attention to in the new integration.

Thanks for your good work :)
I'm hyped to get my hands on the SDK integrated Version

@mkArtakMSFT mkArtakMSFT added area-blazor Includes: Blazor, Razor Components feature-blazor-client-assets labels Nov 19, 2023
@javiercn javiercn self-assigned this Nov 30, 2023
@javiercn javiercn added this to the .NET 9 Planning milestone Nov 30, 2023
@ghost
Copy link

ghost commented Nov 30, 2023

Thanks for contacting us.

We're moving this issue to the .NET 9 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@mkArtakMSFT mkArtakMSFT added the Priority:2 Work that is important, but not critical for the release label Jan 3, 2024
@ghost
Copy link

ghost commented Jan 3, 2024

Thanks for contacting us.

We're moving this issue to the .NET 9 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@javiercn javiercn removed the area-blazor Includes: Blazor, Razor Components label Jan 21, 2025
@javiercn javiercn modified the milestones: .NET 10 Planning, Backlog Jan 21, 2025
@javiercn javiercn removed their assignment Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-blazor-client-assets Priority:2 Work that is important, but not critical for the release
Projects
None yet
Development

No branches or pull requests

3 participants