Merge lp://qastaging/~dominique-ramaekers/command-not-found/thinerBug into lp://qastaging/~command-not-found-developers/command-not-found/trunk

Proposed by Dominique Ramaekers
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp://qastaging/~dominique-ramaekers/command-not-found/thinerBug
Merge into: lp://qastaging/~command-not-found-developers/command-not-found/trunk
Diff against target: 23 lines (+3/-3)
1 file modified
CommandNotFound/CommandNotFound.py (+3/-3)
To merge this branch: bzr merge lp://qastaging/~dominique-ramaekers/command-not-found/thinerBug
Reviewer Review Type Date Requested Status
Michael Vogt (community) Approve
Review via email: mp+312081@code.qastaging.launchpad.net

Description of the change

Fixes Bug #1643167: command-not-found crash on inreasonably long input

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

Looks good, thank you!

review: Approve
Revision history for this message
Michael Vogt (mvo) wrote :

This is now part of http://bazaar.launchpad.net/~ubuntu-core-dev/command-not-found/ubuntu/revision/199 - this is the best branch to target for the command-not-found work, the lp:command-not-found is pointing to something that is not quite ready (trunk). I will upload it soon.

Revision history for this message
Dominique Ramaekers (dominique-ramaekers) wrote :

Thank you to. I won't bug you so much anymore. I'm glad I now can
contribute after years of using OSS. And on a tool that helped me a lot and
still is of good use for me.

Greetings,

Dominique.

Op dinsdag 13 december 2016 07:53:21 CET schreef Michael Vogt
<email address hidden>:
> Review: Approve
>
> Looks good, thank you!

--
Verzonden via Dekko vanaf mijn Ubuntu-apparaat

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches