{"push_id":2984237663,"size":6,"distinct_size":0,"ref":"refs/heads/format-no-error","head":"88106f86c3f84664637663860c6a60ad3532421c","before":"1ee6e4a3a06334f1321c0a814e107b9802e5a46d","commits":[{"sha":"aa67fda7375e3f4b44f1b72881ef1ed95b68a54a","author":{"name":"Rodrigo Chiossi","email":"16e84ec027f969370958c48a4d32dbff338c878c@intel.com"},"message":"build: Handle error on buffer write fail\n\nWhen writing config files or buffers with configuration, mixer should\nfail right away if an error occurs. This provides the user with better\nerrors since it prevents mixer from failing on config usage which\nyields unrelated errors.\n\nSigned-off-by: Rodrigo Chiossi <rodrigo.chiossi@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/aa67fda7375e3f4b44f1b72881ef1ed95b68a54a"},{"sha":"fdb82882980dff79acdf46b8303edf1e4e619dad","author":{"name":"Rodrigo Chiossi","email":"16e84ec027f969370958c48a4d32dbff338c878c@intel.com"},"message":"builder: Handle write error on Stopwatch buffer\n\nUse the logger interface to write stopwatch output. With this change,\nany error that occurs while trying to write the output is handled\ninternally by the log module.\n\nThis patch also fix a bug where the line break was printed to STDOUT\ninstead of the stopwatch buffer.\n\nSigned-off-by: Rodrigo Chiossi <rodrigo.chiossi@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/fdb82882980dff79acdf46b8303edf1e4e619dad"},{"sha":"4c3f790d77e5128c4e894ce12efbcce213a9753c","author":{"name":"Rodrigo Chiossi","email":"16e84ec027f969370958c48a4d32dbff338c878c@intel.com"},"message":"helpers: Handle error on output buffer write fail\n\nIf an error occurred when running an external program, the user should\nreceive a full report of that program's output. If mixer is unable to\nwrite that output, the user should get and error indicating the problem\ninstead of an incomplete output, making it easier to debug the problem.\n\nSigned-off-by: Rodrigo Chiossi <rodrigo.chiossi@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/4c3f790d77e5128c4e894ce12efbcce213a9753c"},{"sha":"70a3f68c580c9da94491ad03f326dd930193c519","author":{"name":"Rodrigo Chiossi","email":"16e84ec027f969370958c48a4d32dbff338c878c@intel.com"},"message":"global: Use log package for Warnings and Errors\n\nThis patch changes how Warnings and Error messages are printed\nthroughout the code to use the `log` package instead of `fmt`. This\nallows for finer control over what is printed and where it is printed\nto.\n\nSigned-off-by: Rodrigo Chiossi <rodrigo.chiossi@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/70a3f68c580c9da94491ad03f326dd930193c519"},{"sha":"c8de3b751b679772bae95b2c5e3cd0dc9b9e705b","author":{"name":"Matthew Johnson","email":"59a0a90a545f8febd0a03a60d2822309ae36580d@intel.com"},"message":"swupd: add contentsize hack to format switch\n\nThe contentsize hack exists to work around a bug in swupd-client\nv3.15.3 which was released during format25 in Clear Linux. Because mixer\nis maintaining backwards compatibility for old formats we can no longer\nremove this hack when a new format bump happens in the OS. Instead we\nneed to apply this arbitrary maximum whenever mixer is building for the\nbad format.\n\nSince the manifest_format file no longer only switches for manifest\ntemplates it was necessary to rename the file and associated _test.go\nfile to format_switch.go and format_switch_test.go, respectively.\n\nAdd testing for this functionality.\n\nSigned-off-by: Matthew Johnson <matthew.johnson@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/c8de3b751b679772bae95b2c5e3cd0dc9b9e705b"},{"sha":"88106f86c3f84664637663860c6a60ad3532421c","author":{"name":"Matthew Johnson","email":"59a0a90a545f8febd0a03a60d2822309ae36580d@intel.com"},"message":"swupd/format_switch: do not fail on 0 format\n\nIt is not the responsibility of the format_switch to check for non-zero\nformat, as this is an internal function and validation on the format\nhappens well before this function is called. It simplifies the caller so\nit doesn't have to handle an error that will never occur and simplifies\nthe function itself so it doesn't have to explicitly check for\nformat == 0.\n\nSigned-off-by: Matthew Johnson <matthew.johnson@intel.com>","distinct":false,"url":"https://api.github.com/repos/matthewrsj/mixer-tools/commits/88106f86c3f84664637663860c6a60ad3532421c"}]}
{"actor":{"display_login":"matthewrsj"}}