diff options
author | Joey Hess <joeyh@joeyh.name> | 2018-02-16 13:47:10 -0400 |
---|---|---|
committer | Joey Hess <joeyh@joeyh.name> | 2018-02-16 13:47:10 -0400 |
commit | 6e37cbeb17fdfc8e1d09e5130d6bb49e91398f98 (patch) | |
tree | 827c919af5a6e78b9aebc201467272843a4b5581 | |
parent | 8dc9b5d6269e89466b87db0bcf75893ef68e2c0a (diff) |
comment
-rw-r--r-- | doc/todo/include_msg_with_possible_reason_why_command___40__e.g._add__41___failed_into_--json_output/comment_4_c2e5041a07787ae84f85b898b1aae6b7._comment | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/doc/todo/include_msg_with_possible_reason_why_command___40__e.g._add__41___failed_into_--json_output/comment_4_c2e5041a07787ae84f85b898b1aae6b7._comment b/doc/todo/include_msg_with_possible_reason_why_command___40__e.g._add__41___failed_into_--json_output/comment_4_c2e5041a07787ae84f85b898b1aae6b7._comment new file mode 100644 index 000000000..f4d171b01 --- /dev/null +++ b/doc/todo/include_msg_with_possible_reason_why_command___40__e.g._add__41___failed_into_--json_output/comment_4_c2e5041a07787ae84f85b898b1aae6b7._comment @@ -0,0 +1,19 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 4""" + date="2018-02-16T17:29:56Z" + content=""" +Currently when there's an exception in processing a file, the error is thrown +and then caught and output to stderr, and this prevents the accumulated +json buffer for the file from being output, since the processing never +finishes. So, you get some error message on stderr, and no indication +in the json what file it belongs to. + +So, perhaps an easier fix would be to emit the json buffer in this case +after the error message. Then stderr output, whether error or warning, +would always precede the json for the same file. + +The consumer would need to use select() over stdin+stderr to observe +the order they were interleaved. May be too much to expect consumers +to get that right. +"""]] |