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

[Pcg] update to latest commit #19023

Merged
merged 4 commits into from Jul 22, 2021
Merged

[Pcg] update to latest commit #19023

merged 4 commits into from Jul 22, 2021

Conversation

Thelta
Copy link
Contributor

@Thelta Thelta commented Jul 20, 2021

Describe the pull request

@ghost
Copy link

ghost commented Jul 20, 2021

CLA assistant check
All CLA requirements met.

@JonLiu1993 JonLiu1993 self-assigned this Jul 21, 2021
@JonLiu1993 JonLiu1993 changed the title Pcg update [Pcg] update to latest commit Jul 21, 2021
@JonLiu1993 JonLiu1993 added category:port-bug The issue is with a library, which is something the port should already support category:port-update The issue is with a library, which is requesting update new revision labels Jul 21, 2021
HEAD_REF master
)

file(INSTALL ${SOURCE_PATH}/include/pcg_extras.hpp DESTINATION ${CURRENT_PACKAGES_DIR}/include)
file(INSTALL ${SOURCE_PATH}/include/pcg_random.hpp DESTINATION ${CURRENT_PACKAGES_DIR}/include)
file(INSTALL ${SOURCE_PATH}/include/pcg_uint128.hpp DESTINATION ${CURRENT_PACKAGES_DIR}/include)
file(INSTALL ${SOURCE_PATH}/LICENSE.txt DESTINATION ${CURRENT_PACKAGES_DIR}/share/pcg RENAME copyright)
file(INSTALL ${SOURCE_PATH}/LICENSE-MIT.txt DESTINATION ${CURRENT_PACKAGES_DIR}/share/pcg RENAME copyright)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
file(INSTALL ${SOURCE_PATH}/LICENSE-MIT.txt DESTINATION ${CURRENT_PACKAGES_DIR}/share/pcg RENAME copyright)
file(INSTALL ${SOURCE_PATH}/LICENSE-MIT.txt DESTINATION ${CURRENT_PACKAGES_DIR}/share/${PORT} RENAME copyright)

@JonLiu1993
Copy link
Member

@Thelta ,Thanks for your pr,after finish your suggesstion changes,please run command "./vcpkg x-add-version pcg --overwrite-version" then commit the changes again

@Thelta
Copy link
Contributor Author

Thelta commented Jul 21, 2021

@Thelta ,Thanks for your pr,after finish your suggesstion changes,please run command "./vcpkg x-add-version pcg --overwrite-version" then commit the changes again

I run the command before commiting as you can see, however ci is still failing. Should i run the command and commit again?

@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Jul 22, 2021
@vicroms vicroms merged commit 1c4205c into microsoft:master Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The header only Pcg library causes compilation errors in x64 targets.
3 participants