3352 2012-10-29 17:21:12 +0000 Improve smart error/status reporting 2013-03-21 14:13:19 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 1.4 All Multiple VERIFIED FIXED Medium+ enhancement 1.4 M5 1 bluelightning bogdan.a.marinescu alexandru.damian alexandrux.palalau jessica.zhang mark.hatle meta.mr.watcher meta.watcher sgw song.liu --- --- --- oldest_to_newest 27015 0 bluelightning 2012-10-29 17:21:12 +0000 The smart utility's output leaves something to be desired. For example: * smart doesn't seem to produce an error on invalid use of commands e.g. smart config name=value (should be smart config --set name=value in this case) * --log-level=debug doesn't seem to actually print any additional information * Blank lines seem to be printed a lot when ideally the output would be a bit more compact This is a little open-ended and probably will involve some cooperation with upstream. We can sit down and document more of the undesirable reporting situations first if needed. (This is less important than the rest of the smart work however and should not hold that back.) 28341 1 bogdan.a.marinescu 2012-12-18 08:56:11 +0000 I uploaded a patch that addresses the first of the issues mentioned by Paul. The second one needs a bit of discussion. What kind of additional information would be needed in this case? I don't want to add too much information in the output. 29038 2 bogdan.a.marinescu 2013-01-22 14:39:40 +0000 Swithing to NEEDINFO (see my previous question). 29084 3 bluelightning 2013-01-23 14:24:44 +0000 It's hard to specify exactly what should be in the debug output, but a very good start would be to ensure RPM's own debugging output gets printed when --log-level=debug is specified. 29812 4 jessica.zhang 2013-02-14 15:57:47 +0000 Bogdan, Please see Paul's last comments 30443 5 bogdan.a.marinescu 2013-03-05 12:28:23 +0000 The general consensus seems to be that this bug is already fixed and additional bugs should be opened for more improvements in the areas of error reporting/improved debug information, so I'm going to close this one. 30969 6 alexandrux.palalau 2013-03-21 14:13:19 +0000 Verified on master branch. Commit ID: a317cc911abe33d0f513d48e7bc4afd319c7e040