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

Add ability to only copy native test files to managed tests in build-test scripts #26451

Conversation

jkoritzinsky
Copy link
Member

As part of supporting #26404, this PR adds additional arguments to build-test.cmd and build-test.sh to enable only copying native test binaries to their respective managed test output directory without building tests. Additionally, it adds another parameter to support skipping generating the Core_Root layout and the CoreFX test host, which can also be used as part of #26404.

To skip the native and managed builds and copy the native assets to the managed test output directory, use the copynativeonly parameter to build-test. To skip setting up Core_Root and the CoreFX test host, use skipgeneratelayout.

This PR also changes runtest.sh to not generate the Core_Root layout when not told where to find Core_Root.

cc: @dotnet/coreclr-infra

Copy link
Member

@trylek trylek left a comment

Choose a reason for hiding this comment

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

Looks great to me, thanks for laying down the foundations for my work item!

@jkoritzinsky
Copy link
Member Author

This test failure is showing up in multiple PRs. Since this PR makes no changes to the process for running CoreFX tests, I'm going to merge it in.

@jkoritzinsky jkoritzinsky merged commit 05c6400 into dotnet:master Aug 30, 2019
@jkoritzinsky jkoritzinsky deleted the optional-individual-native-test-binary-copying branch August 30, 2019 23:08
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…test scripts (dotnet/coreclr#26451)

* Add MSBuild targets neccesary to copy native test assets to managed projects that need them in a way separate from the managed test build.

* Hook up native test binary copying through build-test scripts.

* Unix script changes/cleanup

* Remove unconditional generatelayout in runtest.sh

* Add new options to usage


Commit migrated from dotnet/coreclr@05c6400
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.

3 participants