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

Remove Store user profile #2539

Merged
merged 4 commits into from
May 27, 2017
Merged

Remove Store user profile #2539

merged 4 commits into from
May 27, 2017

Conversation

ramarag
Copy link
Member

@ramarag ramarag commented May 26, 2017

@@ -117,14 +106,16 @@ The host will probe in the following order of precedence for `dotnet run` and ap

+ `$CORE_SERVICING` on Unix or `%ProgramFiles(x86)%\coreservicing` on Windows.

This comment was marked as spam.

+ The global shared package store
+ The Shared Framework directory
+ ~~`DOTNET_HOSTING_OPTIMIZATION_CACHE` deprecated in favor of `DOTNET_SHARED_STORE`~~
+ `--additionalprobingpaths` specified in the command line
+ `runtimeOptions.additionalProbingPaths` (includes NuGet cache probe specified by the CLI for `dotnet run`)
+ Application `bin` directory

**NOTE:** `--additionalprobingpaths` can be passed template paths like below and the host will interpret `{arch}/{tfm}` appropriately to look for assets:
`%USERPROFILE%\.dotnet\store\{arch}\{tfm}` or `$HOME/.dotnet/store/{arch}/{tfm}`

This comment was marked as spam.

if (pal::realpath(&real))
pal::string_t probe_path = path;

if (pal::realpath(&probe_path) && pal::directory_exists(probe_path))

This comment was marked as spam.

This comment was marked as spam.

@@ -255,5 +286,18 @@ private void MoveRuntimeConfigToSubdirectory(TestProjectFixture testProjectFixtu

testProjectFixture.TestProject.RuntimeConfigJson = destRuntimeConfig;
}

private string CreateAStrore(TestProjectFixture testProjectFixture)

This comment was marked as spam.

Copy link
Member

@gkhanna79 gkhanna79 left a comment

Choose a reason for hiding this comment

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

High level looks fine to me, modulo some comments. @eerhardt PTAL as well.

dotnet = dotnet ?? _sdkDotnet;
outputDirectory = outputDirectory ?? _testProject.OutputDirectory;

var publishArgs = new List<string>();

This comment was marked as spam.

.Should()
.Pass()
.And
.HaveStdOutContaining("Hello World");

This comment was marked as spam.

@ramarag
Copy link
Member Author

ramarag commented May 26, 2017

@gkhanna79 @eerhardt i have address all the concerns above

{
realpaths->push_back(real);
//Check if we can extrapolate {arch}<DIR_SEPARATOR>{tfm} for probing stores

This comment was marked as spam.

.And
.HaveStdOutContaining("Hello World")
.And
.HaveStdErrContaining($"Adding tpa entry: {store_path}");

This comment was marked as spam.

This comment was marked as spam.

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.

Remove UserProfile lookup for Store
4 participants