[FIX] Allow spaces in path to target modules directory

OCA / odoo-module-migrator

Python library to migrate odoo base code from a version to another

GNU Affero General Public License v3.0

123 stars 112 forks source link

[FIX] Allow spaces in path to target modules directory #74

Closed gjanssens closed 1 year ago

gjanssens commented 1 year ago

If the target modules directory contains spaces, the shell command would fail before this patch with a "no such file or directory" error as it would choke on the first space. By wrapping the directory name in single quotes before executing the shell command this is avoided.

OCA-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch master-ocabot-merge-pr-74-by-legalsylvain-bump-nobump, awaiting test results.

OCA-git-bot commented 1 year ago

Congratulations, your PR was merged at ce1d698608633deb8f5b81af82283b8f4bf9393f. Thanks a lot for contributing to OCA. ❤️