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

Allow 1 whitespace to match many whitespaces #70

Merged

Conversation

luke-orden
Copy link
Collaborator

@luke-orden luke-orden commented May 4, 2017

The format of syslog messages can change, especially the amount of
whitespace, for example in Junos the date can be shown as:

May  4 11:04:38

Or

May 14 11:04:38

This change makes it so 1 whitespace in line will match any number of
whitespaces. You do still need to be explicit when matching whitespaces
in values

This is linked to #65

The format of syslog messages can change, especially the amount of
whitespace, for example in Junos the date can be shown as:

```
May  4 11:04:38
```
Or
```
May 14 11:04:38
```

This change makes it so 1 whitespace in `line` will match any number of
whitespaces. You do still need to be explicit when matching whitespaces
in `values`
@mirceaulinic mirceaulinic merged commit 8ad179b into napalm-automation:master May 4, 2017
@mirceaulinic mirceaulinic added this to the 0.0.1b2.dev1 milestone May 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants