)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"daaf63fcf874f69fbc81855bbd75ff5ce4d4d2d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"207dc410_1fb5450b","updated":"2022-06-09 17:56:23.000000000","message":"Please update the tox.ini file so that pylint for sw-patch will also include your new folder\n\nie:\n [testenv:pylint]\ncommands \u003d pylint cgcs_patch cgcs_make_patch --rcfile\u003d./pylint.rc\n\n","commit_id":"156ae868358b51c67afa4ea5a834d6e433b420ce"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"86fbbb0e4ba2f74359d628eae2da37116472d7a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"af56ea2d_8b1ae277","updated":"2022-06-09 18:00:27.000000000","message":"There are 2 tox.ini files that would need to be updated\n\nfor flake8,  and is done by adding a line here for the new sub-dir\nhttps://github.com/starlingx/update/blob/master/tox.ini#L107\n\nfor pylint, its located at \nhttps://github.com/starlingx/update/blob/master/sw-patch/cgcs-patch/tox.ini#L112\n\nand can be done using the syntax from my last comment\n","commit_id":"156ae868358b51c67afa4ea5a834d6e433b420ce"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"e947a320d3285e7b3e350370a1f2cc909d7593a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f9c8114c_41d63b0d","in_reply_to":"207dc410_1fb5450b","updated":"2022-06-13 16:52:53.000000000","message":"Done","commit_id":"156ae868358b51c67afa4ea5a834d6e433b420ce"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"e947a320d3285e7b3e350370a1f2cc909d7593a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8c24b707_0af536d5","in_reply_to":"af56ea2d_8b1ae277","updated":"2022-06-13 16:52:53.000000000","message":"Thanks, I updated the 2 files.","commit_id":"156ae868358b51c67afa4ea5a834d6e433b420ce"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"52c8ccf1ee013d58eea88f5ca591c84518d88dc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d9d2562c_717f5e80","updated":"2022-06-14 15:57:36.000000000","message":"Just the one typo (Unknow)","commit_id":"364c85028b30d59475090eb21fd84a28d06d2d73"}],"sw-patch/cgcs-patch/cgcs_make_patch/make_patch.py":[{"author":{"_account_id":31484,"name":"Jessica Castelino","email":"jessica.castelino@windriver.com","username":"jcasteli"},"change_message_id":"d301a90f2b99e4f01328da407b2cbe243257ca63","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/python3"},{"line_number":2,"context_line":"\"\"\""},{"line_number":3,"context_line":"Creates Debian patches"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"65d7123d_f79f14bb","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":1},"updated":"2022-06-08 20:19:13.000000000","message":"Add copyright to all files","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"0197d0ea5df7b07e52be05d8ebf74f6a9553f7a0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/python3"},{"line_number":2,"context_line":"\"\"\""},{"line_number":3,"context_line":"Creates Debian patches"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bfaf537c_daf47e5c","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":1},"in_reply_to":"65d7123d_f79f14bb","updated":"2022-06-08 20:49:30.000000000","message":"Ack","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"52c8ccf1ee013d58eea88f5ca591c84518d88dc5","unresolved":true,"context_lines":[{"line_number":121,"context_line":"            if child.tag \u003d\u003d \"METADATA\":"},{"line_number":122,"context_line":"                self.__parse_metadata(child)"},{"line_number":123,"context_line":"            else:"},{"line_number":124,"context_line":"                msg \u003d f\"Unknow tag \u0027{child.tag}\u0027 under \u003cPATCH_RECIPE\u003e\""},{"line_number":125,"context_line":"                log.exception(msg)"},{"line_number":126,"context_line":"                raise PatchRecipeXMLFail(msg)"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"6c941806_170337e6","line":124,"range":{"start_line":124,"start_character":24,"end_line":124,"end_character":31},"updated":"2022-06-14 15:57:36.000000000","message":"Unknown","commit_id":"364c85028b30d59475090eb21fd84a28d06d2d73"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"8ca457246bd8f0c6c74f0bdbc9bb2628ecf002ce","unresolved":false,"context_lines":[{"line_number":121,"context_line":"            if child.tag \u003d\u003d \"METADATA\":"},{"line_number":122,"context_line":"                self.__parse_metadata(child)"},{"line_number":123,"context_line":"            else:"},{"line_number":124,"context_line":"                msg \u003d f\"Unknow tag \u0027{child.tag}\u0027 under \u003cPATCH_RECIPE\u003e\""},{"line_number":125,"context_line":"                log.exception(msg)"},{"line_number":126,"context_line":"                raise PatchRecipeXMLFail(msg)"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"387f52fe_787d7b8d","line":124,"range":{"start_line":124,"start_character":24,"end_line":124,"end_character":31},"in_reply_to":"6c941806_170337e6","updated":"2022-06-14 16:49:52.000000000","message":"Ack","commit_id":"364c85028b30d59475090eb21fd84a28d06d2d73"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"52c8ccf1ee013d58eea88f5ca591c84518d88dc5","unresolved":true,"context_lines":[{"line_number":459,"context_line":"    Create a patch"},{"line_number":460,"context_line":"    \"\"\""},{"line_number":461,"context_line":"    log.info(\"Creating patch\")"},{"line_number":462,"context_line":"    if os.path.isfile(args.patch_recipe):"},{"line_number":463,"context_line":"        patch_data \u003d PatchRecipeData()"},{"line_number":464,"context_line":"        patch_data.parse_xml(args.patch_recipe)"},{"line_number":465,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":10,"id":"4e5340dd_ec8d30ba","line":462,"range":{"start_line":462,"start_character":22,"end_line":462,"end_character":26},"updated":"2022-06-14 15:57:36.000000000","message":"I guess this is fine.\nThis means that handle_create and handle_prepare can only ever be called from the main method below, otherwise args is undefined, however that main method is the only place the arg parsing is done anyways","commit_id":"364c85028b30d59475090eb21fd84a28d06d2d73"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"8ca457246bd8f0c6c74f0bdbc9bb2628ecf002ce","unresolved":false,"context_lines":[{"line_number":459,"context_line":"    Create a patch"},{"line_number":460,"context_line":"    \"\"\""},{"line_number":461,"context_line":"    log.info(\"Creating patch\")"},{"line_number":462,"context_line":"    if os.path.isfile(args.patch_recipe):"},{"line_number":463,"context_line":"        patch_data \u003d PatchRecipeData()"},{"line_number":464,"context_line":"        patch_data.parse_xml(args.patch_recipe)"},{"line_number":465,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":10,"id":"ce32dacd_9cd63a78","line":462,"range":{"start_line":462,"start_character":22,"end_line":462,"end_character":26},"in_reply_to":"4e5340dd_ec8d30ba","updated":"2022-06-14 16:49:52.000000000","message":"I added it as parameter to both functions in case we want to reuse it in the future.","commit_id":"364c85028b30d59475090eb21fd84a28d06d2d73"}],"sw-patch/cgcs-patch/cgcs_make_patch/patch_recipe_insvc_sample.xml":[{"author":{"_account_id":31484,"name":"Jessica Castelino","email":"jessica.castelino@windriver.com","username":"jcasteli"},"change_message_id":"d301a90f2b99e4f01328da407b2cbe243257ca63","unresolved":true,"context_lines":[{"line_number":1,"context_line":"\u003cPATCH_RECIPE\u003e"},{"line_number":2,"context_line":"    \u003cMETADATA\u003e"},{"line_number":3,"context_line":"        \u003cID\u003eDEBIAN_RR\u003c/ID\u003e"},{"line_number":4,"context_line":"        \u003cSW_VERSION\u003e00.00\u003c/SW_VERSION\u003e"},{"line_number":5,"context_line":"        \u003cSUMMARY\u003eDebian patch test\u003c/SUMMARY\u003e"},{"line_number":6,"context_line":"        \u003cDESCRIPTION\u003e"}],"source_content_type":"application/xml","patch_set":2,"id":"594e9e94_b1af306a","line":3,"range":{"start_line":3,"start_character":12,"end_line":3,"end_character":21},"updated":"2022-06-08 20:19:13.000000000","message":"Should be DEBIAN_INSVC","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"0197d0ea5df7b07e52be05d8ebf74f6a9553f7a0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"\u003cPATCH_RECIPE\u003e"},{"line_number":2,"context_line":"    \u003cMETADATA\u003e"},{"line_number":3,"context_line":"        \u003cID\u003eDEBIAN_RR\u003c/ID\u003e"},{"line_number":4,"context_line":"        \u003cSW_VERSION\u003e00.00\u003c/SW_VERSION\u003e"},{"line_number":5,"context_line":"        \u003cSUMMARY\u003eDebian patch test\u003c/SUMMARY\u003e"},{"line_number":6,"context_line":"        \u003cDESCRIPTION\u003e"}],"source_content_type":"application/xml","patch_set":2,"id":"7fbe1631_44ea1032","line":3,"range":{"start_line":3,"start_character":12,"end_line":3,"end_character":21},"in_reply_to":"594e9e94_b1af306a","updated":"2022-06-08 20:49:30.000000000","message":"Ack","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"}],"sw-patch/cgcs-patch/cgcs_make_patch/patch_recipe_rr_sample.xml":[{"author":{"_account_id":31484,"name":"Jessica Castelino","email":"jessica.castelino@windriver.com","username":"jcasteli"},"change_message_id":"d301a90f2b99e4f01328da407b2cbe243257ca63","unresolved":true,"context_lines":[{"line_number":1,"context_line":"\u003cPATCH_RECIPE\u003e"},{"line_number":2,"context_line":"    \u003cMETADATA\u003e"},{"line_number":3,"context_line":"        \u003cID\u003eDEBIAN_INSCV\u003c/ID\u003e"},{"line_number":4,"context_line":"        \u003cSW_VERSION\u003e00.00\u003c/SW_VERSION\u003e"},{"line_number":5,"context_line":"        \u003cSUMMARY\u003eDebian patch test\u003c/SUMMARY\u003e"},{"line_number":6,"context_line":"        \u003cDESCRIPTION\u003e"}],"source_content_type":"application/xml","patch_set":2,"id":"79abf4dd_3d100989","line":3,"range":{"start_line":3,"start_character":12,"end_line":3,"end_character":24},"updated":"2022-06-08 20:19:13.000000000","message":"Should be DEBIAN_RR","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"},{"author":{"_account_id":34021,"name":"Luis Sampaio","display_name":"Luis Sampaio","email":"luis.sampaio@windriver.com","username":"lsampaio"},"change_message_id":"0197d0ea5df7b07e52be05d8ebf74f6a9553f7a0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"\u003cPATCH_RECIPE\u003e"},{"line_number":2,"context_line":"    \u003cMETADATA\u003e"},{"line_number":3,"context_line":"        \u003cID\u003eDEBIAN_INSCV\u003c/ID\u003e"},{"line_number":4,"context_line":"        \u003cSW_VERSION\u003e00.00\u003c/SW_VERSION\u003e"},{"line_number":5,"context_line":"        \u003cSUMMARY\u003eDebian patch test\u003c/SUMMARY\u003e"},{"line_number":6,"context_line":"        \u003cDESCRIPTION\u003e"}],"source_content_type":"application/xml","patch_set":2,"id":"6e133d7f_73550ac6","line":3,"range":{"start_line":3,"start_character":12,"end_line":3,"end_character":24},"in_reply_to":"79abf4dd_3d100989","updated":"2022-06-08 20:49:30.000000000","message":"Ack","commit_id":"987827512b185a41b1b6737766e0cc33e17cebdb"}]}
