Skip to content

Conversation

@Trenly
Copy link
Contributor

@Trenly Trenly commented Oct 14, 2025


Microsoft Reviewers: Open in CodeFlow

@Trenly Trenly changed the title Enable multi-core compilation Enable MultiProcessorCompilation Oct 14, 2025
@JohnMcPMS
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

florelis
florelis previously approved these changes Oct 15, 2025
@florelis
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Trenly
Copy link
Contributor Author

Trenly commented Oct 15, 2025

@florelis - Can I request a re-run? I'm not seeing what would have caused the PowerShell error

@JohnMcPMS
Copy link
Member

@Trenly these seem to have started when we release 1.11.510 and are spurious. #5807 to start trying to get to the bottom of it.

@Trenly
Copy link
Contributor Author

Trenly commented Oct 15, 2025

@JohnMcPMS - I noticed on Build #196599 there was an ENOSPC - Given that the test seemed to fail on installing PSTools, I'm curious if that step is also occasionally hitting ENOSPC thereby causing the install to fail and thus the test stage to fail, especially since ENOSPC doesn't affect all PRs

See also -

Edit: Created #5808 for tracking

@JohnMcPMS
Copy link
Member

This happens so early in the job that I would be surprised if space were an issue. But regardless it would be good to add collecting the release winget logs in case they also have details on the reason for the error.

@JohnMcPMS
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Trenly
Copy link
Contributor Author

Trenly commented Oct 16, 2025

That failure doesn't make sense to me. . .

I understand it's a linker error, but I don't see how that error would be appearing from this change - unless the dependencies aren't being built in the right order? But then why would it work locally and not here?

@JohnMcPMS
Copy link
Member

That failure doesn't make sense to me. . .

I understand it's a linker error, but I don't see how that error would be appearing from this change - unless the dependencies aren't being built in the right order? But then why would it work locally and not here?

Were you clean building locally? I would expect that is the only way to test for sure.

This warning is suspicious, because all of the unresolved methods are in that file:

x64\Release\typeresolution.obj : warning LNK4042: object specified more than once; extras ignored

It makes me think that the multiprocessor build is processing this file twice and one of them is not so good.

@JohnMcPMS
Copy link
Member

binlog for the (probable) reason, if not why it doesn't fail without:

typeresolution.h is being added to the ClCompile group because it is "Item Type" C/C++ compiler.

In External - Do Not Modify > UndockedRegFreeWinRT > Header Files > typeresolution.h > Properties > General > Item Type > should be C/C++ header.

@Trenly
Copy link
Contributor Author

Trenly commented Oct 16, 2025

That failure doesn't make sense to me. . .
I understand it's a linker error, but I don't see how that error would be appearing from this change - unless the dependencies aren't being built in the right order? But then why would it work locally and not here?

Were you clean building locally? I would expect that is the only way to test for sure.

This warning is suspicious, because all of the unresolved methods are in that file:

x64\Release\typeresolution.obj : warning LNK4042: object specified more than once; extras ignored

It makes me think that the multiprocessor build is processing this file twice and one of them is not so good.

Yes, I made sure to clean the solution first.

@Trenly
Copy link
Contributor Author

Trenly commented Oct 17, 2025

binlog for the (probable) reason, if not why it doesn't fail without:

typeresolution.h is being added to the ClCompile group because it is "Item Type" C/C++ compiler.

In External - Do Not Modify > UndockedRegFreeWinRT > Header Files > typeresolution.h > Properties > General > Item Type > should be C/C++ header.

I could add a condition to specifically exclude the UndockedRegFreeWinRT project, perhaps?

@JohnMcPMS
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Trenly
Copy link
Contributor Author

Trenly commented Oct 17, 2025

/azp run

Thank you

@JohnMcPMS JohnMcPMS merged commit cf43c6a into microsoft:master Oct 17, 2025
9 checks passed
@Trenly Trenly deleted the MulticoreCompilation branch December 23, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants