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

Add CROSS and ROOTFS_DIR environment for ARM#766

Merged
gkhanna79 merged 1 commit into
dotnet:masterfrom
jyoungyun:arm/cross-build
Dec 14, 2016
Merged

Add CROSS and ROOTFS_DIR environment for ARM#766
gkhanna79 merged 1 commit into
dotnet:masterfrom
jyoungyun:arm/cross-build

Conversation

@jyoungyun
Copy link
Copy Markdown

@jyoungyun jyoungyun commented Dec 12, 2016

This PR should be merged after previous PR #747 & #749

The CROSS env is passed through the BuildContext to the build.sh in corehost.
If the value is set, corehost build script gives '--cross' arguments.
ROOTFS_DIR can support to specify the location of rootfs as CoreCLR does.
If ROOTFS_DIR env is not defined,
the default value(./cross/rootfs/${TARGETPLATFORM}/) will be used.

You can build corehost for ARM using following commands.
./build.sh --env-vars TARGETPLATFORM=arm,TARGETRID=ubuntu.16.04-arm,CROSS=1
./build.sh --env-vars TARGETPLATFORM=arm,TARGETRID=ubuntu.16.04-arm,CROSS=1,ROOTFS_DIR=/home/jyoung/git/dotnet/rootfs-coreclr/arm/

Related issue: #729

I posted this PR to share with other developers.
@hqueue

The CROSS env is passed through the BuildContext to the build.sh in corehost.
If the value is set, corehost build script gives '--cross' argument.
ROOTFS_DIR can support to specify the location of rootfs as CoreCLR does.
If ROOTFS_DIR env is not defined,
the default value(./cross/rootfs/${TARGETPLATFORM}/) will be used.

Signed-off-by: Jiyoung Yun <jy910.yun@samsung.com>
@dnfclas
Copy link
Copy Markdown

dnfclas commented Dec 12, 2016

Hi @jyoungyun, 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;

@jyoungyun jyoungyun changed the title [WIP] Add CROSS and ROOTFS_DIR environment for ARM Add CROSS and ROOTFS_DIR environment for ARM Dec 12, 2016
@gkhanna79
Copy link
Copy Markdown
Member

LGTM

@gkhanna79
Copy link
Copy Markdown
Member

@jyoungyun I have merged the other PRs - is this good to merge?

@jyoungyun
Copy link
Copy Markdown
Author

@gkhanna79 Yes, Please merge this patch. Thank you.

@gkhanna79 gkhanna79 merged commit 0ae4248 into dotnet:master Dec 14, 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.

3 participants