)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"9d7e03b8a75852c9710776c59ad2b94f01e6feba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7d297f49_43a8ba8a","updated":"2026-03-18 16:48:32.000000000","message":"\u003e I\u0027m leaning toward making it policy that all logging goes to stderr, and only intentional output\n\nI\u0027m happy with this approach and left a comment about how that will probably impact other parts of the code inline.\n\nDo we want to split this into two changes if we do that though? We can return 2 and shift output file location separately. Though I don\u0027t feel strongly about needing to do that here. One change is probably fine.","commit_id":"0dedbca10a8d4e27ce0fb389892e19363f9fa345"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"3571a8c0cd1680c25614a458570ac5f88f312c9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a66d51a4_10e0613e","updated":"2026-03-18 19:07:01.000000000","message":"Sorry for the previous email reply... assumed gerrit would handle that.\n\nAdded the reno release note and modified the README.rst to contain the return codes, as suggested, in the latest patchset.","commit_id":"9a97635911ca700790518f59620c55a4939901bd"}],"bindep/depends.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"9d7e03b8a75852c9710776c59ad2b94f01e6feba","unresolved":true,"context_lines":[{"line_number":264,"context_line":"        if output_format \u003d\u003d \u0027csv\u0027:"},{"line_number":265,"context_line":"            logging.info(\u0027,\u0027.join(packages_list))"},{"line_number":266,"context_line":"        elif output_format \u003d\u003d \u0027newline\u0027:"},{"line_number":267,"context_line":"            logging.info(\u0027\\n\u0027.join(packages_list))"},{"line_number":268,"context_line":"        return packages_list"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def check_rules(self, rules):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ea280045_9af83b3c","line":267,"updated":"2026-03-18 16:48:32.000000000","message":"If we use fungi\u0027s suggestion that all logging go to stderr and the package listing goes to stdout then we can probably update this to use print(). Then we would have to update some of the test cases that check for output using logger capturing. They would need to capture stdout instead. But that should be doable.","commit_id":"0dedbca10a8d4e27ce0fb389892e19363f9fa345"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"c83f7366ddf6bad708c9ed3463414def00f8af7d","unresolved":true,"context_lines":[{"line_number":264,"context_line":"        if output_format \u003d\u003d \u0027csv\u0027:"},{"line_number":265,"context_line":"            logging.info(\u0027,\u0027.join(packages_list))"},{"line_number":266,"context_line":"        elif output_format \u003d\u003d \u0027newline\u0027:"},{"line_number":267,"context_line":"            logging.info(\u0027\\n\u0027.join(packages_list))"},{"line_number":268,"context_line":"        return packages_list"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def check_rules(self, rules):"}],"source_content_type":"text/x-python","patch_set":1,"id":"4b8c119c_87fb5d0d","line":267,"in_reply_to":"ea280045_9af83b3c","updated":"2026-03-18 19:22:03.000000000","message":"Seems fungi reversed course on this one. Could also be done in a separate change if reconsidered later.","commit_id":"0dedbca10a8d4e27ce0fb389892e19363f9fa345"}]}