Skip to content

Windows build.cmd doesn't support -cmakeargs.#46195

Merged
1 commit merged intodotnet:masterfrom
lateralusX:lateralusX/add-cmakeargs-windows-build
Jan 22, 2021
Merged

Windows build.cmd doesn't support -cmakeargs.#46195
1 commit merged intodotnet:masterfrom
lateralusX:lateralusX/add-cmakeargs-windows-build

Conversation

@lateralusX
Copy link
Member

build.sh supports -cmakeargs making it is possible to pass custom cmake arguments down to native cmake steps, but this feature is missing on Windows when using build.cmd. Trying to do:

.\build.cmd -subset clr -c release -cmakeargs "-DFEATURE_PERFTRACING_C_LIB=1"

on Windows will result in an error since -cmakeargs is not a known command in build.ps1.

Commit adds support for -cmakeargs in build.cmd making sure it gets passed down to runtime build on both CoreClr as well as Mono.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@lateralusX lateralusX force-pushed the lateralusX/add-cmakeargs-windows-build branch from fa1a5ea to c5342cc Compare December 17, 2020 17:36
@ghost
Copy link

ghost commented Dec 17, 2020

Tagging subscribers to this area: @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

build.sh supports -cmakeargs making it is possible to pass custom cmake arguments down to native cmake steps, but this feature is missing on Windows when using build.cmd. Trying to do:

.\build.cmd -subset clr -c release -cmakeargs "-DFEATURE_PERFTRACING_C_LIB=1"

on Windows will result in an error since -cmakeargs is not a known command in build.ps1.

Commit adds support for -cmakeargs in build.cmd making sure it gets passed down to runtime build on both CoreClr as well as Mono.

Author: lateralusX
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@lateralusX lateralusX force-pushed the lateralusX/add-cmakeargs-windows-build branch from 496a8a2 to d758d45 Compare December 18, 2020 15:53
@lateralusX lateralusX force-pushed the lateralusX/add-cmakeargs-windows-build branch 3 times, most recently from 4aa5c70 to 5c1e041 Compare December 18, 2020 22:47
build.sh supports -cmakeargs making it is possible to pass custom
cmake arguments down to native cmake steps, but this feature is missing
on Windows when using build.cmd. Trying to do:

.\build.cmd -subset clr -c release -cmakeargs "-DFEATURE_PERFTRACING_C_LIB=1"

on Windows will result in an error since -cmakeargs is not a known command
in build.ps1.

Commit adds support for -cmakeargs in build.cmd making sure it gets
passed down to runtime build on both CoreClr as well as Mono.
@lateralusX lateralusX force-pushed the lateralusX/add-cmakeargs-windows-build branch from 5c1e041 to 8bbc1bc Compare January 22, 2021 16:33
@ghost
Copy link

ghost commented Jan 22, 2021

Hello @akoeplinger!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 51bf70e into dotnet:master Jan 22, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Feb 21, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants