)]}'
{"files/rpms/ironic":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1cabe85becc3ce60580f822a59df37388d656f92","unresolved":false,"context_lines":[{"line_number":13,"context_line":"syslinux"},{"line_number":14,"context_line":"tftp-server"},{"line_number":15,"context_line":"xinetd"},{"line_number":16,"context_line":"redhat-rpm-config"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"ba7be1f8_6cd9cc82","line":16,"updated":"2015-02-23 23:38:03.000000000","message":"Yeah, guessing this is a work-around for the mysql issues fixed by I731828cac912d1b11eaf2269364411c77a57d76c ?\n\nother bit looks fine","commit_id":"1847030ec3f41f4bd8958d5b5ac9a27a8edc8a74"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"d163213cf500c30a5bc83e0532e8928d6e2100eb","unresolved":false,"context_lines":[{"line_number":13,"context_line":"syslinux"},{"line_number":14,"context_line":"tftp-server"},{"line_number":15,"context_line":"xinetd"},{"line_number":16,"context_line":"redhat-rpm-config"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"ba7be1f8_60cf16b1","line":16,"updated":"2015-02-23 20:42:39.000000000","message":"https://review.openstack.org/#/c/158255/","commit_id":"1847030ec3f41f4bd8958d5b5ac9a27a8edc8a74"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5b9367e8d4a52f0155ea39210a17dc993c0dee43","unresolved":false,"context_lines":[{"line_number":13,"context_line":"syslinux"},{"line_number":14,"context_line":"tftp-server"},{"line_number":15,"context_line":"xinetd"},{"line_number":16,"context_line":"redhat-rpm-config"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"ba7be1f8_07790993","line":16,"in_reply_to":"ba7be1f8_6cd9cc82","updated":"2015-02-24 00:04:23.000000000","message":"Yes, that was it :)\n\nThanks for pointing out that it has been fixed somewhere else. I will remove it from this patch.","commit_id":"1847030ec3f41f4bd8958d5b5ac9a27a8edc8a74"}],"lib/ironic":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"38445989161efe9fb4ee6625c8609966fcccd0bd","unresolved":false,"context_lines":[{"line_number":242,"context_line":"    # binary is not \"stand-alone\" anymore, it depends on some c32 modules"},{"line_number":243,"context_line":"    # to work correctly."},{"line_number":244,"context_line":"    # More info: http://www.syslinux.org/wiki/index.php/Library_modules"},{"line_number":245,"context_line":"    if is_fedora \u0026\u0026 [[ \"$IRONIC_IPXE_ENABLED\" \u003d\u003d \"False\" ]] ; then"},{"line_number":246,"context_line":"        cp -aR $(dirname $IRONIC_PXE_BOOT_IMAGE)/*.{c32,0} $IRONIC_TFTPBOOT_DIR"},{"line_number":247,"context_line":"    else"},{"line_number":248,"context_line":"        cp $IRONIC_PXE_BOOT_IMAGE $IRONIC_TFTPBOOT_DIR"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba7be1f8_9128739b","line":245,"updated":"2015-02-25 20:32:10.000000000","message":"does this apply to rhel7 too?  the comment implies f21+ only but is_fedora is also true for rhel","commit_id":"dbfde3a60f84edfca41ff8dfa1b56e642abc609a"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"abce277b80903b87fb5c5467fca688f036e0b792","unresolved":false,"context_lines":[{"line_number":242,"context_line":"    # binary is not \"stand-alone\" anymore, it depends on some c32 modules"},{"line_number":243,"context_line":"    # to work correctly."},{"line_number":244,"context_line":"    # More info: http://www.syslinux.org/wiki/index.php/Library_modules"},{"line_number":245,"context_line":"    if is_fedora \u0026\u0026 [[ \"$IRONIC_IPXE_ENABLED\" \u003d\u003d \"False\" ]] ; then"},{"line_number":246,"context_line":"        cp -aR $(dirname $IRONIC_PXE_BOOT_IMAGE)/*.{c32,0} $IRONIC_TFTPBOOT_DIR"},{"line_number":247,"context_line":"    else"},{"line_number":248,"context_line":"        cp $IRONIC_PXE_BOOT_IMAGE $IRONIC_TFTPBOOT_DIR"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"ba7be1f8_166b80b7","line":245,"in_reply_to":"ba7be1f8_9128739b","updated":"2015-02-26 09:52:48.000000000","message":"It\u0027s not required for rhel7 at present.\n\nBut as azafekas comment, I could copy the c32 modules for all distros. I will update the patch to do that.","commit_id":"dbfde3a60f84edfca41ff8dfa1b56e642abc609a"}]}