)]}'
{"elements/mariadb-dev/pre-install.d/05-mysql-mariadb-repo":[{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"4c9059f790bc8b9a07ec608ca5798ae7e24c8900","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    apt-get update"},{"line_number":36,"context_line":"}"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"if [[ \"rhel fedora\" \u003d~ \"$DISTRO_NAME\" ]]; then"},{"line_number":39,"context_line":"    add_yum_repo"},{"line_number":40,"context_line":"elif [[ \"ubuntu debian\" \u003d~ \"$DISTRO_NAME\" ]]; then"},{"line_number":41,"context_line":"    add_apt_repo"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"1ae5cdf2_23e6c0b0","line":38,"updated":"2014-06-11 13:52:08.000000000","message":"$DISTRO_NAME for the rhel7 element is rhel7, so i\u0027m pretty sure this comparison is likely to fail here, losing rhel7 support.\n\n [jslagle@teletran-1 elements]$ DISTRO_NAME\u003drhel7\n [jslagle@teletran-1 elements]$ if [[ \"rhel fedora\" \u003d~ \"$DISTRO_NAME\" ]]; then echo yes; else echo no; fi\n no\n\n\nI\u0027m also a bit hesistant to drop CentOS support, although given no centos element exists, I suppose that\u0027s less of an issue.","commit_id":"940d5471f668efd697369cde4d03b2d7646602b6"},{"author":{"_account_id":9268,"name":"Gonéri Le Bouder","email":"goneri@redhat.com","username":"goneri"},"change_message_id":"67c270745415373ea00844b5cc23156248ef744c","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    apt-get update"},{"line_number":36,"context_line":"}"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"if [[ \"rhel fedora\" \u003d~ \"$DISTRO_NAME\" ]]; then"},{"line_number":39,"context_line":"    add_yum_repo"},{"line_number":40,"context_line":"elif [[ \"ubuntu debian\" \u003d~ \"$DISTRO_NAME\" ]]; then"},{"line_number":41,"context_line":"    add_apt_repo"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"1ae5cdf2_0142803d","line":38,"in_reply_to":"1ae5cdf2_23e6c0b0","updated":"2014-06-12 07:39:38.000000000","message":"I restored rhel7 support. Thanks for pointing the issue. Since there is no CentOS root element, I was thinking the code was not used at all. If someone introduces such CentOS element, it will still be possible to add a new entry here. That\u0027s just my opinion, I can revert the changes on this file if you disagree :)","commit_id":"940d5471f668efd697369cde4d03b2d7646602b6"}]}