Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Enable cross-compilation of corehost for ARM#749

Merged
gkhanna79 merged 2 commits into
dotnet:masterfrom
hqueue:arm/cross-rootfs
Dec 13, 2016
Merged

Enable cross-compilation of corehost for ARM#749
gkhanna79 merged 2 commits into
dotnet:masterfrom
hqueue:arm/cross-rootfs

Conversation

@hqueue
Copy link
Copy Markdown
Member

@hqueue hqueue commented Dec 9, 2016

This PR should be merged after previous PR #747 .

Related issue: #723

Add a toolchain.cmake for ARM
Add a new option "--cross" to enable cross-compilation using rootfs.
Use ROOTFS_DIR environmental variable to specify location of rootfs as CoreCLR does.

I posted this PR to share with other developers.
@jyoungyun @idkiller

@dnfclas
Copy link
Copy Markdown

dnfclas commented Dec 9, 2016

Hi @hqueue, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@hqueue
Copy link
Copy Markdown
Member Author

hqueue commented Dec 9, 2016

CC @schellap @ramarag @janvorli @gkhanna79 @kouvel PTAL

You can build corehost for ARM using following command with two PRs for example.

$ ROOTFS_DIR=<core-setup>/cross/rootfs/arm <core-setup>/src/corehost/build.sh "--arch" "arm" "--hostver" "1.1.0" "--fxrver" "1.1.0" "--policyver" "1.1.0" "--rid" "ubuntu.14.04-arm" "--commithash" "f46c58c1404fa8aaac8aed931b0862267a98c3a5" --cross

Related issue: #729

Add a toolchain.cmake for ARM

Add a new option "--cross" to enable cross-compilation using rootfs.
Use ROOTFS_DIR environmental variable to specify location of rootfs
as CoreCLR does.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
Copy link
Copy Markdown
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Copy link
Copy Markdown
Member

@gkhanna79 gkhanna79 left a comment

Choose a reason for hiding this comment

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

LGTM

Use variable to select rootfs for cross build.

Signed-off-by: Hyung-Kyu Choi <hk0110.choi@samsung.com>
@schellap
Copy link
Copy Markdown

LGTM!

@hqueue hqueue changed the title [WIP] Enable cross-compilation of corehost for ARM Enable cross-compilation of corehost for ARM Dec 13, 2016
@hqueue
Copy link
Copy Markdown
Member Author

hqueue commented Dec 13, 2016

It works with PR #747. Please merge it after #747.

@gkhanna79 gkhanna79 merged commit 786a3f4 into dotnet:master Dec 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants