Escape Paths Aruments on Commandline

dizda / CloudBackupBundle

Be able to backup your database(s) and upload it to the cloud (Dropbox, Amazon S3, GoogleDrive, etc.)

MIT License

196 stars 57 forks source link

Escape Paths Aruments on Commandline #100

Closed scasei closed 8 years ago

scasei commented 8 years ago

Paths should be escaped for better portability.

scasei commented 8 years ago

all commits belong togehter, so only last commit 887c1a5d4ec10bac89bb4044508c3469480b597a has a nice build and test run, except php5.3, there the memory limit is breaked

dizda commented 8 years ago

Hey @scasei, thanks for the PR.

What is the reason the escape the path? When someone use a fancy path with special chars? Or this is to support some OS like Windows?

scasei commented 8 years ago

both: Windows and special chars, eg. 'C:\Program Files (x86)\wamp\www'

;)

dizda commented 8 years ago

Nice then, thanks! :-)

dizda commented 8 years ago

https://github.com/dizda/CloudBackupBundle/releases/tag/3.2.4 released, thanks mate!