Store regexp options properly #1326

Merged
merged 1 commit into from Nov 28, 2016

Projects

None yet

2 participants

@okkez
Contributor
okkez commented Nov 25, 2016

No description provided.

@okkez okkez Store regexp options properly
79b59d2
@okkez
Contributor
okkez commented Nov 25, 2016

With Fluentd v0.14.6, fluent-plugin-grok-parser's test passed.
But with Fluentd v0.14.9 , fluent-plugin-grok-parser's test failed.

I noticed that Fluent::Compat::TextParser::RegexpParser#configure does not copy Regexp#options.

@tagomoris
Member

LGTM

@tagomoris tagomoris merged commit d4313ec into fluent:master Nov 28, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment