Skip to content
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

Cannot use multiple spaces between tokens #36

Closed
thomas-mangin opened this issue Aug 27, 2013 · 4 comments
Closed

Cannot use multiple spaces between tokens #36

thomas-mangin opened this issue Aug 27, 2013 · 4 comments
Labels

Comments

@thomas-mangin
Copy link
Member

From vince.be...@gmail.com on July 19, 2013 14:10:55

Hi!

exabgp does not like multiple spaces when tokenizing. This is a bit disturbing. I am unsure where to fix that. Either we could remove contiguous spaces in _cleaned or fix various places that are using split to tokenize.

Original issue: http://code.google.com/p/exabgp/issues/detail?id=36

@thomas-mangin
Copy link
Member Author

From thomas.mangin on July 19, 2013 10:13:41

All those issues will be fixed when the configuration file is in JSON ..

It seems these two patch were not the way to go: https://code.google.com/p/exabgp/source/detail?r=5da0b0b616bbda766ac3d31459515f966c94c012 https://code.google.com/p/exabgp/source/detail?r=b90f55e68439a859540e299156adc566f600b51c Perhaps this will be cleaner ... https://code.google.com/p/exabgp/source/detail?r=28210f303ec68015fc0c970c3c0ef94865110723

@thomas-mangin
Copy link
Member Author

From thomas.mangin on July 19, 2013 11:25:41

Could you please confirm if the latest patch fix the issue ? Thank you.

Status: Fixed

@thomas-mangin
Copy link
Member Author

From vince.be...@gmail.com on July 20, 2013 15:18:36

Yes, it works for me. Thanks! BTW, spaces were not in the configuration file but in the commands sent from a process.

@thomas-mangin
Copy link
Member Author

From thomas.mangin on July 20, 2013 23:00:49

Status: Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant