)]}'
{"openvswitch/Dockerfile.ubuntu_bionic-dpdk":[{"author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"change_message_id":"b99b4cd6cfc1b509d9d4c73b0db29804b84ffae4","unresolved":false,"context_lines":[{"line_number":33,"context_line":"RUN set -ex;\\"},{"line_number":34,"context_line":"    rm -rf Dockerfile.ubuntu_bionic-dpdk ;\\"},{"line_number":35,"context_line":"    apt-get update; \\"},{"line_number":36,"context_line":"    if [ -n \"$OVS_SRC_COMPILE_VERSION\" -a -n \"$DPDK_SRC_COMPILE_VERSION\" ]; then \\"},{"line_number":37,"context_line":"      if [ -n \"$OVS_PATCH_FILE_NAME\" ]; then \\"},{"line_number":38,"context_line":"        OVS_PATCH_TMP_DIR\u003d$(mktemp -d) ;\\"},{"line_number":39,"context_line":"        mv ./${OVS_PATCH_FILE_NAME} ${OVS_PATCH_TMP_DIR}/${OVS_PATCH_FILE_NAME} ;\\"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"3fa7e38b_89f35273","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":74},"updated":"2019-11-20 03:18:05.000000000","message":"Given this logic, you are changing the default build from distributed pkg to source build?\n\nIt\u0027s good to set the default source code version for pkg build. But I don\u0027t think it\u0027s a good idea to make the source build as the default behave.","commit_id":"c200136f4412646d04b5b772bd111ea287a4cf26"},{"author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"change_message_id":"8e6df167f00bb72026df0616a0848814a8ebfd37","unresolved":false,"context_lines":[{"line_number":33,"context_line":"RUN set -ex;\\"},{"line_number":34,"context_line":"    rm -rf Dockerfile.ubuntu_bionic-dpdk ;\\"},{"line_number":35,"context_line":"    apt-get update; \\"},{"line_number":36,"context_line":"    if [ -n \"$OVS_SRC_COMPILE_VERSION\" -a -n \"$DPDK_SRC_COMPILE_VERSION\" ]; then \\"},{"line_number":37,"context_line":"      if [ -n \"$OVS_PATCH_FILE_NAME\" ]; then \\"},{"line_number":38,"context_line":"        OVS_PATCH_TMP_DIR\u003d$(mktemp -d) ;\\"},{"line_number":39,"context_line":"        mv ./${OVS_PATCH_FILE_NAME} ${OVS_PATCH_TMP_DIR}/${OVS_PATCH_FILE_NAME} ;\\"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"3fa7e38b_b15ea9e6","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":74},"in_reply_to":"3fa7e38b_89f35273","updated":"2019-11-20 16:47:24.000000000","message":"I agree with your idea in general, just that in this case, I was specifically targetting functionality needed from the new ovs version and dpdk version, so I am not sure if I am ready to push to have those as the default for all packages.  Is there a path of how and when those are updated?  or is there a better way this can be done perhaps?  I am open to suggestion.","commit_id":"c200136f4412646d04b5b772bd111ea287a4cf26"}]}