Comment 4 for bug 2024325

Revision history for this message
Robie Basak (racb) wrote :

This looks good. Thanks!

I added a line to "Where problems could occur".

I suggest changing the changelog entry to simply:

  * Fix crash on startup (LP: #2024325).

Saying that we backported the fix seems superfluous to me. The people who read this are likely to be non-technical, and those technical can always review the diff and/or dep3 headers to understand the code changes if they're interested. But what we do want to do is make it clear to users that we are actually fixing the bug that they have experienced. "error with library import" sounds like something that could just be an edge case and describes the root cause that the user might not know as opposed to the actual user experience. "crash on startup" makes it clear that if a user was affected by that then that's what we're addressing by this fix.

Please let me know if you're OK with this fix and I'll go ahead and sponsor. After that, it will await review by the SRU team.