-
Notifications
You must be signed in to change notification settings - Fork 31
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
Commit 77e1a17 breaks multiplex.t #26
Comments
Oh bother. This is why it stinks to have a failing test suite under 5.16. I should have tested on an older Perl. I can look into this in the next few days. On Dec 26, 2013, at 1:45, Eli Lindsey notifications@github.com wrote:
|
I had some time, so I fixed this today. FYI, tests work for me on mac in 5.16... The issue was global scope of config changes, we have a couple scripts (multiplex and chain) that can create & invoke operations themselves, so global settings on Getopt::Long::Configure were conflicting. |
FYI, fixed in SHA ce4ecff |
Sorry, I meant 5.18. See Thanks for the fix! On 12/26/2013 12:28 PM, Ben Bernard wrote:
|
Looks like the repeat syntax used for aggregator snippets isn't compatible with bundling:
The text was updated successfully, but these errors were encountered: