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

Add VCPKG section to README.md (fix #161) #180

Merged
merged 3 commits into from
Feb 10, 2021
Merged

Conversation

RDCH106
Copy link
Contributor

@RDCH106 RDCH106 commented Feb 8, 2021

Related issue #161

@pariterre
Copy link
Member

Wow! That is so nice :)
Please ping me when you are ready for merge! Am I supposed to release a new version afterward to trigger everything in vcpkg (for instance, on conda-forge, if I want a new version to be built, I need to create a new specific release)?

@RDCH106
Copy link
Contributor Author

RDCH106 commented Feb 9, 2021

Hi!

The integration is done because ezc3d is integrated in vcpkg. The CI in vcpkg were passed for ezc3d with successfully result:

microsoft/vcpkg#14016

You don't need to do anything. It is already done.

VCPK is not as flexible as conda-forge or pip, each version of VCPKG includes libraries in a specific version and the huge CI process assures that all it is ok. Because of the characteristics of C and C++ it is difficult to design a package manager that lets us specify a specific version of a library. You can check the current version of ezc3d included in vcpkg searching it in the ports folder:

https://github.com/microsoft/vcpkg/tree/master/ports/ezc3d

As you can see the integration is using your release tar.gz package:

https://github.com/microsoft/vcpkg/blob/master/ports/ezc3d/portfile.cmake

You can use ezc3d from vcpkg following the instructions in this PR. Install it using the following command:

vcpkg install ezc3d

It is also possible to specify the architecture, the platform even linkage using triples:

vcpkg install ezc3d:x64-windows

Thank you for support ezc3d library! 😉

Copy link
Member

@pariterre pariterre left a comment

Choose a reason for hiding this comment

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

LGTM

@pariterre pariterre merged commit e239db9 into pyomeca:dev Feb 10, 2021
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.

None yet

2 participants