pupnp / pupnp

libupnp: Build UPnP-compliant control points, devices, and bridges on several operating systems.
https://pupnp.github.io/pupnp
BSD 3-Clause "New" or "Revised" License
349 stars 114 forks source link

Windows build manual significantly out of date #415

Closed Zapunidi closed 1 year ago

Zapunidi commented 1 year ago

It seems that Windows build is either not longer supported or documentation is out of date.

Open the provided workspace build\libupnp.dsw with Visual C++ 6.0 ... In the build directory there are also VC8, VC9 and VC10 folders containing solution files for Visual Studio 2005/2008/2010 respectively.

There is no build folder in latest release version 1.14.13. I suggest either remove windows support from the readme.md file or at least put a note there, mentioning that windows builds are not supported currently. It will take 5 minutes, but will save time greatly for those, who try to make a Windows build.

Vollstrecker commented 1 year ago

Not really unsupported. The cmake instructions one chapter down applies to all plattforms including Win.

mrjimenez commented 1 year ago

I invite you to participate in #416.

Regards, Marcelo.

Zapunidi commented 1 year ago

Thank you for the clarifications. I successfully built the library on Windows with cmake chapter. I would like to help with fixing the windows build documentation, but Marcelo already did the best fix in https://github.com/pupnp/pupnp/pull/416 that is in the spirit with the rest of readme file. The merge is all that is left to do. I consider this issue closed.

Zapunidi commented 1 year ago

My initial problems with Windows build were with VS2013. I tried VS2019 and it worked. Maybe this information will help someone.