The Wayback Machine - https://web.archive.org/web/20201212153819/https://github.com/dotnet/corefx/pull/15528
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

SkipUnmount support and set default clang for cross build #15528

Merged
merged 1 commit into from Jan 27, 2017

Conversation

@gkhanna79
Copy link
Member

@gkhanna79 gkhanna79 commented Jan 26, 2017

Similar to CoreCLR repo, add support to skip unmounting when using build-rootfs.sh in Docker image.

Also, define the default clang toolset to use for cross and regular builds (similar to how it is done in CoreCLR at https://github.com/dotnet/coreclr/blob/master/build.sh#L747).

@janvorli @wtgodbe PTAL

@gkhanna79
Copy link
Member Author

@gkhanna79 gkhanna79 commented Jan 26, 2017

Confirmed that, with this change, we successfully can build CoreFX for Ubuntu 14.04 and 16.04 arm32.

@gkhanna79
Copy link
Member Author

@gkhanna79 gkhanna79 commented Jan 26, 2017

@@ -150,8 +150,8 @@ __UnprocessedBuildArgs=
__CrossBuild=0
__ServerGC=0
__VerboseBuild=false
__ClangMajorVersion=3
__ClangMinorVersion=5
__ClangMajorVersion=0

This comment has been minimized.

@janvorli

janvorli Jan 26, 2017
Member

Hm, what is the difference between setting the default version here and at the other place? If we set it here, it can be overriden by the command line option. If there is no clang option, it would take this default version.
If we do it this new way, then the effect is the same. If the command line option is used, the version is set accordingly. If it is not set, we set it later to the default one.

This comment has been minimized.

@gkhanna79

gkhanna79 Jan 26, 2017
Author Member

I am simply trying to define a consistent way to go about this.

This comment has been minimized.

@janvorli

janvorli Jan 26, 2017
Member

Interesting. I was convinced it is the other way in the build.sh in coreclr. Obviously I was wrong.

Copy link
Member

@janvorli janvorli left a comment

LGTM, thank you!

@karelz karelz merged commit bab18df into dotnet:master Jan 27, 2017
8 checks passed
8 checks passed
Innerloop CentOS7.1 Debug Build and Test Build finished.
Details
Innerloop CentOS7.1 Release Build and Test Build finished.
Details
Innerloop OSX Debug Build and Test Build finished.
Details
Innerloop OSX Release Build and Test Build finished.
Details
Innerloop Ubuntu14.04 Debug Build and Test Build finished.
Details
Innerloop Ubuntu14.04 Release Build and Test Build finished.
Details
Innerloop Windows_NT Debug Build and Test Build finished.
Details
Innerloop Windows_NT Release Build and Test Build finished.
Details
@karelz karelz modified the milestone: 2.0.0 Feb 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.