-
Notifications
You must be signed in to change notification settings - Fork 236
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
Add exported/installed CMake target #123
base: master
Are you sure you want to change the base?
Conversation
This commit adds the `WIL` CMake interface library, which allows the project to be exported and installed for consumption by other CMake projects in the standard fashion. The logic to create the NuGet package has been replaced with the built-in CPack NuGet support introduced with CMake 3.13, which automatically packages any files and directories installed by the target. This replaces the `make_wil_nupkg` custom target with the standard CPack `package` target, which builds the package with all the configured package generators (for now that's just NuGet). Note that the original NuGet specification file used the new license style, whereas CPack still generates a NuGet specification with the deprecated style. This will hopefully be resolved by future versions of CPack. Resolves microsoft#118.
It seems like CMake is failing to initialize on the build servers that create and sign the NuGet package. Here's the output:
The only thing that stands out is that it seems that these build machines are still using VS 2017 (something out of our control, unfortunately). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like goodness but I'm not CMake savvy enough to read this. @dunhor can you check this as I understood you created the cmake and nuget build projects?
There were earlier problems with building the changes on the signing machines (see earlier comment of mine). I haven't checked recently, but I assume those issues still exist. Additionally, #137 from a while ago appears to have addressed the same problem, so there's likely some conflict resolution needed at the very least |
Apologies for abandoning this patch on your lap, I've been unable to find the time recently to ensure it's kept up to date. |
This commit adds the
WIL
CMake interface library, which allows theproject to be exported and installed for consumption by other CMake
projects in the standard fashion.
The logic to create the NuGet package has been replaced with the
built-in CPack NuGet support introduced with CMake 3.13, which
automatically packages any files and directories installed by the
target. This replaces the
make_wil_nupkg
custom target with thestandard CPack
package
target, which builds the package with all theconfigured package generators (for now that's just NuGet).
Note that the original NuGet specification file used the new license
style, whereas CPack still generates a NuGet specification with the
deprecated style. This will hopefully be resolved by future versions of
CPack.
Resolves #118.