Since the new spawn API, which is used by utils_spawn_sync() as well, passing a whole command line in argv[0] doesn't work any longer. So use the new spawn API directly which conveniently offers to pass the command either as command line or as argv.
Tested on Linux and Windows. I bumped the required API version. It seems unfortunately we didn't bump the API version in Geany when merging the spawn API but the currently used 226 is newer than the spawn API and is already available in the last Geany release, so it should be ok. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/372
-- Commit Summary --
* Use new spawn API to fix broken Win32 ctags.exe command execution
-- File Changes --
M geanyctags/src/geanyctags.c (24)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/372.patch https://github.com/geany/geany-plugins/pull/372.diff
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/372
Merged #372.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/372#event-579336011
@eht16 Ah, missed this one. Works fine on linux and I trust you on Windows :-)
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/372#issuecomment-192944062
github-comments@lists.geany.org