Hi Gian, > I just created a new merge request for BE > > The patch just add the bug summary to the output of the "be > depend" command.
(It'd be good to send a copy of the patch itself for review alongside small requests like this one.) Review comments: * There's another place where we print the "blocked by:" string in the same source file, and you haven't modified that one. (Line 156.) * You've changed the indentation of one of the modified lines, removing an indent. I'd understand if this enabled the code to stay within 80 chars, but it doesn't. (Line 188.) * You haven't changed the text for "blocks:", only "blocked by:". (Line 191.) Thanks, - Chris. -- Chris Ball <[email protected]> One Laptop Per Child _______________________________________________ Be-devel mailing list [email protected] http://void.printf.net/cgi-bin/mailman/listinfo/be-devel
