Skip to content

Package improvements #56

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

Merged
merged 6 commits into from
Nov 19, 2017
Merged

Package improvements #56

merged 6 commits into from
Nov 19, 2017

Conversation

bording
Copy link
Member

@bording bording commented Nov 19, 2017

This includes the following changes:

The props file now includes files with ContentWithTargetPath instead of None which means the files will still get copied where they need to go, but they won't show up as items in Solution Explorer.

Instead of adding libgit2_hash.txt and libgit2_filename.txt to assemblies as embedded resources, those values are now added as MSBuild properties. I'll then be able to use those in the relevant target files in LibGit2Sharp.

There are now two props files, one for net461 and greater, which has all of the file copying stuff in it as usual, and one that will be referenced when using anything other than .NET Framework TFMs. That one only has the new MSBuild properties in it. This is primarily here so LibGit2Sharp has access to it since it will be netstandard2.0 only soon. We don't need any of file copy stuff there.

@ethomson Any objections to me merging this and then getting a new package up on nuget.org?

@ethomson
Copy link
Member

👍

@bording bording merged commit e6fde40 into master Nov 19, 2017
@bording bording deleted the package-improvements branch November 19, 2017 01:22
alex-weaver pushed a commit to alex-weaver/libgit2sharp.nativebinaries that referenced this pull request Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants