Comment 8 for bug 2018215

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks for taking a look Robie. The upstream patches you mentioned in comment #6 are indeed touching the same code, however, they are fixing another issue, not the one I am trying to address here. This is the upstream bug they are fixing:

https://bugs.ruby-lang.org/issues/16676

So yes, it would be good to also have this fix but no user reported this issue to us yet. Ideally, we should update ruby2.7 version 2.7.0 that we have in Focal to version 2.7.8 (latest upstream release of this track) which contains those fixes and more, but we do not do microrelease updates to ruby2.7. As I am trying to fix this single issue reported by our user I believe the single patch proposed in this SRU is enough.

About testing (comment #7), as you imagined, it is covered by the upstream test suite. There is test/ruby/test_hash.rb which contains multiple tests covering hash usage. All the tests passed during build and autopkgtest time for me. I do no think we need to do any extra manual test.