-
-
Notifications
You must be signed in to change notification settings - Fork 177
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
Support for vcpkg on Windows #865
Comments
Are there any bindings out there that makes use of it ? Would it conflict with the current automatic system-deps usage ? It looks interesting though |
See gdesmott/system-deps#8 and gdesmott/system-deps#12 for previous discussion and work on this. |
As I mentioned, there is However, to be clear, I am not necessarily bound to using |
See also microsoft/vcpkg#22881 regarding build failure and version update. |
@ruifengx could you elaborate what you mean with "support" for vcpkg? |
That would be automatically locating the package with |
Then I guess there is nothing to do from our side until gdesmott/system-deps#12 is merged. |
Okay, and it seems that PR won't be merged any time soon in its current status. I guess I have to live with the status quo, or try to help that PR land. Thanks anyway. Do you prefer this issue closed, or should I leave it open until the upstream PR is merged? |
I think we can close it and open a new one when it becomes actionable :) |
Looks like
vcpkg
has support forgtk
.vcpkg
has long been my preferred "package manager" for C/C++ libraries, and it looks very popular among Windows programmers (14.8k stars atm). There is a Rust binding forvcpkg
(vcpkg-rs
), and I find it relatively easy to use (see the use ofvcpkg
inrust-sdl2
, for example).Installing
gtk
usingvcpkg
should be as easy asvcpkg install gtk
. Compare this to the steps listed in the Book currently. I wonder whether or not it is possible to add support forvcpkg
here, so that installation on Windows could be less painful (at least from my perspective).If you find this change appropriate, I could work on this and try to submit a PR, but I will need some guidance on which files to modify (it seems the
-sys
crates here are automatically generated, and I currently have very little knowledge on that generator). The change should only involve modifyingCargo.toml
(adding an optional dependency onvcpkg
) andbuild.rs
(making use ofvcpkg
to locate the library to link with), and we should be able to make both modifications backward compatible (e.g. by feature-gating the use ofvcpkg
).The text was updated successfully, but these errors were encountered: