Code review comment for lp:~bladernr/checkbox/1066118-modinfo-parser

Revision history for this message
Jeff Lane  (bladernr) wrote :

Possibly, but modinfo returns very specific output. Unfortunately, without access to the failing machine this was impossible to recreate. Sylvain and I both guessed blank lines, but I have never seen modinfo return a blank line before, ever. My original suspicion was the empty "Depends" field in modinfo output, but that splits out into ('depends','') which is fine.

But your point is well taken, we do need more unit tests for these things :)

« Back to merge proposal