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

Stop build from leaving localpkg cache in src #8934

Merged
merged 1 commit into from
Jan 13, 2017

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Jan 13, 2017

@@ -67,6 +67,9 @@
<PackagesBinDir>$(__PackagesBinDir)</PackagesBinDir>
<PackagesBinDir Condition="'$(__PackagesBinDir)'==''">$(BinDir).nuget\</PackagesBinDir>

<PackageInstallPath>$(__PackageInstallPath)</PackageInstallPath>
Copy link
Member

Choose a reason for hiding this comment

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

@karajas isn't there a way to disable the installing of this package instead of hacking the path here?

Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/dotnet/buildtools/blob/master/src/Microsoft.DotNet.Build.Tasks.Packaging/src/PackageFiles/Packaging.targets#L1217

Just set that property to disable installing this package locally as it isn't needed for you guys.

Copy link
Member Author

Choose a reason for hiding this comment

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

@weshaggard where's the right place to set that property?

Copy link
Member

Choose a reason for hiding this comment

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

In the packaging.props would be the place I think to set it. It needs to be set in all the pkgprojs essentially.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah SkipInstallLocallyBuiltPackages for any packages that you don't want installed.

Copy link
Member Author

Choose a reason for hiding this comment

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

It looks like dir.props is the most reasonable place to set that property - the lowest level file that all the .pkgproj files import is the root-level dir.props (setting it in packages.builds did not work). I'll put up a PR to do so.

Copy link
Member

Choose a reason for hiding this comment

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

OK. I guess you guys don't have a packages.props file like corefx. I would just put it where ever your other packaging properties are getting set.

@wtgodbe
Copy link
Member Author

wtgodbe commented Jan 13, 2017

Updated

@wtgodbe wtgodbe merged commit b3136bb into dotnet:master Jan 13, 2017
@wtgodbe wtgodbe deleted the localpkg branch January 13, 2017 22:06
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Stop build from leaving localpkg cache in src

Commit migrated from dotnet/coreclr@b3136bb
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