Fix Windows build #246

Closed
pal1000 wants to merge 3 commits from pal1000/pkgconf:fix-win-build-tests-feature into master
2 changed files with 2 additions and 2 deletions

View File

@ -35,7 +35,7 @@ jobs:
# the code assumes msvc style printf atm # the code assumes msvc style printf atm
export CFLAGS=-D__USE_MINGW_ANSI_STDIO=0 export CFLAGS=-D__USE_MINGW_ANSI_STDIO=0
meson -Dtests=false _build meson _build
meson compile -C _build meson compile -C _build
debian-meson: debian-meson:

View File

@ -19,5 +19,5 @@ tests = [
# yuck # yuck
foreach test : tests foreach test : tests
test_file = configure_file(input: test + '.sh', output: test, copy: true) test_file = configure_file(input: test + '.sh', output: test, copy: true)

curious- the configure_file here can take a install_mode: 'rwxr-xr-x' argument, perhaps that works on windows too?

curious- the configure_file here can take a `install_mode: 'rwxr-xr-x'` [argument](https://mesonbuild.com/Reference-manual_functions.html#configure_file), perhaps that works on windows too?

install_mode is set at install time by meson install. I'm going to add a build_mode to the fs.copyfile() function, which is the replacement for configure_file(copy : true) (its a lot of historical baggage that configure_file() is used to copy files.

`install_mode` is set at install time by `meson install`. I'm going to add a `build_mode` to the `fs.copyfile()` function, which is the replacement for `configure_file(copy : true)` (its a lot of historical baggage that `configure_file()` is used to copy files.
As promised: https://github.com/mesonbuild/meson/pull/10744

aha, thanks for the information! that looks much better once that lands in

aha, thanks for the information! that looks much better once that lands in
run_command('chmod', '755', test_file, check : true) run_command(find_program('chmod', required : false, disabler : true), '755', test_file, check : true)
endforeach endforeach