optparse breaks when spaces or special characters are in default values #16
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Error occurs when a space is in argument default values (or passed from command line too, I believe). This is because the value assignment is not quoted at all in the generated tmp file.
Also breaks when special characters (such as a single quote, double quote, any bash langauge specials) are in default values or passed.
Example code
Output
Expected result
Should handle spaces & special characters gracefully, without throwing an error.
Fix
Easy fix, wrap assignment in quotes, and escape any quotes in the input. This fixes most special character use cases, better than nothing...
Pull request is on the way.