Skip to content

[blitz] Update to the latest commit and fix no dll and lib generated#10546

Merged
dan-shaw merged 2 commits intomicrosoft:masterfrom
NancyLi1013:dev/NancyLi/10531-fix-blitz
Mar 28, 2020
Merged

[blitz] Update to the latest commit and fix no dll and lib generated#10546
dan-shaw merged 2 commits intomicrosoft:masterfrom
NancyLi1013:dev/NancyLi/10531-fix-blitz

Conversation

@NancyLi1013
Copy link
Copy Markdown
Contributor

There are no dlls or libs generated by blitz current version on Windows platform in vcpkg.

So I update this port to the latest commit to fix this issue.

Related issue #10531

Note: No features need to test.

@HerveAncher
Copy link
Copy Markdown

Dear @NancyLi1013 how can try to install Blitz via vcpkg with this PR?
It is possible to tell at vcpkg to use your repo instead its one?

@HerveAncher
Copy link
Copy Markdown

Done, I directly changed the value in my portfile.cmake.

And....... it works!! The DLL and library (static and dynamic) are properly copied at good places.

@HerveAncher
Copy link
Copy Markdown

Tested for x64-windows, installed is OK.
For x86-windows, compilation seems to failed.

@HerveAncher
Copy link
Copy Markdown

Finally it passed for x86-windows, the first there was an error about the write operation around genvecbfn.exe tools.

So it is OK under Windows 10, with standard tools provided by vcpkg (CMake 3.14.0 + Python 2.7.16)

Copy link
Copy Markdown
Contributor

@PhoebeHui PhoebeHui left a comment

Choose a reason for hiding this comment

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

LGTM

@NancyLi1013
Copy link
Copy Markdown
Contributor Author

/azp run

@PhoebeHui PhoebeHui added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Mar 26, 2020
@dan-shaw dan-shaw merged commit ec11757 into microsoft:master Mar 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants