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

Test: Fixes for FreeBSD in run-test script #3306

Merged
merged 1 commit into from Sep 18, 2015
Merged

Test: Fixes for FreeBSD in run-test script #3306

merged 1 commit into from Sep 18, 2015

Conversation

ghost
Copy link

@ghost ghost commented Sep 18, 2015

Although it is failing with:

Pulling down Microsoft.DotNet.CoreFx.FreeBSD.TemporaryTestHost.0.0.2-prerelease.nupkg
Unzipping to packages/Microsoft.DotNet.CoreFx.FreeBSD.TemporaryTestHost.0.0.2-prerelease
unzip: Unrecognized archive format
Package not laid out as expected

but this change was required to get to this point.

@VSadov, @mmitche. we made necessary changes in BuildTools dotnet/buildtools#180. Is there anything else required to have FreeBSD test host published on myget?

/cc: @janhenke

@janhenke
Copy link
Member

LGTM

1 similar comment
@stephentoub
Copy link
Member

LGTM

nguerrera added a commit that referenced this pull request Sep 18, 2015
Test: Fixes for FreeBSD in run-test script
@nguerrera nguerrera merged commit 4b15f40 into dotnet:master Sep 18, 2015
@mmitche
Copy link
Member

mmitche commented Sep 18, 2015

The temporary test host was a manually created package with some bits that were required but not yet OSS. I am not quite sure how much of the contents were still required. Can you get the tests to run locally (if so then it's not necessary any longer)

@ghost
Copy link
Author

ghost commented Sep 18, 2015

@mmitche,

I am not quite sure how much of the contents were still required

Do you mean this is not required -> /run-test.sh#L86-L160

Can you get the tests to run locally

This is what I am doing, trying to execute ./run-test.sh on FreeBSD locally? :)

@mmitche
Copy link
Member

mmitche commented Sep 18, 2015

Some of the overlay will be required until we have test layout publishing all sorted out for all platforms. However, it's possible that the TemporaryTestHost bit of that overlay is not required right now.

@ghost ghost deleted the test-runner branch February 20, 2016 02:48
@karelz karelz added this to the 1.0.0-rtm milestone Jan 25, 2017
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.

6 participants