Add documentation steps for building corefx for a new architecture#16384
Conversation
| --> Output goes to bin/runtime/netcoreapp-Linux-Debug-x64 | ||
| ``` | ||
|
|
||
| The reason you need to build the managed portion for x64 is to because it depends on runtime packages for the new architecture which don't exist yet so we use another existing architecture such as x64 as a proxy for building the managed binaries. Once all the necessary builds are setup and packages are published this spliting of the build should no longer be needed. |
There was a problem hiding this comment.
@weshaggard I would like to add the comment below.
When you want to run CoreFX unittest or use netcoreapp-Linux-Debug-armel directory, you need to copy managed dll from netcoreapp-Linux-Debug-x64 and CoreCLR results. Currently there is no dotnet and libhost*.so for armel architecture. So before you run unittest, you need to change 'RunTests.sh' script to use corerun instead of dotnet.
There was a problem hiding this comment.
@weshaggard FYI. You can find related workaround using netcoreapp-Linux-Debug-x64 in PR for armel CI #16378 (link to the code: https://github.com/dotnet/corefx/pull/16378/files#diff-9b74089ede263293591a48bb7430a9a4R259)
There was a problem hiding this comment.
Updated docs to add information about testing.
| --> Output goes to bin/runtime/netcoreapp-Linux-Debug-x64 | ||
| ``` | ||
|
|
||
| The reason you need to build the managed portion for x64 is to because it depends on runtime packages for the new architecture which don't exist yet so we use another existing architecture such as x64 as a proxy for building the managed binaries. Once all the necessary builds are setup and packages are published this spliting of the build should no longer be needed. |
There was a problem hiding this comment.
Nit: "is to because" => "is because"
Also removes workaround added for in dotnet#16139.
weshaggard
force-pushed
the
AddDocsForBuildingNewArch
branch
from
a527704 to
a0326fb
Compare
…dingNewArch Add documentation steps for building corefx for a new architecture Commit migrated from dotnet/corefx@76b1479
7 participants
Also removes workaround added for in #16139.
Fixes https://github.com/dotnet/corefx/issues/15503
cc @gkhanna79 @hseok-oh @jyoungyun @hqueue