Added MinGW binary pkg-config wrapper

fdopen / opam-repository-mingw

windows package repository for OPAM (mingw and msvc)

https://fdopen.github.io/opam-repository-mingw/

Creative Commons Zero v1.0 Universal

96 stars 34 forks source link

Added MinGW binary pkg-config wrapper #80

Closed MSoegtropIMC closed 4 years ago

MSoegtropIMC commented 4 years ago

This PR adds a binary MinGW wrapper for pkg-config.

The wrapper povided by Cygwin is a shell script and cannot be called e.g. by MinGW opam.

fdopen commented 4 years ago

This doesn't work.

%arch% is a global variable. It is quite common to use a 32-bit OCaml switch together with 64-bit version of opam.
depext-cygwinports already provides a binary called pkg-config.exe. (There is conflicts: [ ... ]).

MSoegtropIMC commented 4 years ago

Thanks - let me see if I have any issues with the provided pkg-config. I set this PR to draft.

MSoegtropIMC commented 4 years ago

@fdopen : thanks - the depext-cygwinports package has indeed the pkg-config functionality I need and works as expected. I will close this PR.

One question: do you have a list of cygwin packages one should install upfront, that is which depext-cygwinports cannot handle? I currently (mostly for historic reasons) install all cygwin packages I need upfront, but I would like to replace as much as possible with depext. If you don't have such a list, I could create one by removing everything which is handled by depext and see what remains.

fdopen commented 4 years ago

Better keep your manual instructions. I don't know, what will happen to depext-cygwinports. The current usage might not work any longer with opam 2.1. I don't know, what will replaced it and how.

MSoegtropIMC commented 4 years ago

Better keep your manual instructions.

Thanks for the hint - I appreciate it! I was about to change it. Actually I use a batch file to install cygwin with the required packages, so there would be no difference for the user, but a working depext for cygwin would have been nice to install further opam packages.