Win tests compilation fix.

[allexzander]

Signed-off-by: allexzander blackslayer4@gmail.com

Fix for Windows build error of not being able to locate #include <sqlite3.h> in tests.

[camilasan]

You could expand on the commit message with what was the error. Just for later when looking at the git history it will be easy to understand what happened.

[FlexW]

If you change PRIVATE to PUBLIC on line 71 and 73 in src/csync/CMakeLists.txt it should also work and then sqlite3 will not be included globally. I think that should be the proper fix for that.

[allexzander]

If you change PRIVATE to PUBLIC on line 71 and 73 in src/csync/CMakeLists.txt it should also work and then sqlite3 will not be included globally. I think that should be the proper fix for that.

@er-vin @camilasan

This sounds reasonable. Any objections? Asking, since I am not CMAKE-expert.

[er-vin]

I agree with @FlexW this would be a better fix. We should slowly move to those more modern constructs.

[allexzander]

Alright then, @er-vin @FlexW @camilasan

Let's approve + merge it.

[er-vin]

@allexzander I'm fine with the content of the commit now, still could you please address @camilasan comment about have a more descriptive commit message? She's right that this kind of things might come in handy.

[nextcloud-desktop-bot]

AppImage file: Nextcloud-PR-2901-3d266b9246bb61c810cf81858b0c2b6768c988d7-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.