search

kevoreilly / capemon

capemon: CAPE's monitor
GNU General Public License v3.0
102 stars 49 forks source link

Updated docs #51

Closed RazviOverflow closed 1 year ago

RazviOverflow commented 1 year ago

Added brief description on how to compile and add new hooks.

kevoreilly commented 1 year ago

Hi Razvi, I appreciate the PR, however VS2017 is the current targeted version of Visual Studio. I am surprised you have been able to get it to compile with a newer version, which is great, but until the project is itself upgraded to a different compiler version, the documentation needs to reflect the current version.

RazviOverflow commented 1 year ago

Apparently VS2022 was able to Retarget the project successfully. Feels like I just got lucky this time, this isn't usually the case.

RazviOverflow commented 1 year ago

However, I wonder whether VS2017 would be any different while importing the project. Maybe I'll give it a try later and update the docs accordingly.

kevoreilly commented 1 year ago

There is no need to retarget or import anything in VS2017 - it just works.

kevoreilly commented 1 year ago

Hi Razvi, just to let you know I have begun testing VS2022 and so far it seems to work flawlessly. If all the tests pass (as I now assume they will), I will be moving the project to this compiler version in a dedicated commit.