Skip to content

Conversation

@thobianchi
Copy link
Contributor

This removes the requirements to bash4.
I substituted readarray with a while loop and associative array with dynamic variable names.

I tested this on:

$ bash --version
GNU bash, version 3.2.57(1)-release (x86_64-suse-linux-gnu)
Copyright (C) 2007 Free Software Foundation, Inc.

@boomskats
Copy link
Contributor

Sorry @thobianchi, I've literally just seen this!

I'll run some tests over the weekend and merge it if it performs comparatively well with ~1k sessions. Nice work.

@boomskats boomskats merged commit 5a4e8f9 into Boemska:master Apr 3, 2019
@boomskats
Copy link
Contributor

'over the weekend' = 'exactly 1 year later'

Thanks for the fix @thobianchi, works and performs well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants