Skip to content
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

SevenZipSharp.Interop.NoReferences: fix naming of targets file for importing to work #19

Merged
merged 1 commit into from
Nov 30, 2023

Conversation

RoadTrain
Copy link
Contributor

@RoadTrain RoadTrain commented Nov 24, 2023

By convention NuGet will only import .targets file from a package if its name matches the package id.

Previously, for package SevenZipSharp.Interop.NoReferences the .targets file name was SevenZipSharp.Interop.targets, which resulted in NuGet not importing it, and as a result 7z.dll weren't copied to the output folder.

This fixes it by copying the .targets file with a correct name. Unfortunately I had to explicitly list all copied files in the .nuspec but I think it's acceptable.

I verified this fix locally, now the binaries get copied to the output folder.

NuGet will only import `.targets` file from a package if its name matches the package name.
@luuksommers luuksommers merged commit 858090d into luuksommers:master Nov 30, 2023
@luuksommers
Copy link
Owner

luuksommers commented Nov 30, 2023

Thanks for your PR, I'll publish a new version

@luuksommers
Copy link
Owner

I've just published the new version to nuget

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