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

Provide dummy runtime_version.h when skipped #35579

Merged
merged 1 commit into from
Apr 29, 2020
Merged

Provide dummy runtime_version.h when skipped #35579

merged 1 commit into from
Apr 29, 2020

Conversation

am11
Copy link
Member

@am11 am11 commented Apr 28, 2020

With -skipgenerateversion, all MSBuild invocations are skipped.
In that case, we generate a dummy version.c file. Since
runtime_version.h is added as a required header, generate its dummy
variant in the same context.

With `-skipgenerateversion`, all MSBuild invocations are skipped.
In that case, we generate a dummy `version.c` file. Since
`runtime_version.h` is added as a required header, generate its dummy
variant in the same context.
@am11
Copy link
Member Author

am11 commented Apr 28, 2020

cc @jkotas, with this PR changes, following succeeds (from clean repo):

./src/coreclr/build-runtime.sh -skipgenerateversion -nopgooptimize \
    -cmakeargs -DCLR_CMAKE_BUILD_TESTS=1

@jkotas
Copy link
Member

jkotas commented Apr 28, 2020

Thanks for fixing this up!

eng/native/build-commons.sh Outdated Show resolved Hide resolved
@am11
Copy link
Member Author

am11 commented Apr 29, 2020

Builds are green on AzDO, but showing up as In-Progress on GitHub.

@jkotas jkotas merged commit d51bdd6 into dotnet:master Apr 29, 2020
@am11 am11 deleted the feature/build-fixes branch April 29, 2020 13:18
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
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.

None yet

2 participants