)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33394,"name":"Mark Asselstine","email":"mark.asselstine@windriver.com","username":"markawr"},"change_message_id":"6cc9cfeec89bf5fa502c09f6466c8f0490df902b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"Because the multi kernel feature wants to install the oot kernel"},{"line_number":11,"context_line":"drivers\u0027 packages for std and rt kernel into one rootfs, error will"},{"line_number":12,"context_line":"be reported if common files exist."},{"line_number":13,"context_line":"So move the common files into a new package named as ice-common."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":" - PASS: Check that there aren\u0027t common files in the driver packages"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c6e907c9_8c57dc85","line":13,"updated":"2022-05-17 16:05:49.000000000","message":"Please expand on this some. What files are conflicting? how are these handled outside of the common files...","commit_id":"65908a6969b0a9b369f01700c421ec842064134b"},{"author":{"_account_id":32832,"name":"Li Zhou","display_name":"Li Zhou","email":"li.zhou@windriver.com","username":"lzhou2"},"change_message_id":"a13f93813224c98c9626cca7d4ee630df95b97ab","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Because the multi kernel feature wants to install the oot kernel"},{"line_number":11,"context_line":"drivers\u0027 packages for std and rt kernel into one rootfs, error will"},{"line_number":12,"context_line":"be reported if common files exist."},{"line_number":13,"context_line":"So move the common files into a new package named as ice-common."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":" - PASS: Check that there aren\u0027t common files in the driver packages"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5dca868c_f7bc9004","line":13,"in_reply_to":"c6e907c9_8c57dc85","updated":"2022-05-18 10:53:28.000000000","message":"Done","commit_id":"65908a6969b0a9b369f01700c421ec842064134b"}],"kernel-modules/intel-ice/debian/deb_folder/rules":[{"author":{"_account_id":33377,"name":"M. Vefa Bicakci","display_name":"Vefa","email":"vefa.bicakci@windriver.com","username":"vbicakci"},"change_message_id":"8ba1f477dc3e35ef38126c0e65e1305dd55420a6","unresolved":true,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"override_dh_auto_install:"},{"line_number":50,"context_line":"\tdh_install src/$(kmod_name).ko /lib/modules/$(kversion)/extra/$(kmod_name)/"},{"line_number":51,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":52,"context_line":"\tinstall pci.updates debian/$(pkg_common_name)/$(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":53,"context_line":"\tinstall README debian/$(pkg_common_name)/$(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":54,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_mandir)/man7/"},{"line_number":55,"context_line":"\tinstall $(kmod_name).7 debian/$(pkg_common_name)/$(_mandir)/man7/"},{"line_number":56,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_sysconfdir)/modules-load.d/"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"83f69753_fa8e08f4","line":53,"range":{"start_line":51,"start_character":84,"end_line":53,"end_character":89},"updated":"2022-05-18 13:23:57.000000000","message":"I think these should also have permissions set to 0644.","commit_id":"e725a63fed7a5639779482d353ca92abb791616e"},{"author":{"_account_id":32832,"name":"Li Zhou","display_name":"Li Zhou","email":"li.zhou@windriver.com","username":"lzhou2"},"change_message_id":"01adc98453e545391429e5795a0d5de78f3377c7","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"override_dh_auto_install:"},{"line_number":50,"context_line":"\tdh_install src/$(kmod_name).ko /lib/modules/$(kversion)/extra/$(kmod_name)/"},{"line_number":51,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":52,"context_line":"\tinstall pci.updates debian/$(pkg_common_name)/$(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":53,"context_line":"\tinstall README debian/$(pkg_common_name)/$(_defaultdocdir)/kmod-$(kmod_name)-$(version)/"},{"line_number":54,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_mandir)/man7/"},{"line_number":55,"context_line":"\tinstall $(kmod_name).7 debian/$(pkg_common_name)/$(_mandir)/man7/"},{"line_number":56,"context_line":"\tdh_installdirs -p$(pkg_common_name) $(_sysconfdir)/modules-load.d/"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"c2cb8a2d_be756363","line":53,"range":{"start_line":51,"start_character":84,"end_line":53,"end_character":89},"in_reply_to":"83f69753_fa8e08f4","updated":"2022-05-19 10:20:45.000000000","message":"Please refer to my explain for iavf.\nHere I add a fix: update the permissions for files from ice_comms when installing\nbecause they got \"755\" from the source code.","commit_id":"e725a63fed7a5639779482d353ca92abb791616e"}]}