search

nephila / djangocms-installer

Console wizard to bootstrap django CMS projects
https://djangocms-installer.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
177 stars 78 forks source link

Fix handling of project path with spaces. #307

Closed wismill closed 6 years ago

wismill commented 7 years ago

Attempt to use the installer with a path containing spaces currently results in an error.

This PR fixes this issue and is more Pythonic (see hereinafter).

According to the Python docs:

Providing a sequence of arguments is generally preferred, as it allows the module to take care of any required escaping and quoting of arguments (e.g. to permit spaces in file names).

yakky commented 7 years ago

@wismill nice fix (still wondering why I came up with that ugly syntax). Apparently there are failure in tests, could you check them?

yakky commented 6 years ago

Fix #310

yakky commented 6 years ago

Closed via #313