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

[net7.0] Updates the path to illink when building from Windows #15667

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

dalexsoto
Copy link
Member

The directory that contains illink.dll is now net7.0

The directory that contains illink.dll is now net7.0
@dalexsoto dalexsoto added this to the .NET 7 milestone Aug 10, 2022
@dalexsoto dalexsoto added the notes-mention Deserves a mention in release notes label Aug 10, 2022
@dalexsoto dalexsoto marked this pull request as ready for review August 10, 2022 18:02
@vs-mobiletools-engineering-service2

This comment has been minimized.

@dalexsoto
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 [PR Build] Build failed 🔥

Build failed for the job 'Detect API changes'

Pipeline on Agent
Hash: [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Unable to find the contents for the comment: D:\a\1\s\change-detection\results\gh-comment.md does not exist :fire

Pipeline on Agent
Hash: 31823eff789d1a0d30a2145824125604c85f2fdc [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed ❌

No test package could be found for tests on macOS M1 - Mac Big Sur (11.5)

Pipeline on Agent
Hash: 31823eff789d1a0d30a2145824125604c85f2fdc [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests on macOS Mac Catalina (10.15) failed ❌

No test package could be found for tests on macOS Mac Catalina (10.15)

Pipeline on Agent
Hash: 31823eff789d1a0d30a2145824125604c85f2fdc [PR build]

@vs-mobiletools-engineering-service2
Copy link
Collaborator

📚 [PR Build] Artifacts 📚

Packages generated

View packages

Pipeline on Agent XAMBOT-1108.Monterey'
Hash: 31823eff789d1a0d30a2145824125604c85f2fdc [PR build]

@dalexsoto
Copy link
Member Author

we have tested this locally on windows and since it makes it work and it is only a change in the windows targets we'll merge it to have a coherent build.

@dalexsoto dalexsoto merged commit 0e53b99 into xamarin:net7.0 Aug 10, 2022
@dalexsoto dalexsoto deleted the net7.0-illink-fix branch August 10, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-mention Deserves a mention in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants