-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Delete unused Crossgen2PackageRID property #66814
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing!
4e989f1
to
607d570
Compare
607d570
to
34cba85
Compare
Remaining CI failures seem unrelated infra issues. Regarding second commitWhen building natively on FreeBSD, proper crossgen2 binary is produced (i.e. the one with FreeBSD compatible apphost). However, when cross-compiling for FreeBSD from Linux, we had PublishReadyToRun disabled, but we were still producing shipping package with invalid CG2 binary (with linux apphost...). First commit uncovers that issue at package-restore-time and second commit prevents CG2 package generation in FreeBSD cross-compile mode. |
@Thefrank, correct. The apphost package ( The reason why we don't see any error on main branch is because of the wrong
The second commit disables crossgen2 project build on FreeBSD package generation in cross build mode. It was already partially disabled, but not properly excluded since the issue was hidden due to the said bug. When #66866 is resolved, we can enable it back. |
FreeBSD issue is handled by #65948 merge (we do not restore apphost for CG2 on unsupported platforms anymore). I have reverted the second commit and merged Libs test failures are unrelated to this change. |
4d8c509
to
eb1b2e5
Compare
eb1b2e5
to
02b1bc1
Compare
Hi @am11, assume this is ok to merge? |
This was folded into #67636. |
thanks for the update. |
Unused after #65948