On 18.12.23 11:49, vignesh C wrote:
Few comments: 1) Now that the MSVC build scripts are removed, should we have the reference to "MSVC build scripts" here? ltree.h:
I think this note is correct and can be kept, as it explains the historical context.
2) I had seen that if sed/gzip is not available meson build will fail: 2.a) Program gsed sed found: NO meson.build:334:6: ERROR: Program 'gsed sed' not found or not executable
Yes, this would need to be improved. Currently, sed is only required if either selinux or dtrace is enabled, which isn't supported on Windows. But we should adjust the build scripts to not fail the top-level setup run unless those options are enabled.
2.b) Program gzip found: NO meson.build:337:7: ERROR: Program 'gzip' not found or not executable
gzip is only required for certain test suites, so again we should adjust the build scripts to not fail the build but instead skip the tests as appropriate.
