Skip to content
New issue

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

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

Already on GitHub? # to your account

Possibly fixed intermittent build issues #5421

Merged
merged 1 commit into from
Dec 5, 2022

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Dec 5, 2022

I was consistently experiencing locally the build issues we sometimes (well oftentimes) get on azure (can't resolved Dnn.Platform\Dnn.AdminExperience\Dnn.PersonaBar.Extensions\Dnn.PersonaBar.Extensions.csproj)...

So I was trying to pinpoint the possible cause of that issues and when I removed the Project GUIDS from this project file, the build started working. Many of the other .csproj files don't have any ProjectType defined at all and they work fine.

I have confirmed that the install files are still being produced.

One of the GUIDs defines this as an MVC project but it is clearly not, the other defines it as a C# project but the .csproj extension already defines that.

Is anyone is opposed to remove both, then I can bring back the one starting with FAE04....

I am absolutelly not sure this will fix the intermittent build issues, because of them beeing intermittent but if it works, we may have fixed it...

I was consistently experiencing locally the build issues we sometimes (well oftentimes) get on azure (can't resolved Dnn.Platform\Dnn.AdminExperience\Dnn.PersonaBar.Extensions\Dnn.PersonaBar.Extensions.csproj)...

So I was trying to pinpoint the possible cause of that issues and when I removed the Project GUIDS from this project file, the build started working. Many of the other .csproj files don't have any ProjectType defined at all and they work fine.

I have confirmed that the install files are still being produced.

One of the GUIDs defines this as an MVC project but it is clearly not, the other defines it as a C# project but the .csproj extension already defines that.

Is anyone is opposed to remove both, then I can bring back the one starting with FAE04....

I am absolutelly not sure this will fix the intermittent build issues, because of them beeing intermittent but if it works, we may have fixed it...
@valadas valadas added this to the 9.11.1 milestone Dec 5, 2022
@valadas valadas requested a review from bdukes December 5, 2022 00:58
Copy link
Contributor

@mitchelsellers mitchelsellers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build runs for me, worth a shot!

Copy link
Contributor

@bdukes bdukes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

worth a try

@bdukes bdukes merged commit dff105c into dnnsoftware:develop Dec 5, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants