-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to include space in the path to dictionary? #43
Comments
This is not possible right now because the argument handling doesn't do full shell quoting; see #25 for details. I haven't thought about this problem in a long time, but I'll take another look at whether there's a good way to handle it. It may be possible to just preparse argument with Python's |
Old argument handling couldn't handle quoted arguments. This handles quoted arguments in Python and passes an array of strings to the C++ MeCab code. Most of the complexity in this change is not in Python but in SWIG. Most details are handled by a SWIG typedef converter. This should fix #25 and #43.
So I took a look at it and this issue should be fixed in master. Unfortunately that broke Node-based parsing for some reason. Should be fixable at least, though can't say how long it'll take. |
OK, fixed the issues, so this should be resolved in the latest release candidate. Could you please install the latest version like below and tell me if it fixes your issue?
|
Thanks! I have tried two patterns like below and both of them works!
|
OK, thanks for the confirmation. I'll release a new update with this change soon. |
FYI: After this update, the escaped character also need to be changed as below. Before:
After:
|
Fix for this is included in the 1.0 release today. |
I want to include space in path and have tried like below.
All three of them failed as RuntimeError.
Is any way to include space in path?
The text was updated successfully, but these errors were encountered: