)]}'
{"tools/install_prereqs.sh":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f2e27906f85556d7270af69cd46fa39baf548fdb","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    # Do not skip the python-dev* package since it is required for pip"},{"line_number":82,"context_line":"    # package installation."},{"line_number":83,"context_line":"    echo \"Skipping installation of python packages.\""},{"line_number":84,"context_line":"    PACKAGES\u003d$(echo $PACKAGES | perl -pe \u0027s|python-(?!dev)[^ ]*||g\u0027)"},{"line_number":85,"context_line":"fi"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"install_package $PACKAGES"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da86d52c_c004ae4b","line":84,"updated":"2015-02-06 13:33:21.000000000","message":"| egrep -V \u0027^python-\u0027 \n\nmight be better.\n\nThen just add python-dev back in at the end.","commit_id":"f71b2fc033b9d3f3ae2d7db2e362d69a501ee874"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"c7a0cc9108e0bbf6d9f24b9cc4a8b551223db2b3","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    # Do not skip the python-dev* package since it is required for pip"},{"line_number":82,"context_line":"    # package installation."},{"line_number":83,"context_line":"    echo \"Skipping installation of python packages.\""},{"line_number":84,"context_line":"    PACKAGES\u003d$(echo $PACKAGES | perl -pe \u0027s|python-(?!dev)[^ ]*||g\u0027)"},{"line_number":85,"context_line":"fi"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"install_package $PACKAGES"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da86d52c_7eeabe74","line":84,"in_reply_to":"da86d52c_c004ae4b","updated":"2015-02-06 16:02:00.000000000","message":"egrep is a tool for filtering lines, but PACKAGES is a space-separated string.  While I could go through the trouble of converting PACKAGES to lines, it seemed to make more sense to use a tool capable of stream editing like sed or perl.  Perl won out because it supports negative lookahead, but I could reimplement in sed if you\u0027d prefer.  \n\nThe catch with simply adding python-dev back at the end is that the package name is dependent on the platform - it\u0027s python-dev for deb distros, but python-devel for rpm distros.  Between having to conditionally add back the package name (duplicating it from the definition in files/{deb,rpm,suse-rpm}) or retaining it from PACKAGES, I think the latter is preferable.","commit_id":"f71b2fc033b9d3f3ae2d7db2e362d69a501ee874"}]}