NuGet update failes when project path contains spaces.

[TomW-Skyline]

The command arguments need to be surrounded by quotes, because otherwise there are problems when they contain spaces.

dependabot-core/nuget/lib/dependabot/nuget/native_helpers.rb

Line 57 in 89ebc55

proj_path,

[TomW-Skyline]

This is still not working as expected unfortunately

[abdulapopoola]

Sorry to hear that; will reopen the issue and tagging @brettfo for awareness

[brettfo]

@TomW-Skyline Do you have a recent log file you can share? There are likely more places we need to surround arguments.

[TomW-Skyline]

@brettfo here is an example:
updater | running NuGet updater:
updater | /opt/nuget/NuGetUpdater/NuGetUpdater.Cli update --repo-root /home/dependabot/dependabot-updater/repo --solution-or-project /home/dependabot/dependabot-updater/repo/ResourceScheduling_Add\ Contact\ To\ Job_1/ResourceScheduling_Add\ Contact\ To\ Job_1.csproj --dependency Skyline.DataMiner.Utils.InteractiveAutomationScriptToolkit --new-version 8.0.1 --previous-version 7.0.2 --verbose
updater | 2024/02/20 09:16:34 ERROR <job_789283670> Block argument of NuGetConfigCredentialHelpers::patch_nuget_config_for_action causes an exception Unrecognized command or argument 'Contact'.
updater | Unrecognized command or argument 'To'.
updater | Unrecognized command or argument 'Job_1/ResourceScheduling_Add'.
updater | Unrecognized command or argument 'Contact'.
updater | Unrecognized command or argument 'To'.
updater | Unrecognized command or argument 'Job_1.csproj'.
updater |
updater | Description:
updater | Applies the changes from an analysis report to update a dependency.
updater |

[brettfo]

@TomW-Skyline Added PR #9092 to allow proper execution of the tool with spaces in the command line.

[TomW-Skyline]

@brettfo That's great! There is probably no easy way to already test it?

[brettfo]

@TomW-Skyline I created a repo with a space in the path and did a full dependabot update run to verify the path fix. I've also just submitted #9104 which will fail a unit test on a malformed path so we can detect this faster.