emcrisostomo / fswatch

A cross-platform file change monitor with multiple backends: Apple OS X File System Events, *BSD kqueue, Solaris/Illumos File Events Notification, Linux inotify, Microsoft Windows and a stat()-based backend.
https://emcrisostomo.github.io/fswatch/
GNU General Public License v3.0
5.06k stars 330 forks source link

Vcpkg support (Windows package manager) #251

Open YenForYang opened 4 years ago

YenForYang commented 4 years ago

Hi, I was wondering if vcpkg integration could be fixed. Someone posted the issue more than a year ago in vcpkg's repo around a year ago: https://github.com/Microsoft/vcpkg/issues/5843.

According to the post, the problems preventing inclusion are:

  • libfswatch_config.h are not found in source.
  • This port only supports Linux.
  • Without build documentation.
  • Some macro such as PACKAGE_STRING PACKAGE_NAME PACKAGE_BUGREPORT PACKAGE PACKAGE_URL are not found.
  • It seems like that the port has code bug in source. eg: optarg and getopt not found.

There is currently a pull request on fixing CMAKE, but I haven't looked into it: https://github.com/emcrisostomo/fswatch/pull/250#issue-465250042

Related: https://github.com/emcrisostomo/fswatch/issues/176.

Thanks.

birbilis commented 4 years ago

why not use MSYS2 with MINGW-W64 (that one also supports 32-bit btw, the name just implies it's an evolution of the old MINGW) with GNU tooling instead of having to fix CMAKE for building on Windows?

emcrisostomo commented 2 years ago

The only requirement I ask on a PR that supports CMake is that

For this reason #250 has been closed.

There's a newer PR (#282) which is quite elaborated, but it has the same problems, and I've asked the author whether he's willing to help converge to a situation in which the previously mentioned requirements are met.