Comment 1 for bug 1124074

Revision history for this message
Michael Terry (mterry) wrote :

-Simple, modern packaging
-No delta
-No test suite
-No symbols file, but it's C++, so that's understandable
-debian/copyright file is a little malformed (missing license stanza), but that's an issue for NEW, not for MIR. Plus, this is from Debian so not worth a delta.
-Would be nice to see a bug subscriber
-This would normally need a security review, since it's a file parser, but since it's been in main, I'll skip that step. Splitting it out actually makes it easier to handle security issues.

*However*, it does have the lintian warning hardening-no-fortify-functions. Can you just check if that warning is a false or true positive? Otherwise, approved.