)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"d9182262a7f7124b2af0ee1ba1db542874d77cf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b91d7c5c_33f44a6a","updated":"2021-10-13 13:36:04.000000000","message":"Fails build-pkgs.\ndebrepack - INFO - The supported versions are [\u00271.4.59-1\u0027, \u00271.4.53-4+deb10u1\u0027]\n\n\nNote to self: Patches were visually checked, with one expection. Check --with-pcre on the next review.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":32685,"name":"Yue Tao","display_name":"Yue Tao","email":"yue.tao@windriver.com","username":"ytao"},"change_message_id":"5653a726cbf6f3902763617d2e1acfb0baab9be6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0058817b_9a1e8b69","in_reply_to":"b91d7c5c_33f44a6a","updated":"2021-10-14 08:03:09.000000000","message":"I suppose the debrepack is not the latest version. It should no such error in latest version.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"2018727ae8d4992fc338c874bb47142e6ac2bbeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d220eae0_4450c300","updated":"2021-11-05 15:56:44.000000000","message":"Didn\u0027t check if \u0027check-content-legth.patch\u0027 is hard to apply on 1.4.59-1. Assuming it is. We would have gone with 1.4.55 anyway if tpm wasn\u0027t dropped.\nOK here.","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":19124,"name":"Gerry Kopec","email":"gerry.kopec@windriver.com","username":"gkopec"},"change_message_id":"9d116ee0df9297d21c2e504cbb3f194ab8e7d865","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3b4f9972_48ff178b","updated":"2021-11-03 00:07:53.000000000","message":"I was successfully able to build the package in debian.   However on build-image there were a number of unmet dependencies.\nThe following packages have unmet dependencies:\n lighttpd : Depends: libfam0 but it is not installable\n            Depends: mime-support but it is not installable\n lighttpd-mod-geoip : Depends: libgeoip1 (\u003e\u003d 1.6.12) but it is not installable\n lighttpd-mod-trigger-b4-dl : Depends: libmemcached11 but it is not installable\n lighttpd-mod-vhostdb-dbi : Depends: libdbi1 (\u003e\u003d 0.9.0) but it is not installable\n lighttpd-modules-mysql : Depends: libmariadb3 (\u003e\u003d 3.0.0) but it is not installable\nE: Unable to correct problems, you have held broken packages.\n\nHave the dependencies been determined and added to base-bullseye.lst in starlingx/master?","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"b8e769ea8dab379291b8d6d8cec6ae747387b269","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8790bbc0_d6aa8db4","updated":"2021-11-05 15:59:29.000000000","message":"Please raise the base-bullseye.lst review and add a note here, best in the commit meesage.","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":28424,"name":"Frank Miller","email":"frank.miller@eng.windriver.com","username":"sensfan22"},"change_message_id":"05840a68bc3dc4db4f2f20fa43b10660368bdf2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f1fc82bc_50a030b4","updated":"2021-11-05 16:15:46.000000000","message":"This commit is ready to merge. Requesting Cores to review and WFL +1.\nTeam doing Debian packaging has reviewed and approved.\nBuild and test passing on CentOS (boot AIO-SX and unlock controller-0).\nBuild passing on Debian.\n","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":19124,"name":"Gerry Kopec","email":"gerry.kopec@windriver.com","username":"gkopec"},"change_message_id":"d49c873c34a3057e4f3d7757e462bb532dc1ee62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8346ed20_c769e70c","updated":"2021-11-03 21:08:49.000000000","message":"Verified successful package build in debian and no impact on centos via build/install/provision test with AIO-SX in vbox.\n","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":32685,"name":"Yue Tao","display_name":"Yue Tao","email":"yue.tao@windriver.com","username":"ytao"},"change_message_id":"191ee0cc82d0ac5b1ff1ab7fee4a071ff4856ff0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ea934726_6d9b167c","in_reply_to":"3b4f9972_48ff178b","updated":"2021-11-03 02:22:46.000000000","message":"This review focus on the debian folder. Once it has been merged, I will send the second review for updating debian_pkg_dirs and the base-bullseye.lst to enable the package in build-image. \n\nI don\u0027t want to mix them to avoid the merging conflict of any lst.","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"},{"author":{"_account_id":19124,"name":"Gerry Kopec","email":"gerry.kopec@windriver.com","username":"gkopec"},"change_message_id":"d49c873c34a3057e4f3d7757e462bb532dc1ee62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"20ddffc1_666083a5","in_reply_to":"ea934726_6d9b167c","updated":"2021-11-03 21:08:49.000000000","message":"Sounds good.","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"}],"base/lighttpd/debian/deb_patches/0001-lighttpd-backport-spec-include-TiS-changes.patch-fro.patch":[{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"d9182262a7f7124b2af0ee1ba1db542874d77cf3","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  ."},{"line_number":35,"context_line":"  This package contains documentation for lighttpd."},{"line_number":36,"context_line":" "},{"line_number":37,"context_line":"-Package: lighttpd-modules-ldap"},{"line_number":38,"context_line":"-Architecture: any"},{"line_number":39,"context_line":"-Depends:"},{"line_number":40,"context_line":"- ${misc:Depends},"}],"source_content_type":"text/x-diff","patch_set":1,"id":"d4b813de_b7d36e48","line":37,"updated":"2021-10-13 13:36:04.000000000","message":"Add the info about the 3 disabled packages in the commit message. Also add the reason why these 3 are disabled.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":32685,"name":"Yue Tao","display_name":"Yue Tao","email":"yue.tao@windriver.com","username":"ytao"},"change_message_id":"5653a726cbf6f3902763617d2e1acfb0baab9be6","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  ."},{"line_number":35,"context_line":"  This package contains documentation for lighttpd."},{"line_number":36,"context_line":" "},{"line_number":37,"context_line":"-Package: lighttpd-modules-ldap"},{"line_number":38,"context_line":"-Architecture: any"},{"line_number":39,"context_line":"-Depends:"},{"line_number":40,"context_line":"- ${misc:Depends},"}],"source_content_type":"text/x-diff","patch_set":1,"id":"a103d726_ea5243be","line":37,"in_reply_to":"d4b813de_b7d36e48","updated":"2021-10-14 08:03:09.000000000","message":"I will update the commit message.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"}],"base/lighttpd/debian/meta_data.yaml":[{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"d9182262a7f7124b2af0ee1ba1db542874d77cf3","unresolved":true,"context_lines":[{"line_number":1,"context_line":"debver: 1.4.55-1~bpo10+1"},{"line_number":2,"context_line":"debname: lighttpd"},{"line_number":3,"context_line":"archive: http://snapshot.debian.org/archive/debian/20200622T144846Z/pool/main/l/lighttpd/"},{"line_number":4,"context_line":"revision:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5eb94853_9f8075e4","line":1,"range":{"start_line":1,"start_character":8,"end_line":1,"end_character":24},"updated":"2021-10-13 13:36:04.000000000","message":"Is there a reason why 1.4.59-1 which is in bullseye was not chosen?","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":32685,"name":"Yue Tao","display_name":"Yue Tao","email":"yue.tao@windriver.com","username":"ytao"},"change_message_id":"f46b70a5c091fe0e2f2eb6af80a5a45fbb362a47","unresolved":false,"context_lines":[{"line_number":1,"context_line":"debver: 1.4.55-1~bpo10+1"},{"line_number":2,"context_line":"debname: lighttpd"},{"line_number":3,"context_line":"archive: http://snapshot.debian.org/archive/debian/20200622T144846Z/pool/main/l/lighttpd/"},{"line_number":4,"context_line":"revision:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1cc64565_2b4e5f0d","line":1,"range":{"start_line":1,"start_character":8,"end_line":1,"end_character":24},"in_reply_to":"0f25bf5e_6b630ffb","updated":"2021-10-15 06:11:22.000000000","message":"Done","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":32685,"name":"Yue Tao","display_name":"Yue Tao","email":"yue.tao@windriver.com","username":"ytao"},"change_message_id":"5653a726cbf6f3902763617d2e1acfb0baab9be6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"debver: 1.4.55-1~bpo10+1"},{"line_number":2,"context_line":"debname: lighttpd"},{"line_number":3,"context_line":"archive: http://snapshot.debian.org/archive/debian/20200622T144846Z/pool/main/l/lighttpd/"},{"line_number":4,"context_line":"revision:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ab120a25_24c96438","line":1,"range":{"start_line":1,"start_character":8,"end_line":1,"end_character":24},"in_reply_to":"5eb94853_9f8075e4","updated":"2021-10-14 08:03:09.000000000","message":"In order to apply the patch lighttpd-tpm-support.patch. The upstream commit ed62e354ff03eca69a3961b5e1eaf818cbef21f0 refactors the code context in 1.4.59-1. It is difficult to port the patch, so I downgrade the package version. If insist on the 1.4.59-1 version, I recommend just skipping lighttpd-tpm-support.patch, and create a new story to track the tpm feature.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":30852,"name":"Dan Voiculeasa","email":"Dan.Voiculeasa@windriver.com","username":"dvoicule"},"change_message_id":"247cb5f92b2385153f7d23b0b7d976eb9521937b","unresolved":true,"context_lines":[{"line_number":1,"context_line":"debver: 1.4.55-1~bpo10+1"},{"line_number":2,"context_line":"debname: lighttpd"},{"line_number":3,"context_line":"archive: http://snapshot.debian.org/archive/debian/20200622T144846Z/pool/main/l/lighttpd/"},{"line_number":4,"context_line":"revision:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0f25bf5e_6b630ffb","line":1,"range":{"start_line":1,"start_character":8,"end_line":1,"end_character":24},"in_reply_to":"ab120a25_24c96438","updated":"2021-10-14 08:53:28.000000000","message":"Please add this info to the commit message as it provides valuable information.\n\nIn general add information about the decisions made for porting in the commit message.","commit_id":"450b1129ed4d67d5514975faa8e094ec8af37d90"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"939e41130ca32b47b66c851ec32219b9fbe28403","unresolved":true,"context_lines":[{"line_number":1,"context_line":"debver: 1.4.55-1~bpo10+1"},{"line_number":2,"context_line":"debname: lighttpd"},{"line_number":3,"context_line":"dl_path:"},{"line_number":4,"context_line":"  name: lighttpd-debian-1.4.55-1_bpo10+1.tar.gz"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ec741f4d_b218205e","line":1,"range":{"start_line":1,"start_character":16,"end_line":1,"end_character":17},"updated":"2021-11-05 17:45:15.000000000","message":"this looks weird to me, but I guess thats debian","commit_id":"8cf8f7aca33cf15c6a75d77a438b2b748ac1ca06"}]}
